Commits

David Jean Louis committed daeec9d

Fixed issue #26 (menu bar showing for non-staff users) and also updated templates to match the django 1.2 templates

  • Participants
  • Parent commits d723c4a

Comments (0)

Files changed (2)

File admin_tools/menu/templates/admin/base_site.html

 {% extends "admin/base.html" %}
 {% load i18n admin_tools_menu_tags %}
+
 {% block title %}{{ title }} | {% trans 'Django site admin' %}{% endblock %}
 
 {% block extrastyle %}
 {{ block.super }}
+{% if user.is_active and user.is_staff %}
+{% if not is_popup %}
 {% admin_tools_render_menu_css %}
+{% endif %}
+{% endif %}
 {% endblock %}
 
 {% block branding %}
 {% endblock %}
 
 {% block nav-global %}
-{% if user and user.is_authenticated %}
+{% if user.is_active and user.is_staff %}
 {% if not is_popup %}
 {% admin_tools_render_menu %}
 {% endif %}

File admin_tools/theming/templates/admin/base.html

         <div id="branding">
         {% block branding %}{% endblock %}
         </div>
-        {% if user.is_authenticated and user.is_staff %}
+        {% if user.is_active and user.is_staff %}
         <div id="user-tools">
             {% trans 'Welcome,' %}
-            <strong>{% firstof user.first_name user.username %}</strong>.
+            <strong>{% filter force_escape %}{% firstof user.first_name user.username %}{% endfilter %}</strong>.
             {% block userlinks %}
                 {% url django-admindocs-docroot as docsroot %}
                 {% if docsroot %}