From 5ebd12c0709c69a3da3d88fd4ef73f38e95ffcf1 Mon Sep 17 00:00:00 2001
From: Neil Lalonde <neillalonde@gmail.com>
Date: Wed, 5 Aug 2015 15:41:36 -0400
Subject: [PATCH] FEATURE: plugin hooks for topic list nav items

---
 .../discourse/components/nav-item.js.es6      |  2 ++
 .../models/{nav_item.js => nav-item.js.es6}   | 34 +++++++++++++++----
 2 files changed, 29 insertions(+), 7 deletions(-)
 rename app/assets/javascripts/discourse/models/{nav_item.js => nav-item.js.es6} (79%)

diff --git a/app/assets/javascripts/discourse/components/nav-item.js.es6 b/app/assets/javascripts/discourse/components/nav-item.js.es6
index c50741820e9..87ab1007a82 100644
--- a/app/assets/javascripts/discourse/components/nav-item.js.es6
+++ b/app/assets/javascripts/discourse/components/nav-item.js.es6
@@ -1,3 +1,5 @@
+/* You might be looking for navigation-item. */
+
 export default Ember.Component.extend({
   tagName: 'li',
   classNameBindings: ['active'],
diff --git a/app/assets/javascripts/discourse/models/nav_item.js b/app/assets/javascripts/discourse/models/nav-item.js.es6
similarity index 79%
rename from app/assets/javascripts/discourse/models/nav_item.js
rename to app/assets/javascripts/discourse/models/nav-item.js.es6
index 0ea27df11e8..def6a3430a5 100644
--- a/app/assets/javascripts/discourse/models/nav_item.js
+++ b/app/assets/javascripts/discourse/models/nav-item.js.es6
@@ -7,7 +7,7 @@
   @module Discourse
 **/
 
-Discourse.NavItem = Discourse.Model.extend({
+const NavItem = Discourse.Model.extend({
 
   displayName: function() {
     var categoryName = this.get('categoryName'),
@@ -25,7 +25,7 @@ Discourse.NavItem = Discourse.Model.extend({
       extra.categoryName = Discourse.Formatter.toTitleCase(categoryName);
     }
     return I18n.t("filters." + name.replace("/", ".") + ".title", extra);
-  }.property('categoryName,name,count'),
+  }.property('categoryName', 'name', 'count'),
 
   topicTrackingState: function() {
     return Discourse.TopicTrackingState.current();
@@ -45,8 +45,13 @@ Discourse.NavItem = Discourse.Model.extend({
     return null;
   }.property('name'),
 
-  // href from this item
   href: function() {
+    var customHref = null;
+    _.each(NavItem.customNavItemHrefs, function(cb) {
+      customHref = cb.call(this, this);
+      if (customHref) { return false; }
+    }, this);
+    if (customHref) { return customHref; }
     return Discourse.getURL("/") + this.get('filterMode');
   }.property('filterMode'),
 
@@ -79,10 +84,13 @@ Discourse.NavItem = Discourse.Model.extend({
 
 });
 
-Discourse.NavItem.reopenClass({
+NavItem.reopenClass({
+
+  extraArgsCallbacks: [],
+  customNavItemHrefs: [],
 
   // create a nav item from the text, will return null if there is not valid nav item for this particular text
-  fromText: function(text, opts) {
+  fromText(text, opts) {
     var split = text.split(","),
         name = split[0],
         testName = name.split("/")[0],
@@ -92,13 +100,17 @@ Discourse.NavItem.reopenClass({
     if (!Discourse.Category.list() && testName === "categories") return null;
     if (!Discourse.Site.currentProp('top_menu_items').contains(testName)) return null;
 
-    var args = { name: name, hasIcon: name === "unread" };
+    var args = { name: name, hasIcon: name === "unread" }, extra = null, self = this;
     if (opts.category) { args.category = opts.category; }
     if (opts.noSubcategories) { args.noSubcategories = true; }
+    _.each(NavItem.extraArgsCallbacks, function(cb) {
+      extra = cb.call(self, text, opts);
+      _.merge(args, extra);
+    });
     return Discourse.NavItem.create(args);
   },
 
-  buildList: function(category, args) {
+  buildList(category, args) {
     args = args || {};
     if (category) { args.category = category }
 
@@ -118,3 +130,11 @@ Discourse.NavItem.reopenClass({
   }
 
 });
+
+export default NavItem;
+export function extraNavItemProperties(cb) {
+  NavItem.extraArgsCallbacks.push(cb);
+}
+export function customNavItemHref(cb) {
+  NavItem.customNavItemHrefs.push(cb);
+}