ItemSets are now removed after new patch

- ItemSets are now removed after a new patch. - Fixed few things
thomas-cardon · Jul 9, 2018 · 9d2e5fb · 9d2e5fb
1 parent 499c489
commit 9d2e5fb
Show file tree

Hide file tree

Showing 4 changed files with 40 additions and 20 deletions.
diff --git a/objects/ChampionSelect.js b/objects/ChampionSelect.js
@@ -9,6 +9,8 @@ class ChampionSelect extends EventEmitter {
     super();
     this.inChampionSelect = false;
 
+    ItemSetHandler.getItemSets().then(x => console.dir).catch(UI.error);
+
     this.on('firstTick', async () => {
       console.log('Entering Champion Select');
 
@@ -120,7 +122,7 @@ class ChampionSelect extends EventEmitter {
 
           for (let itemset of old) {
             itemset = await ItemSetHandler.parse(champion.key, itemset);
-            await itemset.remove();
+            await itemset.delete();
           }
 
           Mana.status(`Loading ${itemsets.length} sets for ${champion.name}`);

diff --git a/objects/ItemSet.js b/objects/ItemSet.js
@@ -68,7 +68,7 @@ class ItemSet {
     });
   }
 
-  remove() {
+  delete() {
     let p = this.path;
     return new Promise((resolve, reject) => {
       fs.unlink(p, err => {

diff --git a/objects/handlers/ItemSetHandler.js b/objects/handlers/ItemSetHandler.js
@@ -12,38 +12,34 @@ class ItemSetHandler {
     return set;
   }
 
-  static _readFile(path) {
-    return new Promise((resolve, reject) => {
-      fs.readFile(path, 'utf8', function (err, data) {
-        if (err) return reject(err);
-        resolve(JSON.parse(data));
-      });
-    });
-  }
-
   static getItemSets() {
     let self = this;
+    console.log(path.resolve(Mana.store.get('leaguePath') + `\\Config\\Champions\\`));
     return new Promise((resolve, reject) => {
       fs.readdir(path.resolve(Mana.store.get('leaguePath') + `\\Config\\Champions\\`), (err, dir) => {
         if (err) return reject(err);
-        if (dir.length === 0) return resolve([]);
+        if (dir.length === 0) return reject(Error('Champions folder is empty?'));
+
+        let arr = [], res = [];
 
-        let arr = [];
         for (let key of dir)
           arr.push(self.getItemSetsByChampionKey(key));
 
         Promise.all(arr).then(values => {
-          let array = [];
+          for (let x of values)
+            res = res.concat(x);
 
-          for (let x of array)
-            array = array.concat(x);
-
-          resolve(array);
+          resolve(res);
         }).catch(reject);
       });
     });
   }
 
+  static async deleteItemSets(list) {
+    for (let path of list)
+      await this._deleteFile(path);
+  }
+
   static getItemSetsByChampionKey(key) {
     key = key.toLowerCase();
     return new Promise((resolve, reject) => {
@@ -53,12 +49,31 @@ class ItemSetHandler {
 
         let arr = [];
         for (let file of dir)
-          if (file.startsWith('MFLUX_')) arr.push(path.resolve(Mana.store.get('leaguePath'), `\\Config\\Champions\\${key}\\Recommended\\${file}`));
+          if (file.startsWith('MFLUX_')) arr.push(path.resolve(Mana.store.get('leaguePath') + `\\Config\\Champions\\${key}\\Recommended\\${file}`));
 
         resolve(arr);
       });
     });
   }
+
+  static _readFile(path) {
+    return new Promise((resolve, reject) => {
+      fs.readFile(path, 'utf8', function (err, data) {
+        if (err) return reject(err);
+        resolve(JSON.parse(data));
+      });
+    });
+  }
+
+  static _deleteFile(path) {
+    return new Promise((resolve, reject) => {
+      fs.unlink(path, err => {
+        if (err && err.code === 'ENOENT') resolve(false);
+        else if (err) reject(err);
+        else resolve(true);
+      });
+    });
+  }
 }
 
 module.exports = ItemSetHandler;
diff --git a/renderer.js b/renderer.js
@@ -4,11 +4,12 @@
 process.env.NODE_TLS_REJECT_UNAUTHORIZED = "0";
 
 const request = require('request'), rp = require('request-promise-native');
-
 const Store = require('electron-store');
 
 const { dialog } = require('electron').remote;
 
+const ItemSetHandler = require('./objects/handlers/ItemSetHandler');
+
 Mana.version = require('./package.json').version;
 Mana.status = str => {
   $('.status').text(str + '...');
@@ -87,6 +88,8 @@ ipcRenderer.once('lcu-connected', (event, d) => {
     if (Mana.store.get('gameVersion') !== ver) {
       Mana.store.set('runes', {});
       Mana.store.set('summonerspells', {});
+
+      ItemSetHandler.getItemSets().then(x => ItemSetHandler.deleteItemSets(x)).catch(UI.error);
     }
 
     Mana.store.set('gameVersion', ver);