Makes saved query filters visible (#2883).

git-svn-id: svn+ssh://rubyforge.org/var/svn/redmine/trunk@2892 e93f8b46-1217-0410-a6f0-8f06a7374b81

diff --git a/app/views/issues/calendar.rhtml b/app/views/issues/calendar.rhtml
index 000e3cdc47d1c5a4318e263695aab1612f41ad12..5e7ee31b5c0ed664d31b032f0833c780042198bb 100644 (file)
--- a/app/views/issues/calendar.rhtml
+++ b/app/views/issues/calendar.rhtml
@@ -1,16 +1,12 @@
-<% form_tag({}, :id => 'query_form') do %>
-<% if @query.new_record? %>
 <h2><%= l(:label_calendar) %></h2>
+
+<% form_tag({}, :id => 'query_form') do %>
 <fieldset id="filters" class="collapsible">
   <legend onclick="toggleFieldset(this);"><%= l(:label_filter_plural) %></legend>
   <div>
     <%= render :partial => 'queries/filters', :locals => {:query => @query} %>
   </div>
 </fieldset>
-<% else %>
-    <h2><%=h @query.name %></h2>
-    <% html_title @query.name %>
-<% end %>
 
 <p style="float:right;">
 <%= link_to_remote ('&#171; ' + (@month==1 ? "#{month_name(12)} #{@year-1}" : "#{month_name(@month-1)}")), 

diff --git a/app/views/issues/gantt.rhtml b/app/views/issues/gantt.rhtml
index e1526cc1ca86b1308006bed7a8ad34091a442f69..6dc4e7a4b3b4e0a7fc6c8cc7988a04ea4b345ef0 100644 (file)
--- a/app/views/issues/gantt.rhtml
+++ b/app/views/issues/gantt.rhtml
@@ -1,16 +1,12 @@
+<h2><%= l(:label_gantt) %></h2>
+
 <% form_tag(params.merge(:month => nil, :year => nil, :months => nil), :id => 'query_form') do %>
-<% if @query.new_record? %>
-<h2><%=l(:label_gantt)%></h2>
 <fieldset id="filters" class="collapsible">
   <legend onclick="toggleFieldset(this);"><%= l(:label_filter_plural) %></legend>
        <div>
     <%= render :partial => 'queries/filters', :locals => {:query => @query} %>
   </div>
 </fieldset>
-<% else %>
-    <h2><%=h @query.name %></h2>
-    <% html_title @query.name %>
-<% end %>
 
 <p style="float:right;">
 <%= if @gantt.zoom < 4

diff --git a/app/views/issues/index.rhtml b/app/views/issues/index.rhtml
index 408a00eaae546418e5d975b02beb4cfec0607860..d09ebfe066e490ce584f55332e9bad4baed2ad9e 100644 (file)
--- a/app/views/issues/index.rhtml
+++ b/app/views/issues/index.rhtml
@@ -1,8 +1,14 @@
-<% if @query.new_record? %>
-    <h2><%=l(:label_issue_plural)%></h2>
-    <% html_title(l(:label_issue_plural)) %>
-    
-    <% form_tag({ :controller => 'queries', :action => 'new' }, :id => 'query_form') do %>
+<div class="contextual">
+<% if !@query.new_record? && @query.editable_by?(User.current) %>
+       <%= link_to l(:button_edit), {:controller => 'queries', :action => 'edit', :id => @query}, :class => 'icon icon-edit' %>
+       <%= link_to l(:button_delete), {:controller => 'queries', :action => 'destroy', :id => @query}, :confirm => l(:text_are_you_sure), :method => :post, :class => 'icon icon-del' %>
+<% end %>
+</div>
+
+<h2><%= @query.new_record? ? l(:label_issue_plural) : h(@query.name) %></h2>
+<% html_title(@query.new_record? ? l(:label_issue_plural) : @query.name) %>
+
+<% form_tag({ :controller => 'queries', :action => 'new' }, :id => 'query_form') do %>
     <%= hidden_field_tag('project_id', @project.to_param) if @project %>
                <div id="query_form_content">
     <fieldset id="filters" class="collapsible">
@@ -33,21 +39,10 @@
                          :update => "content",
                        }, :class => 'icon icon-reload'  %>
                        
-    <% if User.current.allowed_to?(:save_queries, @project, :global => true) %>
+    <% if @query.new_record? && User.current.allowed_to?(:save_queries, @project, :global => true) %>
     <%= link_to l(:button_save), {}, :onclick => "$('query_form').submit(); return false;", :class => 'icon icon-save' %>
     <% end %>
     </p>
-    <% end %>
-<% else %>
-    <div class="contextual">
-    <% if @query.editable_by?(User.current) %>
-    <%= link_to l(:button_edit), {:controller => 'queries', :action => 'edit', :id => @query}, :class => 'icon icon-edit' %>
-    <%= link_to l(:button_delete), {:controller => 'queries', :action => 'destroy', :id => @query}, :confirm => l(:text_are_you_sure), :method => :post, :class => 'icon icon-del' %>
-    <% end %>
-    </div>
-    <h2><%=h @query.name %></h2>
-    <div id="query_form"></div>
-    <% html_title @query.name %>
 <% end %>
 
 <%= error_messages_for 'query' %>