Adds dynamic columns selection on the issue list (#4272).

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

3 files changed, 19 insertions, 11 deletions

diff --git a/app/views/issues/index.rhtml b/app/views/issues/index.rhtml
index 5eddb7052..5b8fa05a7 100644
--- a/app/views/issues/index.rhtml
+++ b/app/views/issues/index.rhtml
@@ -20,8 +20,16 @@
     <fieldset class="collapsible collapsed">
     	<legend onclick="toggleFieldset(this);"><%= l(:label_options) %></legend>
     	<div style="display: none;">
-				<%= l(:field_group_by) %>
-				<%= select_tag('group_by', options_for_select([[]] + @query.groupable_columns.collect {|c| [c.caption, c.name.to_s]}, @query.group_by)) %>
+    		<table>
+    			<tr>
+    				<td><%= l(:field_column_names) %></td>
+						<td><%= render :partial => 'queries/columns', :locals => {:query => @query} %></td>
+					</tr>
+					<tr>
+						<td><%= l(:field_group_by) %></td>
+						<td><%= select_tag('group_by', options_for_select([[]] + @query.groupable_columns.collect {|c| [c.caption, c.name.to_s]}, @query.group_by)) %></td>
+					</tr>
+				</table>
 			</div>
 		</fieldset>
 		</div>
@@ -29,6 +37,7 @@
 
     <%= link_to_remote l(:button_apply), 
                        { :url => { :set_filter => 1 },
+                         :before => 'selectAllOptions("selected_columns");',
                          :update => "content",
                          :with => "Form.serialize('query_form')"
                        }, :class => 'icon icon-checked' %>
@@ -40,7 +49,7 @@
                        }, :class => 'icon icon-reload'  %>
                        
     <% 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' %>
+    <%= link_to l(:button_save), {}, :onclick => "selectAllOptions('selected_columns'); $('query_form').submit(); return false;", :class => 'icon icon-save' %>
     <% end %>
     </p>
 <% end %>
diff --git a/app/views/queries/_columns.rhtml b/app/views/queries/_columns.rhtml
index 1a481adae..e11b286b6 100644
--- a/app/views/queries/_columns.rhtml
+++ b/app/views/queries/_columns.rhtml
@@ -1,10 +1,6 @@
-<% content_tag 'fieldset', :id => 'columns', :style => (query.has_default_columns? ? 'display:none;' : nil) do %>
-<legend><%= l(:field_column_names) %></legend>
-
-<%= hidden_field_tag 'query[column_names][]', '', :id => nil %>
-<table>
+<table style="border-collapse: collapse; border:0;">
 	<tr>
-		<td><%= select_tag 'available_columns',
+		<td style="padding-left:0"><%= select_tag 'available_columns',
 		          options_for_select((query.available_columns - query.columns).collect {|column| [column.caption, column.name]}),
 		          :multiple => true, :size => 10, :style => "width:150px" %>
 		</td>
@@ -15,12 +11,11 @@
 			 onclick="moveOptions(this.form.selected_columns, this.form.available_columns);" />
 		</td>
 		<td><%= select_tag 'query[column_names][]',
-		          options_for_select(@query.columns.collect {|column| [column.caption, column.name]}),
+		          options_for_select(query.columns.collect {|column| [column.caption, column.name]}),
 		          :id => 'selected_columns', :multiple => true, :size => 10, :style => "width:150px" %>
 		</td>
 	</tr>
 </table>
-<% end %>
 
 <% content_for :header_tags do %>
 <%= javascript_include_tag 'select_list_move' %>
diff --git a/app/views/queries/_form.rhtml b/app/views/queries/_form.rhtml
index 577699d70..dcafe9f60 100644
--- a/app/views/queries/_form.rhtml
+++ b/app/views/queries/_form.rhtml
@@ -37,5 +37,9 @@
 <% end %>
 </fieldset>
 
+<% content_tag 'fieldset', :id => 'columns', :style => (query.has_default_columns? ? 'display:none;' : nil) do %>
+<legend><%= l(:field_column_names) %></legend>
 <%= render :partial => 'queries/columns', :locals => {:query => query}%>
+<% end %>
+
 </div>