]> source.dussan.org Git - redmine.git/commitdiff
Removed TabularFormBuilder references in views.
authorJean-Philippe Lang <jp_lang@yahoo.fr>
Fri, 9 Dec 2011 18:40:28 +0000 (18:40 +0000)
committerJean-Philippe Lang <jp_lang@yahoo.fr>
Fri, 9 Dec 2011 18:40:28 +0000 (18:40 +0000)
git-svn-id: svn+ssh://rubyforge.org/var/svn/redmine/trunk@8142 e93f8b46-1217-0410-a6f0-8f06a7374b81

13 files changed:
app/helpers/application_helper.rb
app/views/issue_statuses/edit.html.erb
app/views/issue_statuses/new.html.erb
app/views/issues/_attributes.html.erb
app/views/issues/_edit.html.erb
app/views/my/account.html.erb
app/views/projects/settings/_repository.html.erb
app/views/projects/settings/_wiki.html.erb
app/views/trackers/edit.html.erb
app/views/trackers/new.html.erb
app/views/users/_general.html.erb
app/views/users/_preferences.html.erb
app/views/users/new.html.erb

index 58e1cf8ec2987904ebedc9efdbcdcdf54a4575e9..ddfc44a8cb0f9c3dbad1a8d88a86707b4710eb64 100644 (file)
@@ -882,6 +882,20 @@ module ApplicationHelper
     form_for(*args, &proc)
   end
 
+  def labelled_fields_for(*args, &proc)
+    args << {} unless args.last.is_a?(Hash)
+    options = args.last
+    options.merge!({:builder => TabularFormBuilder})
+    fields_for(*args, &proc)
+  end
+
+  def labelled_remote_form_for(*args, &proc)
+    args << {} unless args.last.is_a?(Hash)
+    options = args.last
+    options.merge!({:builder => TabularFormBuilder})
+    remote_form_for(*args, &proc)
+  end
+
   def back_url_hidden_field_tag
     back_url = params[:back_url] || request.env['HTTP_REFERER']
     back_url = CGI.unescape(back_url.to_s)
index 82a1a2e554373affe52d2be79bcac73890cc99a4..584fe06a90e1a2e94bd89474b62490c66db7a55e 100644 (file)
@@ -1,6 +1,6 @@
 <h2><%= link_to l(:label_issue_status_plural), issue_statuses_path %> &#187; <%=h @issue_status %></h2>
 
-<% form_for @issue_status, :builder => TabularFormBuilder do |f| %>
+<% labelled_form_for @issue_status do |f| %>
   <%= render :partial => 'form', :locals => {:f => f} %>
   <%= submit_tag l(:button_save) %>
 <% end %>
index cefd493b4268e21f48c9cf0d31a9b212e6182816..6ab77b2146959fb1c5aee3285b64ccbce83a7485 100644 (file)
@@ -1,6 +1,6 @@
 <h2><%= link_to l(:label_issue_status_plural), issue_statuses_path %> &#187; <%=l(:label_issue_status_new)%></h2>
 
-<% form_for @issue_status, :builder => TabularFormBuilder do |f| %>
+<% labelled_form_for @issue_status do |f| %>
   <%= render :partial => 'form', :locals => {:f => f} %>
   <%= submit_tag l(:button_create) %>
 <% end %>
index 76bb624a3733bf65e43da461c2d4d9c5a2a05086..c4a087daeef03d3ee86dd96ae8d71a703edde78f 100644 (file)
@@ -1,4 +1,4 @@
-<% fields_for :issue, @issue, :builder => TabularFormBuilder do |f| %>
+<% labelled_fields_for :issue, @issue do |f| %>
 
 <div class="splitcontentleft">
 <% if @issue.new_record? || @allowed_statuses.any? %>
index 022d159f14575132e7b71f2e4edf2a171bfadb4d..ffffeb175ee1e6daee6bcc3e45f119434adc5a4e 100644 (file)
@@ -8,7 +8,7 @@
     <% end %>
     <% if User.current.allowed_to?(:log_time, @project) %>
         <fieldset class="tabular"><legend><%= l(:button_log_time) %></legend>
-        <% fields_for :time_entry, @time_entry, { :builder => TabularFormBuilder, :lang => current_language} do |time_entry| %>
+        <% labelled_fields_for :time_entry, @time_entry do |time_entry| %>
         <div class="splitcontentleft">
         <p><%= time_entry.text_field :hours, :size => 6, :label => :label_spent_time %> <%= l(:field_hours) %></p>
         </div>
index e3bcd4504dc6dcc0c3bb5bf978ecb48deaf43a37..f59c3dd1d7461b36cde9ffd5b68122c294164fc0 100644 (file)
@@ -6,9 +6,7 @@
 <h2><%=l(:label_my_account)%></h2>
 <%= error_messages_for 'user' %>
 
-<% form_for :user, @user, :url => { :action => "account" },
-                          :builder => TabularFormBuilder,
-                          :lang => current_language,
+<% labelled_form_for :user, @user, :url => { :action => "account" },
                           :html => { :id => 'my_account_form' } do |f| %>
 <div class="splitcontentleft">
 <fieldset class="box tabular">
index cba6c4e166096927a8c5f46cf2e7a9e8f13f6e2a..bbfc105de04050703840e1071ac9692a4f8d7e68 100644 (file)
@@ -1,7 +1,5 @@
-<% remote_form_for :repository, @repository,
-                   :url => { :controller => 'repositories', :action => 'edit', :id => @project },
-                   :builder => TabularFormBuilder,
-                   :lang => current_language do |f| %>
+<% labelled_remote_form_for :repository, @repository,
+                   :url => { :controller => 'repositories', :action => 'edit', :id => @project } do |f| %>
 
 <%= error_messages_for 'repository' %>
 
index 00da9303956052f8151554e5a4482501d8adc71d..e627adc07cbea4b0ea9f37086cff6e250cf480bd 100644 (file)
@@ -1,7 +1,5 @@
-<% remote_form_for :wiki, @wiki,
-                   :url => { :controller => 'wikis', :action => 'edit', :id => @project },
-                   :builder => TabularFormBuilder,
-                   :lang => current_language do |f| %>
+<% labelled_remote_form_for :wiki, @wiki,
+                   :url => { :controller => 'wikis', :action => 'edit', :id => @project } do |f| %>
 
 <%= error_messages_for 'wiki' %>
 
index d7ba2591c233eba103cabce2466b3a2204c8f9e8..b553704a02e6459f99c26b006d6b6a8860817530 100644 (file)
@@ -1,5 +1,5 @@
 <h2><%= link_to l(:label_tracker_plural), trackers_path %> &#187; <%=h @tracker %></h2>
 
-<% form_for @tracker, :builder => TabularFormBuilder do |f| %>
+<% labelled_form_for @tracker do |f| %>
 <%= render :partial => 'form', :locals => { :f => f } %>
 <% end %>
index 93b0bc1ae60fd3c33d46d9964ad9c91cb2be4ed0..e4852579320af98abd9d4319521eb7eb7ff852ac 100644 (file)
@@ -1,5 +1,5 @@
 <h2><%= link_to l(:label_tracker_plural), trackers_path %> &#187; <%=l(:label_tracker_new)%></h2>
 
-<% form_for @tracker, :builder => TabularFormBuilder do |f| %>
+<% labelled_form_for @tracker do |f| %>
 <%= render :partial => 'form', :locals => { :f => f } %>
 <% end %>
index de81e0365b391207db9702bf84fb9bffa8f8cf15..b332e856e09116c83786fbf2533806e2b19cfba6 100644 (file)
@@ -1,4 +1,4 @@
-<% form_for @user, :builder => TabularFormBuilder do |f| %>
+<% labelled_form_for @user do |f| %>
   <%= render :partial => 'form', :locals => { :f => f } %>
   <% if @user.active? && email_delivery_enabled? -%>
     <p><label><%= check_box_tag 'send_information', 1, true %> <%= l(:label_send_information) %></label></p>
index 57f050b102510708f3c53045e28da302afc35f94..a5a0e5cb133abe3d05beefcb09c6bd6badb5b211 100644 (file)
@@ -1,4 +1,4 @@
-<% fields_for :pref, @user.pref, :builder => TabularFormBuilder, :lang => current_language do |pref_fields| %>
+<% labelled_fields_for :pref, @user.pref do |pref_fields| %>
 <p><%= pref_fields.check_box :hide_mail %></p>
 <p><%= pref_fields.select :time_zone, ActiveSupport::TimeZone.all.collect {|z| [ z.to_s, z.name ]}, :include_blank => true %></p>
 <p><%= pref_fields.select :comments_sorting, [[l(:label_chronological_order), 'asc'], [l(:label_reverse_chronological_order), 'desc']] %></p>
index fc58b09885293ae3599541c9e08692a9af273da2..eb8a9f93d43086a6bbbefe8dae9fdd9bfc33e448 100644 (file)
@@ -1,6 +1,6 @@
 <h2><%= link_to l(:label_user_plural), users_path %> &#187; <%=l(:label_user_new)%></h2>
 
-<% form_for @user, :builder => TabularFormBuilder do |f| %>
+<% labelled_form_for @user do |f| %>
   <%= render :partial => 'form', :locals => { :f => f } %>
   <% if email_delivery_enabled? %>
   <p><label><%= check_box_tag 'send_information', 1, true %> <%= l(:label_send_information) %></label></p>