Handle main camera without parent (#206)

* Clean up some gratuitous logging.

* Fix handling of unexpected case with no parent to scene's main camera.

* Up patch version.

Author: fast-slow-still

Assets/WorldLocking.Core/Scripts/AlignSubtree.cs

@@ -292,7 +292,7 @@ private void CheckInternalWiring()
             if (orienter == null)
             {
                 orienter = GetComponentInChildren<Orienter>();
-                Debug.Log($"No Orienter found on {name}, implicit Orienter found in subtree is {(orienter == null ? "null" : orienter.name)}");
+                Debug.LogWarning($"No Orienter found on {name}, implicit Orienter found in subtree is {(orienter == null ? "null" : orienter.name)}");
             }
             if (Orienter != null)
             {

Assets/WorldLocking.Core/Scripts/Fragment.cs

@@ -168,7 +168,7 @@ public void AdjustAll(IPlugin plugin)
                 }
                 else
                 {
-                    Debug.Log($"No adjustment during refreeze for {attach.AnchorId.FormatStr()}");
+                    Debug.LogWarning($"No adjustment during refreeze for {attach.AnchorId.FormatStr()}");
                 }
 
             }

Assets/WorldLocking.Core/Scripts/SpacePin.cs

@@ -2,6 +2,7 @@
 // Licensed under the MIT License. See LICENSE in the project root for license information.
 
 //#define WLT_LOG_SAVE_LOAD
+//#define WLT_EXTRA_LOGGING
 
 using System;
 using System.Collections;
@@ -200,6 +201,14 @@ private void DebugLogSaveLoad(string message)
             Debug.Log($"F={Time.frameCount} {message}");
 #endif // WLT_LOG_SAVE_LOAD
         }
+
+        private void DebugLogExtra(string message)
+        {
+#if WLT_EXTRA_LOGGING
+            Debug.Log(msg);
+#endif // WLT_EXTRA_LOGGING
+        }
+
         private void CheckDependencies()
         {
             /// Cache the WorldLockingManager as a dependency.
@@ -314,19 +323,19 @@ protected Pose ExtractModelPose()
                 case ModelPositionSourceEnum.Transform:
                     {
                         modelPose = ExtractModelPoseFromTransform();
-                        Debug.Log($"Extracted pose from transform on {name}");
+                        DebugLogExtra($"Extracted pose from transform on {name}");
                     }
                     break;
                 case ModelPositionSourceEnum.RendererBounds:
                     {
                         modelPose = ExtractModelPoseFromRenderer();
-                        Debug.Log($"Extracted pose from renderer on {name}");
+                        DebugLogExtra($"Extracted pose from renderer on {name}");
                     }
                     break;
                 case ModelPositionSourceEnum.ColliderBounds:
                     {
                         modelPose = ExtractModelPoseFromCollider();
-                        Debug.Log($"Extracted pose from collider on {name}");
+                        DebugLogExtra($"Extracted pose from collider on {name}");
                     }
                     break;
                 default:

Assets/WorldLocking.Core/Scripts/WorldLockingManager.cs

@@ -29,7 +29,7 @@ public class WorldLockingManager
         /// allowing quick visual verification of the version of World Locking Tools for Unity currently installed.
         /// It has no effect in code, but serves only as a label.
         /// </summary>
-        public static string Version => "1.5.0";
+        public static string Version => "1.5.1";
 
         /// <summary>
         /// The configuration settings may only be set as a block.
@@ -506,10 +506,10 @@ private void ApplyNewSettings()
                 if (Camera.main != null)
                 {
                     string parentName = Camera.main.transform.parent != null ? Camera.main.transform.parent.name : "null";
-                    Debug.Log($"No camera parent set on WorldLockingManager, using parent {parentName} of scene's main camera.");
+                    Debug.LogWarning($"No camera parent set on WorldLockingManager, using parent {parentName} of scene's main camera.");
                     CameraParent = Camera.main.transform.parent;
                 }
-                else
+                else if (ApplyAdjustment)
                 {
                     Debug.LogError("No CameraParent set on WorldLockingManager, and no main camera to infer parent from.");
                 }
@@ -519,15 +519,15 @@ private void ApplyNewSettings()
             {
                 if (CameraParent != null && CameraParent.parent != null)
                 {
-                    Debug.Log($"No Adjustment Frame set on WorldLockingManager, using Transform {CameraParent.parent.gameObject.name} from scene's main camera hierarchy.");
+                    Debug.LogWarning($"No Adjustment Frame set on WorldLockingManager, using Transform {CameraParent.parent.gameObject.name} from scene's main camera hierarchy.");
                     AdjustmentFrame = CameraParent.parent;
                 }
-                else if (Camera.main != null)
+                else if (CameraParent != null)
                 {
-                    Debug.Log($"No Adjustment Frame set on WorldLockingManager, using root Transform {Camera.main.transform.root.gameObject.name} from scene's main camera.");
-                    AdjustmentFrame = Camera.main.transform.root;
+                    Debug.LogWarning($"No Adjustment Frame set on WorldLockingManager, using root Transform {CameraParent.transform.root.gameObject.name} from scene's main camera.");
+                    AdjustmentFrame = CameraParent.transform.root;
                 }
-                else
+                else if (ApplyAdjustment)
                 {
                     Debug.LogError("No Adjustment Frame set and no main camera to infer node from.");
                 }
@@ -561,7 +561,7 @@ private void Update()
             }
             if (ApplyAdjustment && (AdjustmentFrame == null))
             {
-                Debug.Log("No WLM update because no adjustment frame set");
+                Debug.LogWarning($"F={Time.frameCount}: No WLM update because no adjustment frame set");
                 ErrorStatus = "no adjustment frame";
                 return;
             }