From 46e579b6e6c607ffa36e7c14ae3a3a183bd30d30 Mon Sep 17 00:00:00 2001
From: Paul Eggleton <paul.eggleton@linux.intel.com>
Date: Wed, 6 Mar 2013 09:16:19 +0000
Subject: [PATCH] Show count of layers to be reviewed next to Review link

Add a badge next to the Review link (when shown for users with the
publish permission) on all pages showing how many layers need to be
reviewed, if any.

Signed-off-by: Paul Eggleton <paul.eggleton@linux.intel.com>
---
 TODO                             | 1 -
 base.html                        | 7 ++++++-
 layerindex/context_processors.py | 3 ++-
 3 files changed, 8 insertions(+), 3 deletions(-)

diff --git a/TODO b/TODO
index 1d26136..e1026fe 100644
--- a/TODO
+++ b/TODO
@@ -16,7 +16,6 @@ Later:
 * Style/extend about page?
 * Style recipe info page?
 * Style machine list on detail
-* Show count of layers to be reviewed next to review button
 * Something to help with compatibility (although maybe this should just be handled using the existing versioned layer dependencies in layer.conf)
 * Query backend service? i.e. special URL to query information for external apps/scripts
 * Tool for finding/comparing duplicate recipes?
diff --git a/base.html b/base.html
index 75871fe..1989dd4 100644
--- a/base.html
+++ b/base.html
@@ -53,7 +53,12 @@
                     {% block submitlink %}<li><a href="{% url submit_layer %}">Submit layer</a></li>{% endblock %}
                     {% if user.is_authenticated %}
                     {% if perms.layeritem.publish_layer %}
-                        <li><a href="{% url layer_list_review %}">Review</a></li>
+                        <li><a href="{% url layer_list_review %}">
+                            Review
+                            {% if unpublished_count > 0 %}
+                            <span class="badge badge-warning">{{ unpublished_count }}</span>
+                            {% endif %}
+                        </a></li>
                     {% endif %}
                     <li class="divider-vertical"></li>
                     <li class="dropdown">
diff --git a/layerindex/context_processors.py b/layerindex/context_processors.py
index 8e2c4e7..90fc0f8 100644
--- a/layerindex/context_processors.py
+++ b/layerindex/context_processors.py
@@ -4,7 +4,7 @@
 #
 # Licensed under the MIT license, see COPYING.MIT for details
 
-from layerindex.models import Branch
+from layerindex.models import Branch, LayerItem
 
 def layerindex_context(request):
     current_branch = request.session.get('branch', None)
@@ -13,4 +13,5 @@ def layerindex_context(request):
     return {
         'all_branches': Branch.objects.all(),
         'current_branch': current_branch,
+        'unpublished_count': LayerItem.objects.filter(status='N').count(),
     }
\ No newline at end of file