Merge pull request #774 from Martii/Issue-641

Shift `flags` to `flags.critical` in affected models

Auto-merge...migration complete... updating VPS

Author: Martii

controllers/discussion.js

@@ -338,7 +338,7 @@ function postComment(aUser, aDiscussion, aContent, aCreator, aCallback) {
     created: created,
     rating: 0,
     creator: aCreator,
-    flags: 0,
+    flags: { critical: 0 },
     flagged: false,
     id: created.getTime().toString(16),
     _discussionId: aDiscussion._id,

controllers/script.js

@@ -257,7 +257,7 @@ var getScriptPageTasks = function (aOptions) {
 
     removeLib.removeable(Script, script, authedUser, function (aCanRemove, aAuthor) {
       aOptions.canRemove = aCanRemove;
-      aOptions.flags = script.flags || 0;
+      aOptions.flags = (script.flags ? script.flags.critical : null) || 0;
       aOptions.removeUrl = '/remove' + (script.isLib ? '/libs/' : '/scripts/') + script.installNameSlug;
 
       if (!aCanRemove) {

controllers/scriptStorage.js

@@ -472,7 +472,7 @@ exports.storeScript = function (aUser, aMeta, aBuf, aCallback, aUpdate) {
           about: '',
           updated: new Date(),
           votes: 0,
-          flags: 0,
+          flags: { critical: 0 },
           installName: installName,
           fork: null,
           meta: isLibrary ? { name: aMeta } : aMeta,

controllers/user.js

@@ -79,7 +79,7 @@ var setupUserModerationUITask = function (aOptions) {
 
       removeLib.removeable(User, user, authedUser, function (aCanRemove, aAuthor) {
         aOptions.canRemove = aCanRemove;
-        aOptions.flags = user.flags || 0;
+        aOptions.flags = (user.flags ? user.flags.critical : null) || 0;
 
         if (!aCanRemove) {
           return aCallback();

libs/flag.js

@@ -87,19 +87,40 @@ function getThreshold(aModel, aContent, aAuthor, aCallback) {
 exports.getThreshold = getThreshold;
 
 function saveContent(aModel, aContent, aAuthor, aFlags, aCallback) {
-  if (!aContent.flags) { aContent.flags = 0; }
-  aContent.flags += aFlags;
+  if (!aContent.flags) {
+    aContent.flags = {};
+  }
+
+  if (!aContent.flags.critical) {
+    aContent.flags.critical = 0;
+  }
+  aContent.flags.critical += aFlags;
 
-  if (aContent.flags >= thresholds[aModel.modelName] * (aAuthor.role < 4 ? 2 : 1)) {
+  if (aContent.flags.critical >= thresholds[aModel.modelName] * (aAuthor.role < 4 ? 2 : 1)) {
     return getThreshold(aModel, aContent, aAuthor, function (aThreshold) {
-      aContent.flagged = aContent.flags >= aThreshold;
-      aContent.save(function (aErr, aContent) { aCallback(aContent.flagged); });
+      aContent.flagged = aContent.flags.critical >= aThreshold;
+
+      aContent.save(function (aErr, aContent) {
+        if (aErr) {
+          console.warn('Error flagging content', aErr);
+          aCallback(null);
+          return;
+        }
+        aCallback(aContent.flagged);
+      });
     });
   } else {
     aContent.flagged = false;
   }
 
-  aContent.save(function (aErr, aContent) { aCallback(aContent.flagged); });
+  aContent.save(function (aErr, aContent) {
+    if (aErr) {
+      console.warn('Error unflagging content', aErr);
+      aCallback(null);
+      return;
+    }
+    aCallback(aContent.flagged);
+  });
 }
 exports.saveContent = saveContent;
 
@@ -111,7 +132,13 @@ function flag(aModel, aContent, aUser, aAuthor, aCallback) {
   });
 
   flag.save(function (aErr, aFlag) {
-    if (!aContent.flags) { aContent.flags = 0; }
+    if (!aContent.flags) {
+      aContent.flags = {};
+    }
+
+    if (!aContent.flags.critical) {
+      aContent.flags.critical = 0;
+    }
     if (!aContent.flagged) { aContent.flagged = false; }
 
     saveContent(aModel, aContent, aAuthor, aUser.role < 4 ? 2 : 1, aCallback);
@@ -130,9 +157,17 @@ exports.unflag = function (aModel, aContent, aUser, aCallback) {
   if (!aUser) { return aCallback(null); }
 
   getFlag(aModel, aContent, aUser, function (aFlag) {
-    if (!aFlag) { return aCallback(null); }
+    if (!aFlag) {
+      return aCallback(null);
+    }
 
-    if (!aContent.flags) { aContent.flags = 0; }
+    if (!aContent.flags) {
+      aContent.flags = {};
+    }
+
+    if (!aContent.flags.critical) {
+      aContent.flags.critical = 0;
+    }
     if (!aContent.flagged) { aContent.flagged = false; }
 
     function removeFlag(aAuthor) {

libs/modelParser.js

@@ -175,10 +175,11 @@ var parseScript = function (aScriptData) {
   script.isFork = script.fork && script.fork.length > 0;
 
   // Script Good/Bad bar.
-  var sumVotesAndFlags = script.votes + script.flags;
+  var criticalFlags = (script.flags ? script.flags.critical : null) || 0;
+  var sumVotesAndFlags = script.votes + criticalFlags;
 
   var votesRatio = sumVotesAndFlags > 0 ? script.votes / sumVotesAndFlags : 1;
-  var flagsRatio = sumVotesAndFlags > 0 ? script.flags / sumVotesAndFlags : 0;
+  var flagsRatio = sumVotesAndFlags > 0 ? criticalFlags / sumVotesAndFlags : 0;
 
   var votesPercent = votesRatio * 100;
   var flagsPercent = flagsRatio * 100;

libs/modelQuery.js

@@ -198,7 +198,7 @@ var applyModelListQueryFlaggedFilter = function (aModelListQuery, aOptions, aFla
         case 'absolute':
           if (aOptions.isAdmin) {
             aOptions.filterAbsolute = true;
-            aModelListQuery.and({ flagsAbsolute: { $gt: 0 } }); // TODO: This does not exist yet
+            aModelListQuery.and({ 'flags.absolute': { $gt: 0 } }); // TODO: Exists in schema but needs linkage
             break;
           }
           // fallthrough
@@ -212,7 +212,7 @@ var applyModelListQueryFlaggedFilter = function (aModelListQuery, aOptions, aFla
             aOptions.isFlagged = 'true';
           }
 
-          aModelListQuery.and({ flags: { $gt: 0 } });
+          aModelListQuery.and({ 'flags.critical': { $gt: 0 } });
           break;
       }
 

models/comment.js

@@ -18,7 +18,10 @@ var commentSchema = new Schema({
 
   // Moderation
   creator: Boolean,
-  flags: Number,
+  flags: {
+    critical: Number,
+    absolute: Number
+  },
   flagged: Boolean,
 
   // Extra info

models/script.js

@@ -20,8 +20,11 @@ var scriptSchema = new Schema({
   updated: Date,
 
   // Moderation
-  votes: Number, // upvotes negate flags
-  flags: Number,
+  votes: Number, // upvotes negate flags.critical
+  flags: {
+    critical: Number,
+    absolute: Number
+  },
   flagged: Boolean,
   installName: String,
 

models/user.js

@@ -23,7 +23,10 @@ var userSchema = new Schema({
 
   // Moderation
   role: Number,
-  flags: Number,
+  flags: {
+    critical: Number,
+    absolute: Number
+  },
   flagged: Boolean,
   sessionIds: [String]
 });