WIP: Migrations

Author: MCStreetguy

lib/snippet-injector.js

@@ -13,8 +13,8 @@ export default {
 
   activate(state) {
     this.storage = new Storage({dir: '/storage/snippet-injector/snippets/'});
-    console.log(this.storage);
-
+    this.storage.migrate();
+    
     // Events subscribed to in atom's system can be easily cleaned up with a CompositeDisposable
     this.subscriptions = new CompositeDisposable();
     // Register command that toggles this view
@@ -23,8 +23,6 @@ export default {
       'snippet-injector:insert': () => this.insert(),
       'snippet-injector:delete': () => this.delete()
     }));
-
-    console.log('Current Version: '+Util.getPackageVersion());
   },
 
   deactivate() {

lib/snippet.js

@@ -8,48 +8,29 @@ export default class Snippet {
   title = '';
   tags = new Array();
   lang = '';
-  uid = '';
-  version = '';
+  uid = Date.now().toString();
+  version = Util.getPackageVersion();
 
   constructor(state) {
     if(typeof state === 'object') {
-      if(state.title !== null && state.title !== undefined) {
+      if(Util.isset(state.title,'string')) {
         this.title = state.title;
-      } else {
-        this.title = "This snippet has no title.";
       }
-      if(state.tags !== null && state.tags !== undefined) {
+      if(Util.isset(state.tags,'object')) {
         this.tags = state.tags;
-      } else {
-        this.tags = new Array();
       }
-      if(state.content !== null && state.content !== undefined) {
+      if(Util.isset(state.content,'string')) {
         this.content = state.content;
-      } else {
-        this.content = "This snippet has no content";
       }
-      if(state.lang !== null && state.lang !== undefined) {
+      if(Util.isset(state.lang,'string')) {
         this.lang = state.lang;
-      } else {
-        this.lang = "";
       }
-      if(state.uid !== null && state.uid !== undefined) {
+      if(Util.isset(state.uid,'string')) {
         this.uid = state.uid;
-      } else {
-        this.uid = "";
       }
-      if(state.version !== null && state.uid !== undefined) {
+      if(Util.isset(state.version,'string')) {
         this.version = state.version;
-      } else {
-        this.version = '0.0.0';
       }
-    } else {
-      this.title = "";
-      this.tags = new Array();
-      this.content = "";
-      this.lang = "";
-      this.uid = Date.now().toString();
-      this.version = Util.getPackageVersion();
     }
   }
 

lib/storage.js

@@ -112,14 +112,32 @@ export default class Storage {
       throw new TypeError('Storage directory has not been initialized.');
     } else {
       var files = this.retrieveFiles();
+      var changed = 0;
+      var migrated = 'The following snippets have been migrated:\n';
       if(files.length > 0) {
-        file.forEach(function(current) {
+        files.forEach(function(current,index) {
           if(current.endsWith('.snippet.json')) {
             fs.renameSync(this.storageDir+this.dir+current,this.storageDir+this.dir+current.replace('.snippet.json','.json'));
             current = current.replace('.snippet.json','.json');
+          } else {
+            current = current.replace('.json','');
+          }
+
+          var snippet = new Snippet(JSON.parse(this.retrieveFile(current)));
+          var res = Util.compareVersions(Util.getPackageVersion(),snippet.getVersion());
+          if(res !== false && res === Util.getPackageVersion()) {
+            this.store(snippet);
+            this.deleteFile(current);
+            migrated += snippet.getTitle();
+            if(index < files.length-1) {
+              migrated += ', ';
+            }
+            changed++;
           }
-          var snippet = new Snippet(JSON.parse(this.retrieveFile(current.replace('.json',''))));
         });
+        if(changed > 0) {
+          atom.notifications.addInfo('Successfully migrated snippets within local storage to newer version.', {detail: migrated});
+        }
       }
     }
   }