Removed legacy subscription method

I don't really want to have to include support for it
This commit is contained in:
benji7425 2017-01-22 17:11:12 +00:00
parent 8eb30b6d15
commit f4a8a40cd5
2 changed files with 13 additions and 52 deletions

View File

@ -6,9 +6,7 @@
"messageDeleteDelay": 10000,
"youtubeMode": true,
"allowSubscriptions": true,
"legacySubscriptions": false,
"subscribersRoleID": "268153160273625088",
"legacySubscribersFile": "./subscribers.json",
"logFile": "./log",
"userCommands": {
"subscribe": "!subscribe",

View File

@ -106,7 +106,7 @@ var DiscordClient = {
//send a messsage containing the new feed link to our discord channel
DiscordClient.bot.sendMessage({
to: Config.channelID,
message: "<@" + Config.subscribersRole + ">" + link
message: Subscriptions.mention() + link
});
},
//actions to perform when certain messages are detected, along with channel or user requirements
@ -121,17 +121,6 @@ var DiscordClient = {
action: (user, userID, channelID, message) => { if (Config.allowSubscriptions) Subscriptions.unsubscribe(user, userID, channelID, message); },
channelID: Config.channelID
},
{
message: Config.userCommands.subscribersList,
action: (user, userID, channelID, message) => {
if (Config.allowSubscriptions && Config.legacySubscriptions)
DiscordClient.bot.sendMessage({
to: Config.channelID,
message: DiscordClient.bot.fixMessage("<@" + Subscriptions.subscribers.join("> <@") + ">")
}, (err, response) => { setTimeout(() => { DiscordClient.bot.deleteMessage({ channelID: channelID, messageID: response.id }); }, Config.messageDeleteDelay); });
},
channelID: Config.channelID
},
{
message: Config.userCommands.help,
action: (user, userID, channelID, message) => {
@ -166,21 +155,11 @@ var DiscordClient = {
};
var Subscriptions = {
parse: function () {
JsonFile.readFile(Config.legacySubscribersFile, (err, obj) => {
if (err) Log.error("Unable to parse json subscribers file", err);
this.subscribers = obj || [];
});
},
subscribe: function (user, userID, channelID, message) {
if (Config.legacySubscriptions)
this.legacy.subscribe();
else {
DiscordClient.bot.addToRole({
userID: userID,
roleID: Config.subscribersRoleID
});
}
DiscordClient.bot.addToRole({
userID: userID,
roleID: Config.subscribersRoleID
});
Log.event("Subscribed user " + (user ? user + "(" + userID + ")" : userID));
@ -189,14 +168,12 @@ var Subscriptions = {
message: "You have successfully subscribed"
}, (err, response) => { setTimeout(() => { DiscordClient.bot.deleteMessage({ channelID: channelID, messageID: response.id }); }, Config.messageDeleteDelay); });
},
unsubscribe: function (user, userID, channelID, message) {
if (Config.legacySubscriptions)
this.legacy.unsubscribe();
else
DiscordClient.bot.removeFromRole({
userID: userID,
roleID: Config.subscribersRoleID
});
DiscordClient.bot.removeFromRole({
userID: userID,
roleID: Config.subscribersRoleID
});
Log.event("Unsubscribed user " + (user ? user + "(" + userID + ")" : userID));
@ -205,23 +182,9 @@ var Subscriptions = {
message: "You have successfully unsubscribed"
}, (err, response) => { setTimeout(() => { DiscordClient.bot.deleteMessage({ channelID: channelID, messageID: response.id }); }, Config.messageDeleteDelay); });
},
legacy: {
subscribers: [],
subscribe: function (user, userID, channelID, message) {
if (this.subscribers.indexOf(userID) === -1) {
this.subscribers.push(userID); //subscribe the user if they aren't already subscribed
this.writeToFile();
}
},
unsubscribe: function (user, userID, channelID, message) {
if (this.subscribers.indexOf(userID) > -1) {
this.subscribers.splice(this.subscribers.indexOf(userID), 1);
this.writeToFile();
}
},
writeToFile: function () {
JsonFile.writeFile(Config.legacySubscribersFile, this.subscribers, (err) => { if (err) Log.error("Unable to write subscribers to json file", err); });
},
mention: function () {
return "<@" + Config.subscribersRole + "> ";
}
};