Merge pull request #7 from ljharb/factoid_username_tracking

Adding the factoid's creator and a list of its editors to the JSON database

lib/factoidserv/index.js

@@ -9,22 +9,43 @@ var FactoidServer = module.exports = function(filename) {
 	this.db = new JSONSaver(filename);
 };
 
+var caseInsensitiveDeDupingInsert = function (array, value) {
+	var valueLower = value.toLowerCase();
+	var found = array.some(function (item, index) {
+		if (valueLower === item.toLowerCase()) {
+			array[index] = value;
+			return true;
+		});
+	});
+	if (!found) {
+		array.push(value);
+	}
+};
 
-FactoidServer.prototype.learn = function(key, value) {
+
+FactoidServer.prototype.learn = function(key, value, username) {
+	if (!username) {
+		username = '';
+	}
 
 	var db = this.db.object.factoids;
 
 	var popularity = 0;
+	var creator = username;
+	var editors = [];
 	key = key.toLowerCase();
 
 	if (typeof db[key] !== "undefined") {
 		if (typeof db[key].alias !== "undefined") {
-			return this.learn(db[key].alias, value);
+			return this.learn(db[key].alias, value, username);
 		}
+		creator = db[key].creator;
+		editors = db[key].editors || editors;
+		caseInsensitiveDeDupingInsert(editors, username);
 		popularity = db[key].popularity || 0;
 	}
 
-	db[key] = {value: value, popularity: popularity};
+	db[key] = {value: value, popularity: popularity, creator: creator, editors: editors};
 	this.db.activity();
 };
 

shared.js

@@ -125,7 +125,7 @@ var Shared = module.exports = {
 
 			/* Setting the text of a factoid */ 
 			if (operation === "=") {
-				this.factoids.learn(factoid, value);
+				this.factoids.learn(factoid, value, context.sender.name);
 				context.channel.send_reply(context.sender, "Learned `"+factoid+"`.");
 				return;
 
@@ -139,7 +139,7 @@ var Shared = module.exports = {
 				if (old === result) {
 					context.channel.send_reply(context.sender, "Nothing changed.");
 				} else {
-					this.factoids.learn(factoid, result);
+					this.factoids.learn(factoid, result, context.sender.name);
 					context.channel.send_reply(context.sender, "Changed `"+factoid+
 						"` to: "+result);
 				}