fix(mobile): moved debug logging to desktop only

iOSonntag · Oct 6, 2023 · f4adaa9 · f4adaa9
1 parent ddb0fb5
commit f4adaa9
Showing 1 changed file with 11 additions and 4 deletions.
diff --git a/src/util/logger.ts b/src/util/logger.ts
@@ -1,11 +1,12 @@
+import { Platform } from 'obsidian';
 import { PluginInfo } from 'src/core/plugin-info';
 
 
 
 export class Log {
 
   /**
-     * Logs only if `process.env.NODE_ENV !== 'production'`.
+     * Logs only if `Platform.isDesktopApp` and if available `process.env.NODE_ENV !== 'production'`.
      * 
      * Adds useful information to the log message and then logs it using `console.log`.
      * 
@@ -14,10 +15,16 @@ export class Log {
      */
   static debug(message: string, ...args: any[])
   {
-    if (process.env.NODE_ENV === 'production') return;
+    if (Platform.isDesktopApp)
+    {
+      // in case it is unknown we assume production, even though it is not correct
+      // it serves the purpose of not logging, when not in dev mode.
 
-
-    console.log(`[DEBUG][${PluginInfo.pluginId}]: ${message}`, ...args);
+      if ((process?.env?.NODE_ENV ?? 'production') !== 'production')
+      {
+        console.log(`[DEBUG][${PluginInfo.pluginId}]: ${message}`, ...args);
+      }
+    }
   }