From 6ea4f168a26185bf3c804456e1af5bd6114d3cc6 Mon Sep 17 00:00:00 2001
From: Joshua Dwire <joshua@joshuadwire.com>
Date: Tue, 15 Mar 2016 22:43:30 -0400
Subject: [PATCH] Remove link to settings page for non-admins

---
 resources/views/header.blade.php | 20 +++++++++++---------
 1 file changed, 11 insertions(+), 9 deletions(-)

diff --git a/resources/views/header.blade.php b/resources/views/header.blade.php
index 6fa6cb75a8..57f0409a9f 100644
--- a/resources/views/header.blade.php
+++ b/resources/views/header.blade.php
@@ -488,19 +488,21 @@
       
       <ul class="nav navbar-nav navbar-right navbar-settings"> 
         <li class="dropdown">
-          <a href="{{ URL::to('/settings') }}" class="dropdown-toggle">
-            <span class="glyphicon glyphicon-cog" title="{{ trans('texts.settings') }}"/>
-          </a>
-          <ul class="dropdown-menu">
-            @if (Utils::isAdmin())
+          @if (Utils::isAdmin())
+            <a href="{{ URL::to('/settings') }}" class="dropdown-toggle">
+              <span class="glyphicon glyphicon-cog" title="{{ trans('texts.settings') }}"/>
+            </a>
+            <ul class="dropdown-menu">
               @foreach (\App\Models\Account::$basicSettings as $setting)
                   <li>{!! link_to('settings/' . $setting, uctrans("texts.{$setting}")) !!}</li>
               @endforeach
               <li><a href="{{ url('settings/' . ACCOUNT_INVOICE_SETTINGS) }}">{!! uctrans('texts.advanced_settings') . Utils::getProLabel(ACCOUNT_ADVANCED_SETTINGS) !!}</a></li>
-            @else
-              <li>{!! link_to('settings/user_details', uctrans("texts.user_details")) !!}</li>
-            @endif
-          </ul>
+            </ul>
+          @else
+            <a href="{{ URL::to('/settings/user_details') }}" class="dropdown-toggle">
+              <span class="glyphicon glyphicon-user" title="{{ trans('texts.settings') }}"/>
+            </a>
+          @endif
         </li>
       </ul>