Finished work

Author: MCStreetguy

CHANGELOG.md

@@ -7,7 +7,6 @@
 
 #### Features
 - Debug functionalities *(for easier bug reporting)*
-- Edit option added
 #### Improvements
 - Path resolvement
 #### Patches

README.md

@@ -41,6 +41,29 @@ You will be prompted for the snippet name to delete.
 > "Delete snippet"
 
 
+##### **snippet-injector:toggledebug**
+This command toggles all debugging options for the package.
+Debug informations are logged in Atom's console.
+
+*Please notice that this command is just available through command palette!*
+
+
+##### **snippet-injector:toggle-time-debug**
+This command toggles debugging of function durations.
+Debug informations are logged in Atom's console.
+
+*Name in menus:*
+> "Toggle timing debug"
+
+
+##### **snippet-injector:toggle-object-debug**
+This command toggles debugging of the package state.
+Debug informations are logged in Atom's console.
+
+*Name in menus:*
+> "Toggle object debug"
+
+
 ---
 
 ### External Sources

lib/debug.json

@@ -10,5 +10,8 @@
     "timings": false,
     "objectLogging": false
   },
-  "usrPref": null
-}
+  "usrPref": {
+    "timings": false,
+    "objectLogging": false
+  }
+}

lib/snippet-injector.js

@@ -42,7 +42,10 @@ export default {
     this.subscriptions.add(atom.commands.add('atom-workspace', {
       'snippet-injector:create': () => this.create(),
       'snippet-injector:insert': () => this.insert(),
-      'snippet-injector:delete': () => this.delete()
+      'snippet-injector:delete': () => this.delete(),
+      'snippet-injector:toggle-time-debug': () => this.debugTime(),
+      'snippet-injector:toggle-object-debug': () => this.debugObjects(),
+      'snippet-injector:toggledebug': () => this.debug()
     }));
 
     if(this.debug.allowDebug) {
@@ -76,6 +79,37 @@ export default {
     return new Object();
   },
 
+  debug() {
+    this.debugTime();
+    this.debugObjects();
+  }
+
+  debugTime() {
+    if(this.debug.allowDebug) {
+      this.debug.usrPref.timings = !this.debug.usrPref.timings;
+      if(this.debug.usrPref.timings) {
+        atom.notifications.addInfo('Enabled timings debug.', {detail: 'Changes take effect on next full restart of Atom.'});
+      } else {
+        atom.notifications.addInfo('Disabled timings debug.', {detail: 'Changes take effect on next full restart of Atom.'});
+      }
+    } else {
+      atom.notifications.addWarning('Cannot toggle debug mode since it is disabled.', null);
+    }
+  },
+
+  debugObjects() {
+    if(this.debug.allowDebug) {
+      this.debug.usrPref.objectLogging = !this.debug.usrPref.objectLogging;
+      if(this.debug.usrPref.objectLogging) {
+        atom.notifications.addInfo('Enabled object debug.', {detail: 'Changes take effect on next full restart of Atom.'});
+      } else {
+        atom.notifications.addInfo('Disabled object debug.', {detail: 'Changes take effect on next full restart of Atom.'});
+      }
+    } else {
+      atom.notifications.addWarning('Cannot toggle debug mode since it is disabled.', null);
+    }
+  },
+
   create() {
     console.groupCollapsed("Snippet Injector - DEBUG");
     console.time("snippet-injector:create duration");

lib/storage.js

@@ -70,8 +70,8 @@ export default class Storage {
     }
 
     if(Util.isset(snippet,Snippet)) {
-      fs.writeFileSync(this.storageDir+this.dir+snippet.getUID().split(' ').join('-')+'.json',JSON.stringify(snippet,null,2));
-      var temp = fs.existsSync(this.storageDir+this.dir+snippet.getUID().split(' ').join('-')+'.json');
+      fs.writeFileSync(path.join(this.storageDir,this.dir,snippet.getUID().split(' ').join('-')+'.json'),JSON.stringify(snippet,null,2));
+      var temp = fs.existsSync(path.join(this.storageDir,this.dir,snippet.getUID().split(' ').join('-')+'.json'));
       if(window.snippet_injector_debug.allowDebug && window.snippet_injector_debug.usrPref.timings) {
         console.timeEnd("storage:store duration");
       }
@@ -91,7 +91,7 @@ export default class Storage {
       if(window.snippet_injector_debug.allowDebug && window.snippet_injector_debug.usrPref.timings) {
         console.time("storage:retrieveFiles duration");
       }
-      var temp = fs.readdirSync(this.storageDir+this.dir);
+      var temp = fs.readdirSync(path.join(this.storageDir,this.dir));
       if(window.snippet_injector_debug.allowDebug && window.snippet_injector_debug.usrPref.timings) {
         console.timeEnd("storage:retrieveFiles duration");
       }
@@ -106,7 +106,7 @@ export default class Storage {
       if(window.snippet_injector_debug.allowDebug && window.snippet_injector_debug.usrPref.timings) {
         console.time("storage:retrieveFile duration");
       }
-      var temp = fs.readFileSync(this.storageDir+this.dir+uid.split(' ').join('-')+'.json');
+      var temp = fs.readFileSync(path.join(this.storageDir,this.dir,uid.split(' ').join('-')+'.json'));
       if(window.snippet_injector_debug.allowDebug && window.snippet_injector_debug.usrPref.timings) {
         console.timeEnd("storage:retrieveFile duration");
       }
@@ -135,8 +135,8 @@ export default class Storage {
       if(window.snippet_injector_debug.allowDebug && window.snippet_injector_debug.usrPref.timings) {
         console.time("storage:deleteFile duration");
       }
-      if(fs.existsSync(this.storageDir+this.dir+uid.split(' ').join('-')+'.json')) {
-        fs.unlinkSync(this.storageDir+this.dir+uid.split(' ').join('-')+'.json');
+      if(fs.existsSync(path.join(this.storageDir,this.dir,uid.split(' ').join('-')+'.json'))) {
+        fs.unlinkSync(path.join(this.storageDir,this.dir,uid.split(' ').join('-')+'.json'));
         if(window.snippet_injector_debug.allowDebug && window.snippet_injector_debug.usrPref.timings) {
           console.timeEnd("storage:deleteFile duration");
         }
@@ -166,7 +166,7 @@ export default class Storage {
         var _this = this;
         files.forEach(function(current) {
           if(current.endsWith('.snippet.json')) {
-            fs.renameSync(store+current,store+current.replace('.snippet.json','.json'));
+            fs.renameSync(path.join(store,current),path.join(store,current.replace('.snippet.json','.json')));
             current = current.replace('.snippet.json','');
           } else {
             current = current.replace('.json','');

menus/snippet-injector.json

@@ -39,6 +39,17 @@
             {
               "label": "Delete Snippet",
               "command": "snippet-injector:delete"
+            },
+            {
+              "type": "separator"
+            },
+            {
+              "label": "Toggle timing debug",
+              "command": "snippet-injector:toggle-time-debug"
+            },
+            {
+              "label": "Toggle object debug",
+              "command": "snippet-injector:toggle-object-debug"
             }
           ]
         }