From e8b23b48a2f1a42875360ded73f03706f38f0434 Mon Sep 17 00:00:00 2001
From: Roberto Rosario <Roberto.Rosario.Gonzalez@gmail.com>
Date: Fri, 15 Apr 2011 00:04:06 -0400
Subject: [PATCH] Fixed diagnostics url resolution

---
 apps/main/api.py                                 |  3 +++
 .../navigation/templates/generic_navigation.html | 16 ++++++++--------
 2 files changed, 11 insertions(+), 8 deletions(-)

diff --git a/apps/main/api.py b/apps/main/api.py
index 3e29269d47..ed27b8a49d 100644
--- a/apps/main/api.py
+++ b/apps/main/api.py
@@ -1,8 +1,11 @@
+from django.core.urlresolvers import reverse
+
 diagnostics = {}
 
 
 def register_diagnostic(namespace, title, link):
     namespace_dict = diagnostics.get(namespace, {'title': None, 'links': []})
     namespace_dict['title'] = title
+    link['url'] = link.get('url', reverse(link['view']))
     namespace_dict['links'].append(link)
     diagnostics[namespace] = namespace_dict
diff --git a/apps/navigation/templates/generic_navigation.html b/apps/navigation/templates/generic_navigation.html
index 49043d9e7d..0ddd529491 100644
--- a/apps/navigation/templates/generic_navigation.html
+++ b/apps/navigation/templates/generic_navigation.html
@@ -2,17 +2,17 @@
 
 <span class="nowrap">
 {% for link in object_navigation_links %}
-{% if link.permissions %}
-    {% check_permissions request.user link.permissions.namespace link.permissions.permissions %}
-    {% if permission %}
+    {% if link.permissions %}
+        {% check_permissions request.user link.permissions.namespace link.permissions.permissions %}
+        {% if permission %}
+            {% if as_li %}<li>{% endif %}
+                <a class="{{ link.class }}" href="{{ link.url }}">{% if link.famfam %}<span class="famfam active famfam-{{ link.famfam|default:'link' }}"></span>{% endif %}{{ link.text|capfirst }}{% if link.error %} - {{ link.error }}{% endif %}{% if link.active %}<span class="famfam active famfam-resultset_previous"></span>{% endif %}</a>{% if horizontal %}{% if not forloop.last %} | {% endif %}{% endif %}
+            {% if as_li %}</li>{% endif %}    
+        {% endif %}
+    {% else %}
         {% if as_li %}<li>{% endif %}
             <a class="{{ link.class }}" href="{{ link.url }}">{% if link.famfam %}<span class="famfam active famfam-{{ link.famfam|default:'link' }}"></span>{% endif %}{{ link.text|capfirst }}{% if link.error %} - {{ link.error }}{% endif %}{% if link.active %}<span class="famfam active famfam-resultset_previous"></span>{% endif %}</a>{% if horizontal %}{% if not forloop.last %} | {% endif %}{% endif %}
         {% if as_li %}</li>{% endif %}    
     {% endif %}
-{% else %}
-    {% if as_li %}<li>{% endif %}
-        <a class="{{ link.class }}" href="{{ link.url }}">{% if link.famfam %}<span class="famfam active famfam-{{ link.famfam|default:'link' }}"></span>{% endif %}{{ link.text|capfirst }}{% if link.error %} - {{ link.error }}{% endif %}{% if link.active %}<span class="famfam active famfam-resultset_previous"></span>{% endif %}</a>{% if horizontal %}{% if not forloop.last %} | {% endif %}{% endif %}
-    {% if as_li %}</li>{% endif %}    
-{% endif %}
 {% endfor %}
 </span>