From 6bf23484539bc0f697bdb9906a4bca1b1419efde Mon Sep 17 00:00:00 2001
From: tobes <toby.junk@gmail.com>
Date: Tue, 26 Feb 2013 13:27:28 +0000
Subject: [PATCH] [#471] Fix up nav_named_link() so now uses nav_link()

---
 ckan/lib/helpers.py                    | 23 ++++++++++++++++-------
 ckan/templates_legacy/layout_base.html |  2 +-
 2 files changed, 17 insertions(+), 8 deletions(-)

diff --git a/ckan/lib/helpers.py b/ckan/lib/helpers.py
index d7e09a557d8..11a34eb5875 100644
--- a/ckan/lib/helpers.py
+++ b/ckan/lib/helpers.py
@@ -335,28 +335,37 @@ def _create_link_text(text, **kwargs):
     )
 
 
-def nav_link(text, controller, **kwargs):
+def nav_link(text, *args, **kwargs):
     '''
     params
     class_: pass extra class(s) to add to the <a> tag
     icon: name of ckan icon to use within the link
     condition: if False then no link is returned
     '''
-    if kwargs.pop('condition', True):
+    if len(args) > 1:
+        raise Exception('Too many unnamed parameters supplied')
+    if len == 1:
         kwargs['controller'] = controller
-        link = _link_to(text, **kwargs)
+        log.warning('h.nav_link() please supply controller as a named '
+                    'parameter not a positional one')
+    named_route = kwargs.pop('named_route', '')
+    if kwargs.pop('condition', True):
+        if named_route:
+            link = _link_to(text, named_route, **kwargs)
+        else:
+            link = _link_to(text, **kwargs)
     else:
         link = ''
     return link
 
 
 @maintain.deprecated('h.nav_named_link is deprecated please '
-                     'use h.build_nav\nNOTE: the order of the first two '
-                     'parameters is reversed in the new function')
-def nav_named_link(text, name, **kwargs):
+                     'use h.nav_link\nNOTE: you will need to pass the '
+                     'route_name as a named parameter')
+def nav_named_link(text, named_route, **kwargs):
     '''Create a link for a named route.
     Deprecated in ckan 2.0 '''
-    return build_nav(name, text, **kwargs)
+    return nav_link(text, named_route=named_route, **kwargs)
 
 
 @maintain.deprecated('h.subnav_link is deprecated please '
diff --git a/ckan/templates_legacy/layout_base.html b/ckan/templates_legacy/layout_base.html
index 8f63ce4ff4b..fb2a4eb5c7d 100644
--- a/ckan/templates_legacy/layout_base.html
+++ b/ckan/templates_legacy/layout_base.html
@@ -75,7 +75,7 @@
           <div id="mainmenu">
             <span py:if="h.check_access('package_create')">${h.nav_link(_('Add a dataset'), controller='package', action='new')}</span>
             ${h.nav_link(_('Search'), controller='package', action='search', highlight_actions = 'new index')}
-            ${h.build_nav('%s_index' % h.default_group_type(), _('Groups'))}
+            ${h.nav_link(_('Groups'), named_route='%s_index' % h.default_group_type())}
             ${h.nav_link(_('About'), controller='home', action='about')}
           </div>
         </div>