Remove hardcoded passthrough GUID

Replaced PASSTHROUGH_GUID = "e38af599-4575-46ff-a040-459703dbcd36"; with a new property isPassthrough in the environment scriptable object.
icosa-foundation · Dec 19, 2024 · 6b84199 · 6b84199
1 parent 7b9f6a1
commit 6b84199
Show file tree

Hide file tree

Showing 4 changed files with 4 additions and 7 deletions.
diff --git a/Assets/Resources/Environments/Passthrough/Passthrough.asset b/Assets/Resources/Environments/Passthrough/Passthrough.asset
@@ -14,7 +14,6 @@ MonoBehaviour:
   m_EditorClassIdentifier: 
   m_Guid:
     m_storage: e38af599-4575-46ff-a040-459703dbcd36
-  m_Description: ENVIRONMENT_PASSTHROUGH
   m_EnvironmentDescription:
     m_TableReference:
       m_TableCollectionName: GUID:c84355079ab3f3e4f8f3812258805f86
@@ -63,6 +62,7 @@ MonoBehaviour:
   m_WidgetHome: {x: 0, y: 0, z: 0}
   m_SkyboxColorA: {r: 0, g: 0, b: 0, a: 0}
   m_SkyboxColorB: {r: 0, g: 0, b: 0, a: 0}
+  isPassthrough: 1
   references:
     version: 2
     RefIds: []
diff --git a/Assets/Scripts/Environment.cs b/Assets/Scripts/Environment.cs
@@ -175,6 +175,7 @@ public Material m_SkyboxMaterial
 
         public Color m_SkyboxColorA;
         public Color m_SkyboxColorB;
+        public bool isPassthrough;
     }
 
 } // namespace TiltBrush
diff --git a/Assets/Scripts/GUI/LightingPopUpWindow.cs b/Assets/Scripts/GUI/LightingPopUpWindow.cs
@@ -30,7 +30,6 @@ namespace TiltBrush
 
     public class LightingPopUpWindow : PagingPopUpWindow
     {
-        private const string PASSTHROUGH_GUID = "e38af599-4575-46ff-a040-459703dbcd36";
 
         private string m_CurrentPresetGuid;
         [SerializeField] private Transform m_PassthroughControls;
@@ -105,8 +104,7 @@ public void RemovePassthrough()
         {
             foreach (var env in m_Environments)
             {
-                // Passthrough
-                if (env.m_Guid.ToString() == PASSTHROUGH_GUID)
+                if (env.isPassthrough)
                 {
                     m_Environments.Remove(env);
                     break;

diff --git a/Assets/Scripts/GUI/MultiplayerPanel.cs b/Assets/Scripts/GUI/MultiplayerPanel.cs
@@ -298,15 +298,13 @@ private Tuple<bool, string> CheckIfRoomExist()
 
         }
 
-        // hardoded copied from LightingPopUpWindow
-        private const string PASSTHROUGH_GUID = "e38af599-4575-46ff-a040-459703dbcd36";
         private Tuple<bool, string> CheckIfPassThroughEnvironment()
         {
 
             if (MultiplayerManager.m_Instance != null && MultiplayerManager.m_Instance.State == ConnectionState.IN_LOBBY)
             {
                 TiltBrush.Environment targetEnvironment = SceneSettings.m_Instance.GetDesiredPreset();
-                if (targetEnvironment.m_Guid.ToString() == PASSTHROUGH_GUID)
+                if (targetEnvironment.isPassthrough)
                     return Tuple.Create(true, m_AlertsPassThroughAcive.GetLocalizedString());
             }