Convert various moderator? tests to use cancancan permissions checks

[rails.git] / app / views / layouts / _header.html.erb

diff --git a/app/views/layouts/_header.html.erb b/app/views/layouts/_header.html.erb
index 946f95febf22f23aee9ba1391a70c2ab247be514..c95cc4a75d447375c544c3b38d1a5a22e1dd6a4f 100644 (file)
--- a/app/views/layouts/_header.html.erb
+++ b/app/views/layouts/_header.html.erb
@@ -3,9 +3,9 @@
     <a href="<%= root_path %>" class="geolink">
       <picture>
         <source srcset="<%= image_path "osm_logo.svg" %>" type="image/svg+xml"></source>
     <a href="<%= root_path %>" class="geolink">
       <picture>
         <source srcset="<%= image_path "osm_logo.svg" %>" type="image/svg+xml"></source>

-        <%= image_tag "osm_logo.png", :srcset => image_path("osm_logo.svg"), :alt => t('layouts.logo.alt_text'), :class => 'logo' %>
+        <%= image_tag "osm_logo.png", :srcset => image_path("osm_logo.svg"), :alt => t("layouts.logo.alt_text"), :class => "logo" %>

       </picture>
       </picture>

-      <%= t 'layouts.project_name.h1' %>
+      <%= t "layouts.project_name.h1" %>

     </a>
   </h1>
   <a href="#" id="menu-icon"></a>
     </a>
   </h1>
   <a href="#" id="menu-icon"></a>


@@ -13,14 +13,16 @@
     <%= content_for :header %>
     <ul>
       <li id="edit_tab" class="dropdown <%= current_page_class(edit_path) %>">
     <%= content_for :header %>
     <ul>
       <li id="edit_tab" class="dropdown <%= current_page_class(edit_path) %>">

-        <%= link_to t('layouts.edit'), edit_path, :class => "tab geolink editlink",
-                    :id => 'editanchor',
-                    :data => { :editor => preferred_editor }
-        %><a class='dropdown-toggle' data-toggle='dropdown' href='#'><b class="caret"></b></a>
+        <%= link_to t("layouts.edit"),
+                    edit_path,
+                    :class => "tab geolink editlink",
+                    :id => "editanchor",
+                    :data => { :editor => preferred_editor } %>
+        <a class='dropdown-toggle' data-toggle='dropdown' href='#'><b class="caret"></b></a>

         <ul class='dropdown-menu'>
           <% Editors::RECOMMENDED_EDITORS.each do |editor| %>
             <li>
         <ul class='dropdown-menu'>
           <% Editors::RECOMMENDED_EDITORS.each do |editor| %>
             <li>

-              <%= link_to t('layouts.edit_with', :editor => t("editor.#{editor}.description")),
+              <%= link_to t("layouts.edit_with", :editor => t("editor.#{editor}.description")),

                           edit_path(:editor => editor),
                           :data => { :editor => editor },
                           :class => "geolink editlink" %>
                           edit_path(:editor => editor),
                           :data => { :editor => editor },
                           :class => "geolink editlink" %>


@@ -29,81 +31,85 @@
         </ul>
       </li>
       <li id="history_tab" class="<%= current_page_class(history_path) %>">
         </ul>
       </li>
       <li id="history_tab" class="<%= current_page_class(history_path) %>">

-        <%= link_to t('layouts.history'), history_path, :class => 'tab geolink' %>
+        <%= link_to t("layouts.history"), history_path, :class => "tab geolink" %>

       </li>
       <li id="export_tab" class="<%= current_page_class(export_path) %>">
       </li>
       <li id="export_tab" class="<%= current_page_class(export_path) %>">

-        <%= link_to t('layouts.export'), export_path, :class => 'tab geolink' %>
+        <%= link_to t("layouts.export"), export_path, :class => "tab geolink" %>

       </li>
     </ul>
   </nav>
   <nav class='secondary'>
     <ul>
       </li>
     </ul>
   </nav>
   <nav class='secondary'>
     <ul>

-      <% if current_user and ( current_user.administrator? or current_user.moderator? ) %>
+      <% if can? :index, Issue %>

         <li class="compact-hide <%= current_page_class(issues_path) %>">
         <li class="compact-hide <%= current_page_class(issues_path) %>">

-          <%= link_to issues_path(:status => 'open') do %>
-            <%= t('layouts.issues') %>
+          <%= link_to issues_path(:status => "open") do %>
+            <%= t("layouts.issues") %>

             <%= open_issues_count %>
           <% end -%>
         </li>
       <% end %>
             <%= open_issues_count %>
           <% end -%>
         </li>
       <% end %>

-      <li class="compact-hide <%= current_page_class(traces_path) %>"><%= link_to t('layouts.gps_traces'), traces_path %></li>
-      <li class="compact-hide <%= current_page_class(diary_path) %>"><%= link_to t('layouts.user_diaries'), diary_path %></li>
-      <li class="compact-hide <%= current_page_class(copyright_path) %>"><%= link_to t('layouts.copyright'), copyright_path %></li>
-      <li class="compact-hide <%= current_page_class(help_path) %>"><%= link_to t('layouts.help'), help_path %></li>
-      <li class="compact-hide <%= current_page_class(about_path) %>"><%= link_to t('layouts.about'), about_path %></li>
+      <li class="compact-hide <%= current_page_class(traces_path) %>"><%= link_to t("layouts.gps_traces"), traces_path %></li>
+      <li class="compact-hide <%= current_page_class(diary_entries_path) %>"><%= link_to t("layouts.user_diaries"), diary_entries_path %></li>
+      <li class="compact-hide <%= current_page_class(copyright_path) %>"><%= link_to t("layouts.copyright"), copyright_path %></li>
+      <li class="compact-hide <%= current_page_class(help_path) %>"><%= link_to t("layouts.help"), help_path %></li>
+      <li class="compact-hide <%= current_page_class(about_path) %>"><%= link_to t("layouts.about"), about_path %></li>

       <li id="compact-secondary-nav" class="dropdown">
       <li id="compact-secondary-nav" class="dropdown">

-        <a class="dropdown-toggle" data-toggle="dropdown" href="#"><%= t 'layouts.more' %> <b class="caret"></b></a>
+        <a class="dropdown-toggle" data-toggle="dropdown" href="#"><%= t "layouts.more" %> <b class="caret"></b></a>

         <ul class="dropdown-menu">
         <ul class="dropdown-menu">

-          <% if can? :index, Issue %>
+          <% if Settings.status != "database_offline" && can?(:index, Issue) %>

             <li class="<%= current_page_class(issues_path) %>">
             <li class="<%= current_page_class(issues_path) %>">

-              <%= link_to issues_path(:status => 'open') do %>
+              <%= link_to issues_path(:status => "open") do %>

                 <%= open_issues_count %>
                 <%= open_issues_count %>

-                <%= t('layouts.issues') %>
+                <%= t("layouts.issues") %>

               <% end -%>
             </li>
           <% end %>
               <% end -%>
             </li>
           <% end %>

-          <li class="<%= current_page_class(traces_path) %>"><%= link_to t('layouts.gps_traces'), traces_path %></li>
-          <li class="<%= current_page_class(diary_path) %>"><%= link_to t('layouts.user_diaries'), diary_path %></li>
-          <li class="<%= current_page_class(copyright_path) %>"><%= link_to t('layouts.copyright'), copyright_path %></li>
-          <li class="<%= current_page_class(help_path) %>"><%= link_to t('layouts.help'), help_path %></li>
-          <li class="<%= current_page_class(about_path) %>"><%= link_to t('layouts.about'), about_path %></li>
+          <li class="<%= current_page_class(traces_path) %>"><%= link_to t("layouts.gps_traces"), traces_path %></li>
+          <li class="<%= current_page_class(diary_entries_path) %>"><%= link_to t("layouts.user_diaries"), diary_entries_path %></li>
+          <li class="<%= current_page_class(copyright_path) %>"><%= link_to t("layouts.copyright"), copyright_path %></li>
+          <li class="<%= current_page_class(help_path) %>"><%= link_to t("layouts.help"), help_path %></li>
+          <li class="<%= current_page_class(about_path) %>"><%= link_to t("layouts.about"), about_path %></li>

         </ul>
       </li>
     </ul>
     <% if current_user && current_user.id %>
       <div class='dropdown user-menu logged-in'>
         <a class='dropdown-toggle' data-toggle='dropdown' href="#">
         </ul>
       </li>
     </ul>
     <% if current_user && current_user.id %>
       <div class='dropdown user-menu logged-in'>
         <a class='dropdown-toggle' data-toggle='dropdown' href="#">

-          <%= user_thumbnail_tiny(current_user, :width => 25, :height => 25)
-          %><%= render :partial => 'layouts/inbox'
-        %><span class="user-button"><span class='username'><%= current_user.display_name %></span>
-          <b class="caret"></b></span>
+          <%= user_thumbnail_tiny(current_user, :width => 25, :height => 25) %>
+          <%= render :partial => "layouts/inbox" %>
+          <span class="user-button">
+            <span class='username'>
+              <%= current_user.display_name %>
+            </span>
+            <b class="caret"></b>
+          </span>

         </a>
         <ul class='dropdown-menu'>
           <li>
             <%= link_to inbox_messages_path do %>
               <span class='count-number'><%= number_with_delimiter(current_user.new_messages.size) %></span>
         </a>
         <ul class='dropdown-menu'>
           <li>
             <%= link_to inbox_messages_path do %>
               <span class='count-number'><%= number_with_delimiter(current_user.new_messages.size) %></span>

-              <%= t('users.show.my messages') %>
+              <%= t("users.show.my messages") %>

             <% end %>
           </li>
           <li>
             <% end %>
           </li>
           <li>

-            <%= link_to t('users.show.my profile'), user_path(current_user) %>
+            <%= link_to t("users.show.my profile"), user_path(current_user) %>

           </li>
           <li>
           </li>
           <li>

-            <%= link_to t('users.show.my settings'), :controller => 'users', :action => 'account', :display_name => current_user.display_name %>
+            <%= link_to t("users.show.my settings"), :controller => "users", :action => "account", :display_name => current_user.display_name %>

           </li>
           <li class="divider"></li>
           <li>
             <%= yield :greeting %>
           </li>
           <li>
           </li>
           <li class="divider"></li>
           <li>
             <%= yield :greeting %>
           </li>
           <li>

-            <%= link_to t('layouts.logout'), logout_path(:session => session.id, :referer => request.fullpath) %>
+            <%= link_to t("layouts.logout"), logout_path(:session => session.id, :referer => request.fullpath), :class => "geolink" %>

           </li>
         </ul>
       </div>
     <% else %>
       <ul class="user-menu clearfix">
           </li>
         </ul>
       </div>
     <% else %>
       <ul class="user-menu clearfix">

-        <li><%= link_to t('layouts.log_in'), login_path(:referer => request.fullpath) %></li>
-        <li><%= link_to t('layouts.sign_up'), user_new_path %></li>
+        <li><%= link_to t("layouts.log_in"), login_path(:referer => request.fullpath), :class => "geolink" %></li>
+        <li><%= link_to t("layouts.sign_up"), user_new_path %></li>

       </ul>
     <% end %>
   </nav>
       </ul>
     <% end %>
   </nav>