correct undefined var in template and fix 'admin only' menu type. fixes

#2681.
zikula · Dec 23, 2015 · 05c79b5 · 05c79b5
1 parent 1400121
commit 05c79b5
Show file tree

Hide file tree

Showing 4 changed files with 42 additions and 30 deletions.
diff --git a/src/themes/BootstrapTheme/Resources/views/Include/admin_menu.html.twig b/src/themes/BootstrapTheme/Resources/views/Include/admin_menu.html.twig
@@ -0,0 +1,39 @@
+<div class="navbar navbar-inverse navbar-fixed-top">
+      <div class="container">
+        <div class="navbar-header">
+          <button type="button" class="navbar-toggle" data-toggle="collapse" data-target=".navbar-collapse">
+            <span class="icon-bar"></span>
+            <span class="icon-bar"></span>
+            <span class="icon-bar"></span>
+          </button>
+          <a class="navbar-brand" href="{{ pagevars.homepath }}">{{ getModVar('ZConfig', 'sitename') }}</a>
+        </div>
+        <div class="navbar-collapse collapse">
+          <ul class="nav navbar-nav">
+            <li>{{ adminPanelMenu() }}</li>
+            <li><a href="{{ pagevars.homepath }}">{{ __('Home') }}</a></li>
+            {% if hasPermission('ZikulaSettingsModule::', '::', 'ACCESS_ADMIN') %}
+            <li><a href="{{ path('zikulasettingsmodule_admin_index') }}">{{ __('Settings') }}</a></li>
+            {% endif %}
+            {% if hasPermission('ZikulaExtensionsModule::', '::', 'ACCESS_ADMIN') %}
+            <li><a href="{{ path('zikulaextensionsmodule_admin_index') }}">{{ __('Extensions') }}</a></li>
+            {% endif %}
+            {% if hasPermission('ZikulaBlocksModule::', '::', 'ACCESS_EDIT') %}
+            <li><a href="{{ path('zikulablocksmodule_admin_view') }}">{{ __('Blocks') }}</a></li>
+            {% endif %}
+            {% if hasPermission('ZikulaUsersModule::', '::', 'ACCESS_MODERATE') %}
+            <li><a href="{{ path('zikulausersmodule_admin_index') }}">{{ __('Users') }}</a></li>
+            {% endif %}
+            {% if hasPermission('ZikulaGroupsModule::', '::', 'ACCESS_EDIT') %}
+            <li><a href="{{ path('zikulagroupsmodule_admin_index') }}">{{ __('Groups') }}</a></li>
+            {% endif %}
+            {% if hasPermission('ZikulaPermissionsModule::', '::', 'ACCESS_ADMIN') %}
+            <li><a href="{{ path('zikulapermissionsmodule_admin_index') }}">{{ __('Permission rules') }}</a></li>
+            {% endif %}
+            {% if hasPermission('ZikulaThemeModule::', '::', 'ACCESS_EDIT') %}
+            <li><a href="{{ path('zikulathememodule_admin_index') }}">{{ __('Themes') }}</a></li>
+            {% endif %}
+          </ul>
+        </div><!--/.navbar-collapse -->
+      </div>
+    </div>
diff --git a/src/themes/BootstrapTheme/Resources/views/Include/main_menu.html.twig b/src/themes/BootstrapTheme/Resources/views/Include/main_menu.html.twig
@@ -9,37 +9,11 @@
           <a class="navbar-brand" href="{{ pagevars.homepath }}">{{ getModVar('ZConfig', 'sitename') }}</a>
         </div>
         <div class="navbar-collapse collapse">
-        {% if pagetype == 'admin' %}
           <ul class="nav navbar-nav">
             <li>{{ adminPanelMenu() }}</li>
-            <li><a href="{{ pagevars.homepath }}">{{ __('Home') }}</a></li>
-            {% if hasPermission('ZikulaSettingsModule::', '::', 'ACCESS_ADMIN') %}
-            <li><a href="{{ path('zikulasettingsmodule_admin_index') }}">{{ __('Settings') }}</a></li>
-            {% endif %}
-            {% if hasPermission('ZikulaExtensionsModule::', '::', 'ACCESS_ADMIN') %}
-            <li><a href="{{ path('zikulaextensionsmodule_admin_index') }}">{{ __('Extensions') }}</a></li>
-            {% endif %}
-            {% if hasPermission('ZikulaBlocksModule::', '::', 'ACCESS_EDIT') %}
-            <li><a href="{{ path('zikulablocksmodule_admin_view') }}">{{ __('Blocks') }}</a></li>
-            {% endif %}
-            {% if hasPermission('ZikulaUsersModule::', '::', 'ACCESS_MODERATE') %}
-            <li><a href="{{ path('zikulausersmodule_admin_index') }}">{{ __('Users') }}</a></li>
-            {% endif %}
-            {% if hasPermission('ZikulaGroupsModule::', '::', 'ACCESS_EDIT') %}
-            <li><a href="{{ path('zikulagroupsmodule_admin_index') }}">{{ __('Groups') }}</a></li>
-            {% endif %}
-            {% if hasPermission('ZikulaPermissionsModule::', '::', 'ACCESS_ADMIN') %}
-            <li><a href="{{ path('zikulapermissionsmodule_admin_index') }}">{{ __('Permission rules') }}</a></li>
-            {% endif %}
-            {% if hasPermission('ZikulaThemeModule::', '::', 'ACCESS_EDIT') %}
-            <li><a href="{{ path('zikulathememodule_admin_index') }}">{{ __('Themes') }}</a></li>
-            {% endif %}
-          </ul>
-        {% else %}
-          <ul class="nav navbar-nav">
             <li class="active"><a href="{{ pagevars.homepath }}" title="{{ __('Go to the sites home page') }}">{{ __('Home') }}</a></li>
             <li><a href="{{ path('zikulausersmodule_user_index') }}" title="{{ __('Go to your account panel') }}">{{ __('My Account') }}</a></li>
-            <li><a href="{{ path('zikulasearchmodule_user_main') }}" title="{{ __('Search this site') }}">{{ __('Site search') }}</a></li>
+            <li><a href="{{ path('zikulasearchmodule_user_form') }}" title="{{ __('Search this site') }}">{{ __('Site search') }}</a></li>
             <li class="dropdown">
               <a href="#" class="dropdown-toggle" data-toggle="dropdown">Dropdown <b class="caret"></b></a>
               <ul class="dropdown-menu">
@@ -62,7 +36,6 @@
             </div>
             <button type="submit" class="btn btn-success">Sign in</button>
           </form>
-        {% endif %}
         </div><!--/.navbar-collapse -->
       </div>
     </div>
diff --git a/src/themes/BootstrapTheme/Resources/views/admin.html.twig b/src/themes/BootstrapTheme/Resources/views/admin.html.twig
@@ -1,4 +1,4 @@
 {% include 'ZikulaBootstrapTheme:Include:header.html.twig' %}
-{% include 'ZikulaBootstrapTheme:Include:main_menu.html.twig' %}
+{% include 'ZikulaBootstrapTheme:Include:admin_menu.html.twig' %}
 {% include "ZikulaBootstrapTheme:Body:" ~ themevars.admin ~ ".html.twig" %}
 {% include 'ZikulaBootstrapTheme:Include:footer.html.twig' %}
diff --git a/src/themes/BootstrapTheme/Resources/views/home.html.twig b/src/themes/BootstrapTheme/Resources/views/home.html.twig
@@ -1,4 +1,4 @@
 {% include 'ZikulaBootstrapTheme:Include:header.html.twig' %}
 {% include 'ZikulaBootstrapTheme:Include:main_menu.html.twig' %}
 {% include "ZikulaBootstrapTheme:Body:" ~ themevars.home ~ ".html.twig" %}
-{% include 'ZikulaBootstrapTheme:Include:footer.html.twig' %}
+{% include 'ZikulaBootstrapTheme:Include:footer.html.twig' %}