Use an handler instead of onsubmit attribute.

git-svn-id: http://svn.redmine.org/redmine/trunk@13616 e93f8b46-1217-0410-a6f0-8f06a7374b81

diff --git a/app/views/queries/edit.html.erb b/app/views/queries/edit.html.erb
index a1534f82dad986dd390d9410e11a730d7097ff9e..eba719fa1428ceffbcc21db27f635d1d46e63dc3 100644 (file)
--- a/app/views/queries/edit.html.erb
+++ b/app/views/queries/edit.html.erb
@@ -1,6 +1,6 @@
 <h2><%= l(:label_query) %></h2>
 
-<%= form_tag(query_path(@query), :onsubmit => 'selectAllOptions("selected_columns");', :method => :put) do %>
+<%= form_tag(query_path(@query), :method => :put) do %>
   <%= render :partial => 'form', :locals => {:query => @query} %>
   <%= submit_tag l(:button_save) %>
 <% end %>

diff --git a/app/views/queries/new.html.erb b/app/views/queries/new.html.erb
index 7081eee246ed983fc3ceacc654d9f132478e4fb7..3c1ff3b1c459b979fa0d0d99a4a47eddb598a758 100644 (file)
--- a/app/views/queries/new.html.erb
+++ b/app/views/queries/new.html.erb
@@ -1,6 +1,6 @@
 <h2><%= l(:label_query_new) %></h2>
 
-<%= form_tag(@project ? project_queries_path(@project) : queries_path, :onsubmit => 'selectAllOptions("selected_columns");') do %>
+<%= form_tag(@project ? project_queries_path(@project) : queries_path) do %>
   <%= render :partial => 'form', :locals => {:query => @query} %>
   <%= submit_tag l(:button_save) %>
 <% end %>

diff --git a/app/views/settings/_issues.html.erb b/app/views/settings/_issues.html.erb
index 83b4acd9819ab1735b6a78642ca5284c6d601f54..fac6d6dea7f7b037e108849cdd89cdfb64c5a232 100644 (file)
--- a/app/views/settings/_issues.html.erb
+++ b/app/views/settings/_issues.html.erb
@@ -1,4 +1,4 @@
-<%= form_tag({:action => 'edit', :tab => 'issues'}, :onsubmit => 'selectAllOptions("selected_columns");') do %>
+<%= form_tag({:action => 'edit', :tab => 'issues'}) do %>
 
 <div class="box tabular settings">
 <p><%= setting_check_box :cross_project_issue_relations %></p>

diff --git a/public/javascripts/application.js b/public/javascripts/application.js
index 38193b8a03868f3f0ebb000b97e78a66ddcc2eb0..1577da6b85497d5a9251308b45e5ddcf5d6095ae 100644 (file)
--- a/public/javascripts/application.js
+++ b/public/javascripts/application.js
@@ -288,7 +288,6 @@ function toggleMultiSelect(el) {
 }
 
 function submit_query_form(id) {
-  selectAllOptions("selected_columns");
   $('#'+id).submit();
 }
 

diff --git a/public/javascripts/select_list_move.js b/public/javascripts/select_list_move.js
index d542ac6245ee465ef7506e4b3bac9b1ced0a89ca..db05854243f593c546dabda458509b2c5cbed440 100644 (file)
--- a/public/javascripts/select_list_move.js
+++ b/public/javascripts/select_list_move.js
@@ -22,6 +22,8 @@ function moveOptionBottom(theSel) {
   $(theSel).find('option:selected').detach().appendTo($(theSel));
 }
 
-function selectAllOptions(id) {
-  $('#'+id).find('option').prop('selected', true);
-}
+$(document).ready(function(){
+  $('.query-columns').closest('form').submit(function(){
+    $('#selected_columns option').prop('selected', true);
+  });
+});