Repair issue count

* Still a partial bug fix... this entire slug methodology still needs to be refactored which is going to be lengthy. * Surprised no one noticed, including me until now, that issue counts were all zero on the badge. Followup from OpenUserJS@1f82419 Applies to OpenUserJS#819, OpenUserJS#200, and Originally applies to OpenUserJS#262 ... not part of OpenUserJS#262 (comment)
Martii · Nov 25, 2015 · bde55db · bde55db
1 parent 2c25da0
commit bde55db
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 4 deletions.
diff --git a/controllers/issue.js b/controllers/issue.js
@@ -169,7 +169,7 @@ exports.list = function (aReq, aRes, aNext) {
 
       // Show the number of open issues
       scriptOpenIssueCountQuery = Discussion.find({
-        category: script.issuesCategorySlug,
+        category: decodeURIComponent(script.issuesCategorySlug),
         open: { $ne: false }
       });
       tasks.push(countTask(scriptOpenIssueCountQuery, options, 'issueCount'));
@@ -292,7 +292,7 @@ exports.view = function (aReq, aRes, aNext) {
 
         // Show the number of open issues
         scriptOpenIssueCountQuery = Discussion.find({
-          category: script.issuesCategorySlug,
+          category: decodeURIComponent(script.issuesCategorySlug),
           open: { $ne: false }
         });
         tasks.push(countTask(scriptOpenIssueCountQuery, options, 'issueCount'));

diff --git a/controllers/script.js b/controllers/script.js
@@ -87,7 +87,7 @@ var getScriptPageTasks = function (aOptions) {
 
   // Show the number of open issues
   scriptOpenIssueCountQuery = Discussion.find({ category: scriptStorage
-    .caseSensitive(script.issuesCategorySlug, true), open: {$ne: false} });
+    .caseSensitive(decodeURIComponent(script.issuesCategorySlug), true), open: {$ne: false} });
   tasks.push(countTask(scriptOpenIssueCountQuery, aOptions, 'issueCount'));
 
   // Show the groups the script belongs to

diff --git a/controllers/user.js b/controllers/user.js
@@ -1655,7 +1655,7 @@ exports.editScript = function (aReq, aRes, aNext) {
 
         // Show the number of open issues
         scriptOpenIssueCountQuery = Discussion.find({ category: scriptStorage
-            .caseSensitive(script.issuesCategorySlug, true), open: {$ne: false} });
+            .caseSensitive(decodeURIComponent(script.issuesCategorySlug), true), open: {$ne: false} });
         tasks.push(countTask(scriptOpenIssueCountQuery, options, 'issueCount'));
 
         //---