Indev (#121)

* Download.js Fix

* Fixes

* No longer failed

CHANGELOG.md

@@ -11,6 +11,8 @@ and this project adheres to [Semantic Versioning](http://semver.org/).
 - New Initialize Function to alleviate undefined errors
 
 ### Changed
+- Changed info to no longer mention Evie because she said she was tired of it.. kek
+- New runCommandInhibitors should be much faster and prioritizes inhibitors via a prioritiy configuration setting.
 - Old Configuration system now points to the new configuration system, to ease the trouble of updating to newer versions of Komada
 - Pieces now have  specific order they load in. (Functions, Providers, Commands, Inhibitors, Monitors, Events)
 - Confs.js uses new configuration system now

commands/System/info.js

@@ -1,6 +1,5 @@
 exports.run = (client, msg) => {
-  msg.channel.sendMessage(`This bot is built on the Komada framework, a plug-and-play bot builder made by <@139412744439988224> (LuckyEvie#4611 , aka \`root\`).
-For more information visit: <https://komada.js.org>`);
+  msg.channel.sendMessage("This bot is built on the Komada framework, a plug-and-play bot builder made by Dirigeant's team of dedicated developers. For more information visit: <https://komada.js.org>");
 };
 
 exports.conf = {

functions/runCommandInhibitors.js

@@ -1,18 +1,15 @@
-module.exports = (client, msg, cmd, selective = false) => new Promise((resolve, reject) => {
-  const mps = [true];
-  let i = 1;
+module.exports = (client, msg, cmd) => new Promise((resolve, reject) => {
   let usage;
-  client.commandInhibitors.forEach((mProc, key) => {
-    if (key === "usage") usage = i;
-    if (!mProc.conf.spamProtection || !selective) {
-      mps.push(mProc.run(client, msg, cmd));
-    }
-    i++;
-  });
-  Promise.all(mps)
+  const priority = client.commandInhibitors.array();
+  const sorted = priority.sort((a, b) => a.conf.priority > b.conf.priority);
+  sorted.forEach((inhib) => {
+    inhib.run(client, msg, cmd)
       .then((value) => {
-        resolve(value[usage]);
-      }, (reason) => {
-        reject(reason);
+        if (value) usage = value;
+      })
+      .catch((error) => {
+        reject(error);
       });
+  });
+  resolve(usage);
 });

inhibitors/disable.js

@@ -1,6 +1,7 @@
 exports.conf = {
   enabled: true,
   spamProtection: false,
+  priority: 9,
 };
 
 exports.run = (client, msg, cmd) => new Promise((resolve, reject) => {

inhibitors/guildOnly.js

@@ -1,6 +1,7 @@
 exports.conf = {
   enabled: true,
   spamProtection: false,
+  priority: 8,
 };
 
 exports.run = (client, msg, cmd) => new Promise((resolve, reject) => {

inhibitors/missingBotPermissions.js

@@ -1,6 +1,7 @@
 exports.conf = {
   enabled: true,
   spamProtection: false,
+  priority: 7,
 };
 
 exports.run = (client, msg, cmd) => new Promise((resolve, reject) => {

inhibitors/permissions.js

@@ -1,6 +1,7 @@
 exports.conf = {
   enabled: true,
   spamProtection: false,
+  priority: 10,
 };
 
 exports.run = (client, msg, cmd) => new Promise(async (resolve, reject) => {

inhibitors/requiredFuncs.js

@@ -1,6 +1,7 @@
 exports.conf = {
   enabled: true,
   spamProtection: false,
+  priority: 6,
 };
 
 exports.run = (client, msg, cmd) => new Promise((resolve, reject) => {

inhibitors/usage.js

@@ -3,6 +3,7 @@ const url = require("url");
 exports.conf = {
   enabled: true,
   spamProtection: true,
+  priority: 5,
 };
 
 exports.run = (client, msg, cmd, args = undefined) => new Promise((resolve, reject) => {

package.json

@@ -1,6 +1,6 @@
 {
   "name": "komada",
-  "version": "0.14.1",
+  "version": "0.14.2",
   "author": "Evelyne Lachance",
   "description": "Komada: Croatian for 'pieces', is a modular bot system including reloading modules and easy to use custom commands.",
   "main": "app.js",