Errors fixed and symbols checking improved

Author: inc8877

Editor/Unity.DebugModeDefineSymbol.Editor.asmdef

@@ -1,3 +1,16 @@
 {
-	"name": "Unity.DebugModeDefineSymbol.Editor"
-}
+    "name": "Unity.DebugModeDefineSymbol.Editor",
+    "rootNamespace": "",
+    "references": [],
+    "includePlatforms": [
+        "Editor"
+    ],
+    "excludePlatforms": [],
+    "allowUnsafeCode": false,
+    "overrideReferences": false,
+    "precompiledReferences": [],
+    "autoReferenced": true,
+    "defineConstraints": [],
+    "versionDefines": [],
+    "noEngineReferences": false
+}

Editor/UnityDebugModeDefineSymbol.cs

@@ -6,15 +6,13 @@ namespace UnityDebugModeDefineSymbol.Editor
 {
     internal static class DebugModeDefineSymbol
     {
-        private static bool _debugIsOn;
         private const string debugModeSymbol = "DEBUG_MODE_IN_USE";
 
 
         [MenuItem("Tools/Debug Mode Definition/On")]
         internal static void TurnOnDebugModeDefinition()
         {
-            _debugIsOn = GetState(); 
-            if (_debugIsOn)
+            if (DefineDebugModeActivity())
             {
                 EditorUtility.DisplayDialog("Debug mode definition is turned On",
                     "DEBUG_MODE_IN_USE is already included in the scripting define symbols", "OK");
@@ -24,17 +22,15 @@ internal static void TurnOnDebugModeDefinition()
                 if (EditorUtility.DisplayDialog("Turn on the debug mode definition?",
                     "If the debug mode enabled, all the code in the definition 'DEBUG_MODE_IN_USE' will be executed", "Yes", "No"))
                 {
-                    _debugIsOn = true;
                     PlayerPrefs.SetInt("debugModeInUseKey", 1);
-                    DefineSymbols();
+                    DefineSymbols(true);
                 }
             }
         }
         [MenuItem("Tools/Debug Mode Definition/Off")]
         internal static void TurnOffDebugModeDefinition()
         {
-            _debugIsOn = GetState();
-            if (!_debugIsOn)
+            if (!DefineDebugModeActivity())
             {
                 EditorUtility.DisplayDialog("Debug mode definition is already turned off",
                     "The code in the 'DEBUG_MODE_IN_USE' definition is not executed", "OK");
@@ -44,23 +40,30 @@ internal static void TurnOffDebugModeDefinition()
                 if (EditorUtility.DisplayDialog("Turn off the debug mode definition?",
                     "The code in the 'DEBUG_MODE_IN_USE' definition will not be execute", "Yes", "No"))
                 {
-                    _debugIsOn = false;
                     PlayerPrefs.SetInt("debugModeInUseKey", 0);
-                    DefineSymbols();
+                    DefineSymbols(false);
                 }
             }
         }
 
-        private static bool GetState() => PlayerPrefs.GetInt("debugModeInUseKey", 0) == 1;
+        private static bool DefineDebugModeActivity()
+        {
+            BuildTargetGroup currentTarget = EditorUserBuildSettings.selectedBuildTargetGroup;
+        
+            List<string> scriptingSymbols = 
+                new List<string>(PlayerSettings.GetScriptingDefineSymbolsForGroup(currentTarget).Split(';'));
+
+            return scriptingSymbols.Contains(debugModeSymbol);
+        }
 
-        private static void DefineSymbols()
+        private static void DefineSymbols(bool state)
         {
             BuildTargetGroup currentTarget = EditorUserBuildSettings.selectedBuildTargetGroup;
 
             List<string> scriptingSymbols = 
                 new List<string>(PlayerSettings.GetScriptingDefineSymbolsForGroup(currentTarget).Split(';'));
 
-            if (_debugIsOn)
+            if (state)
             {
                 if (!scriptingSymbols.Contains(debugModeSymbol)) scriptingSymbols.Add(debugModeSymbol);
                 else return;