Hide apps and sys icons if user is not an admin

plinth/context_processors.py

@@ -23,6 +23,7 @@ from django.utils.translation import ugettext as _, ugettext_noop
 import re
 
 from plinth import cfg
+from plinth.utils import is_user_admin
 
 
 def common(request):
@@ -41,5 +42,6 @@ def common(request):
         'cfg': cfg,
         'submenu': cfg.main_menu.active_item(request),
         'active_menu_urls': active_menu_urls,
-        'box_name': _(cfg.box_name)
+        'box_name': _(cfg.box_name),
+        'user_is_admin': is_user_admin(request.user)
     }

plinth/templates/base.html

@@ -96,7 +96,7 @@
       </div>
       <div class="collapse navbar-collapse">
         <ul class="nav navbar-nav">
-          {% if user.is_authenticated %}
+          {% if user.is_authenticated and user_is_admin %}
             <li>
               <a href="{% url 'index' %}" title="{% trans "Home" %}">
                 {% trans "Home" %}

plinth/tests/test_context_processors.py

@@ -18,7 +18,9 @@
 """
 Test module for Plinth's custom context processors.
 """
+from unittest.mock import Mock
 
+from django.contrib.auth.models import User
 from django.http import HttpRequest
 from django.test import TestCase
 
@@ -35,6 +37,8 @@ class ContextProcessorsTestCase(TestCase):
 
         request = HttpRequest()
         request.path = '/aaa/bbb/ccc/'
+        request.user = Mock()
+        request.user.groups.filter().exists = Mock(return_value=True)
         response = cp.common(request)
         self.assertIsNotNone(response)
 
@@ -55,6 +59,8 @@ class ContextProcessorsTestCase(TestCase):
         """Verify that the common() function works for border conditions."""
         request = HttpRequest()
         request.path = ''
+        request.user = Mock()
+        request.user.groups.filter().exists = Mock(return_value=True)
         response = cp.common(request)
         self.assertEqual([], response['active_menu_urls'])