Use <%= instead of <% for block helpers that use concat

diff --git a/app/views/diary_entry/edit.html.erb b/app/views/diary_entry/edit.html.erb
index b5abd667b3bb148a596e9f7065328823f08a9f90..8728427be852a8d96a5b885c8264fb3324bc866c 100644 (file)
--- a/app/views/diary_entry/edit.html.erb
+++ b/app/views/diary_entry/edit.html.erb
@@ -2,7 +2,7 @@
 
 <%= error_messages_for 'diary_entry' %>
 
-<% form_for :diary_entry do |f| %>
+<%= form_for :diary_entry do |f| %>
   <table class="diary_entry">
     <tr valign="top">
       <td class="fieldName"><%= t 'diary_entry.edit.subject' -%></td>

diff --git a/app/views/diary_entry/view.html.erb b/app/views/diary_entry/view.html.erb
index 3bc9323e3352b006dfbe6670937dfa601130f160..2261e256831b5b023ca9210336f9000a22a5ccce 100644 (file)
--- a/app/views/diary_entry/view.html.erb
+++ b/app/views/diary_entry/view.html.erb
@@ -13,7 +13,7 @@
 
   <%= error_messages_for 'diary_comment' %>
 
-  <% form_for :diary_comment, @diary_comment, :url => { :action => 'comment' } do |f| %>
+  <%= form_for :diary_comment, @diary_comment, :url => { :action => 'comment' } do |f| %>
     <%= f.text_area :body, :cols => 80, :rows => 5 %>
     <br />
     <br />

diff --git a/app/views/export/_start.html.erb b/app/views/export/_start.html.erb
index 722ba9be574145b3f1b4cdfc5e81d7e00a83ae5a..fce26902beff0a8c4ca535ce8780e7b7eff2296f 100644 (file)
--- a/app/views/export/_start.html.erb
+++ b/app/views/export/_start.html.erb
@@ -1,4 +1,4 @@
-<% form_tag :action => "finish" do %>
+<%= form_tag :action => "finish" do %>
 
   <p class="export_heading"><%= t'export.start.area_to_export' %></p>
 

diff --git a/app/views/message/new.html.erb b/app/views/message/new.html.erb
index 85de1e51b708f1936daf7c344edaa290c8e8d7a9..f2005bb00c1d13aa5558e57881bed3f997c70758 100644 (file)
--- a/app/views/message/new.html.erb
+++ b/app/views/message/new.html.erb
@@ -2,7 +2,7 @@
 
 <%= error_messages_for 'message' %>
 
-<% form_for :message, :url => { :action => "new", :display_name => @to_user.display_name } do |f| %>
+<%= form_for :message, :url => { :action => "new", :display_name => @to_user.display_name } do |f| %>
   <table>
     <tr valign="top">
       <td class="fieldName"><%= t'message.new.subject' %></td>

diff --git a/app/views/oauth/oauthorize.html.erb b/app/views/oauth/oauthorize.html.erb
index 28178a85c945560a0d840a57b6b384d4e484c8c4..ffb403be17c8a0951bb6f00d508674e9bade0f6f 100644 (file)
--- a/app/views/oauth/oauthorize.html.erb
+++ b/app/views/oauth/oauthorize.html.erb
@@ -1,6 +1,6 @@
 <h1>Authorize access to your account</h1>
 <p><%= t('oauth.oauthorize.request_access', :app_name => link_to(@token.client_application.name, @token.client_application.url), :user => link_to(@user.display_name, :controller => :user, :action => :view, :display_name => @user.display_name)) %></p>
-<% form_tag authorize_url do %>
+<%= form_tag authorize_url do %>
   <%= hidden_field_tag "oauth_token", @token.token %>
   <%- if params[:oauth_callback] -%>
     <%= hidden_field_tag "oauth_callback", params[:oauth_callback] %>

diff --git a/app/views/oauth_clients/edit.html.erb b/app/views/oauth_clients/edit.html.erb
index 085ddef4ec5dbf9d1c24c4a625624005ed971da8..bee1bd183d43ae91a89c15134a18b5d2851750a9 100644 (file)
--- a/app/views/oauth_clients/edit.html.erb
+++ b/app/views/oauth_clients/edit.html.erb
@@ -1,5 +1,5 @@
 <h1><%= t'oauth_clients.edit.title' %></h1>
-<% form_for :client_application, @client_application, :url => oauth_client_path(@client_application.user.display_name, @client_application), :html => { :method => :put } do |f| %>
+<%= form_for :client_application, @client_application, :url => oauth_client_path(@client_application.user.display_name, @client_application), :html => { :method => :put } do |f| %>
   <%= render :partial => "form", :locals => { :f => f } %>
   <br/>
   <%= submit_tag t'oauth_clients.edit.submit' %>

diff --git a/app/views/oauth_clients/index.html.erb b/app/views/oauth_clients/index.html.erb
index d95c68bcb09d9a41552109fbaf252546b4eef9a0..91c71f90b0a0266a1b918b0651bb6c2b9f0be092 100644 (file)
--- a/app/views/oauth_clients/index.html.erb
+++ b/app/views/oauth_clients/index.html.erb
@@ -6,11 +6,11 @@
   <tr><th><%= t'oauth_clients.index.application' %></th>
     <th><%= t'oauth_clients.index.issued_at' %></th><th>&nbsp;</th></tr>
   <% @tokens.each do |token|%>
-    <% content_tag_for :tr, token do %>
+    <%= content_tag_for :tr, token do %>
       <td><%= link_to token.client_application.name, token.client_application.url %></td>
       <td><%= token.authorized_at %></td>
       <td>
-       <% form_tag :controller => 'oauth', :action => 'revoke' do %>
+       <%= form_tag :controller => 'oauth', :action => 'revoke' do %>
        <%= hidden_field_tag 'token', token.token %>
        <%= submit_tag t('oauth_clients.index.revoke') %>
        <% end %>
@@ -25,7 +25,7 @@
 <% else %>
 <p><%= t'oauth_clients.index.registered_apps' %></p>
 <% @client_applications.each do |client|%>
-  <% div_for client do %>
+  <%= div_for client do %>
     <%= link_to client.name, :action => :show, :id => client.id %>
   <% end %>
 <% end %>

diff --git a/app/views/oauth_clients/new.html.erb b/app/views/oauth_clients/new.html.erb
index 292c53fc0e0433353f8e0c28d64b8414a8a7c1f9..2cd748ac653620bb8f32b7082b0d8409175972bc 100644 (file)
--- a/app/views/oauth_clients/new.html.erb
+++ b/app/views/oauth_clients/new.html.erb
@@ -1,5 +1,5 @@
 <h1><%= t'oauth_clients.new.title' %></h1>
-<% form_for :client_application, :url => { :action => :create } do |f| %>
+<%= form_for :client_application, :url => { :action => :create } do |f| %>
    <%= render :partial => "form", :locals => { :f => f } %>
    <br />
    <%= submit_tag t('oauth_clients.new.submit') %>

diff --git a/app/views/trace/create.html.erb b/app/views/trace/create.html.erb
index 4b168fab0d11a8ccfa6220aa73987e8a8c01b4f6..f4815a9460ed9ae3e37c99076cf916418e753d5d 100644 (file)
--- a/app/views/trace/create.html.erb
+++ b/app/views/trace/create.html.erb
@@ -2,7 +2,7 @@
 
 <%= error_messages_for 'trace' %>
 
-<% form_for :trace, @trace, :url => { :action => "create" }, :html => { :multipart => true } do |f| %>
+<%= form_for :trace, @trace, :url => { :action => "create" }, :html => { :multipart => true } do |f| %>
 <table>
   <tr><td class="fieldName"><%= t'trace.trace_form.upload_gpx' %></td><td><%= f.file_field :gpx_file, :size => 50, :maxlength => 255 %></td></tr>
   <tr><td class="fieldName"><%= t'trace.trace_form.description' %></td><td><%= f.text_field :description, :size => 50, :maxlength => 255 %></td></tr>

diff --git a/app/views/trace/edit.html.erb b/app/views/trace/edit.html.erb
index 202f9039d573b5a81ed954818c91f2af6ee40941..e1382a1be570dc37b980596f4046f9948eadbdbd 100644 (file)
--- a/app/views/trace/edit.html.erb
+++ b/app/views/trace/edit.html.erb
@@ -2,7 +2,7 @@
 
 <img src="<%= url_for :controller => 'trace', :action => 'picture', :id => @trace.id, :display_name => @trace.user.display_name %>">
 
-<% form_for :trace, @trace do |f| %>
+<%= form_for :trace, @trace do |f| %>
 
 <table border="0">
   <tr>

diff --git a/app/views/user/account.html.erb b/app/views/user/account.html.erb
index bdc52fb8c1056a2dc6701fa10ee0a4b04eb22443..606e8c35830532af1f6425a8e3abf11a3054e1bb 100644 (file)
--- a/app/views/user/account.html.erb
+++ b/app/views/user/account.html.erb
@@ -1,6 +1,6 @@
 <h2><%= t 'user.account.my settings' %></h2>
 <%= error_messages_for 'user' %>
-<% form_for :user, :html => { :multipart => true } do |f| %>
+<%= form_for :user, :html => { :multipart => true } do |f| %>
 <table id="accountForm">
   <tr>
     <td class="fieldName"><%= t 'user.new.display name' %></td>

diff --git a/app/views/user/list.html.erb b/app/views/user/list.html.erb
index fac7c442fd375d5152628b9aedda4ae0db226641..e90a6de8b744c43edbe7da0d7d75f051f84d19b5 100644 (file)
--- a/app/views/user/list.html.erb
+++ b/app/views/user/list.html.erb
@@ -3,7 +3,7 @@
 <h1><%= t('user.list.heading') %></h1>
 
 <% unless @users.empty? %>
-  <% form_tag do %>
+  <%= form_tag do %>
     <%= hidden_field_tag :status, params[:status] if params[:status] %>
     <%= hidden_field_tag :ip, params[:ip] if params[:ip] %>
     <%= hidden_field_tag :page, params[:page] if params[:page] %>

diff --git a/app/views/user/login.html.erb b/app/views/user/login.html.erb
index 2d90ba7cfd1d5a69ea8eced427b210a04b576ed4..8c079d0f1374e8e140e02b22506623f943eb87b4 100644 (file)
--- a/app/views/user/login.html.erb
+++ b/app/views/user/login.html.erb
@@ -3,7 +3,7 @@
   <div id="login_login">
     <h1><%= t 'user.login.heading' %></h1>
 
-    <% form_tag({ :action => "login" }, { :id => "login_form" }) do %>
+    <%= form_tag({ :action => "login" }, { :id => "login_form" }) do %>
       <%= hidden_field_tag('referer', h(params[:referer])) %>
 
       <p><%= t 'user.login.with username' %></p>

diff --git a/app/views/user/logout.html.erb b/app/views/user/logout.html.erb
index e6d0dec597ba1ff9eff6595ac98e1db9684921ce..8cf7624749611fb87e9dd74b5344e56cead1e206 100644 (file)
--- a/app/views/user/logout.html.erb
+++ b/app/views/user/logout.html.erb
@@ -1,5 +1,5 @@
 <h1><%= t 'user.logout.heading' %></h1>
-<% form_tag :action => "logout" do %>
+<%= form_tag :action => "logout" do %>
   <%= hidden_field_tag("referer", h(params[:referer])) %>
   <%= hidden_field_tag("session", request.session_options[:id]) %>
   <%= submit_tag t('user.logout.logout_button') %>

diff --git a/app/views/user/lost_password.html.erb b/app/views/user/lost_password.html.erb
index cdee822e4f68f3540800de6d0da88fbe94461b36..db1ee742d251e88046e7dd4878dc041bf5c5b1f9 100644 (file)
--- a/app/views/user/lost_password.html.erb
+++ b/app/views/user/lost_password.html.erb
@@ -2,7 +2,7 @@
 
 <p><%= t 'user.lost_password.help_text' %></p>
 
-<% form_tag :action => 'lost_password' do %>
+<%= form_tag :action => 'lost_password' do %>
 <table>
   <tr>
     <td class="fieldName"><%= t 'user.lost_password.email address' %></td>

diff --git a/app/views/user/new.html.erb b/app/views/user/new.html.erb
index 730676355a8531aaa4601479d33f402de9a31c69..a988afd80a946dff19d64833d01d9070cfa3c5c2 100644 (file)
--- a/app/views/user/new.html.erb
+++ b/app/views/user/new.html.erb
@@ -12,7 +12,7 @@
 
 <%= error_messages_for 'user' %>
 
-<% form_tag :action => 'terms' do %>
+<%= form_tag :action => 'terms' do %>
   <%= hidden_field_tag('referer', h(@referer)) unless @referer.nil? %>
 
   <table id="signupForm">

diff --git a/app/views/user/reset_password.html.erb b/app/views/user/reset_password.html.erb
index bb00e8a6e83181c0789b99caa7d9a67e3ba6f43a..adb714b00d26560a511f27676f06ee259c95afc5 100644 (file)
--- a/app/views/user/reset_password.html.erb
+++ b/app/views/user/reset_password.html.erb
@@ -2,7 +2,7 @@
 
 <%= error_messages_for :user %>
 
-<% form_tag do %>
+<%= form_tag do %>
 <%= hidden_field_tag(:token, params[:token]) %>
 <table id="loginForm">
   <tr><td class="fieldName"><%= t 'user.reset_password.password' %></td><td><%= password_field(:user, :pass_crypt, {:value => '', :size => 30, :maxlength => 255, :tabindex => 4}) %></td></tr>

diff --git a/app/views/user/terms.html.erb b/app/views/user/terms.html.erb
index e4644e8151758a6180acc98d07ced0db70d0863f..db052bb24f8afdb5502d7fafd864424d92cfab26 100644 (file)
--- a/app/views/user/terms.html.erb
+++ b/app/views/user/terms.html.erb
@@ -3,7 +3,7 @@
 <p><%= t 'user.terms.read and accept' %></p>
 
 <!-- legale is <%= @legale %> -->
-<% form_tag :action => 'terms' do %>
+<%= form_tag :action => 'terms' do %>
   <p>
     <%= t 'user.terms.legale_select' %>
     <% [['france', 'FR'], ['italy', 'IT'], ['rest_of_world', 'GB']].each do |name,legale| %>
@@ -25,7 +25,7 @@
   <%= render :partial => "terms" %>
 </div>
 
-<% form_tag({:action => "save"}, { :id => "termsForm" }) do %>
+<%= form_tag({:action => "save"}, { :id => "termsForm" }) do %>
   <p>
     <label for="confirm_pd_checkbox"><%= t 'user.terms.consider_pd' %></label>
     <%= check_box('user', 'consider_pd') %>

diff --git a/app/views/user_blocks/edit.html.erb b/app/views/user_blocks/edit.html.erb
index 387335d9d07e4b6e20248e38228bee58c5078bb6..777022dab8cf416541e57cc95f03d7930b34d009 100644 (file)
--- a/app/views/user_blocks/edit.html.erb
+++ b/app/views/user_blocks/edit.html.erb
@@ -4,7 +4,7 @@
                            h(@user_block.user.display_name),
                            {:controller => 'user', :action => 'view', :display_name => @user_block.user.display_name})) %></h1>
 
-<% form_for(@user_block) do |f| %>
+<%= form_for(@user_block) do |f| %>
   <%= f.error_messages %>
 
   <p>

diff --git a/app/views/user_blocks/new.html.erb b/app/views/user_blocks/new.html.erb
index 3d3e6855c5cf835d8bffc82eafbd5a882de849be..739d0b08bb2b437952d58504e9b46e63f0a7baca 100644 (file)
--- a/app/views/user_blocks/new.html.erb
+++ b/app/views/user_blocks/new.html.erb
@@ -4,7 +4,7 @@
                            h(@this_user.display_name),
                            {:controller => 'user', :action => 'view', :display_name => @this_user.display_name})) %></h1>
 
-<% form_for(@user_block) do |f| %>
+<%= form_forevxr(@user_block) do |f| %>
   <%= f.error_messages %>
 
   <p>

diff --git a/app/views/user_blocks/revoke.html.erb b/app/views/user_blocks/revoke.html.erb
index 9f142c80271d3339ddaf19b8cdc0ebc1171b469f..0819357d393693ba8ab669ec8df815f294de8cf2 100644 (file)
--- a/app/views/user_blocks/revoke.html.erb
+++ b/app/views/user_blocks/revoke.html.erb
@@ -14,7 +14,7 @@
   <%= t('user_block.revoke.time_future', :time => distance_of_time_in_words_to_now(@user_block.ends_at)) %>
 </b></p>
 
-<% form_for :revoke, :url => { :action => "revoke" } do |f| %>
+<%= form_for :revoke, :url => { :action => "revoke" } do |f| %>
   <%= f.error_messages %>
 <p>  
   <%= check_box_tag 'confirm', 'yes' %>

diff --git a/app/views/user_roles/grant.html.erb b/app/views/user_roles/grant.html.erb
index cee4112b7fd868882b41980f1c5687009303b4ff..f88ec23847448a8e12fff7aff9c597ed4e99d36f 100644 (file)
--- a/app/views/user_roles/grant.html.erb
+++ b/app/views/user_roles/grant.html.erb
@@ -1,4 +1,4 @@
-<% form_tag request.request_uri do %>
+<%= form_tag request.request_uri do %>
 <%= hidden_field_tag 'nonce', @nonce %>
 <% @title = t('user_role.grant.heading') %>
 <h1><%= t('user_role.grant.heading') %></h1>

diff --git a/app/views/user_roles/revoke.html.erb b/app/views/user_roles/revoke.html.erb
index e5aadeae08da62b0fc2c57450b4c50cc23b3eedb..a31a6d81a6a8a0f1fa393f9e014842bced77c16d 100644 (file)
--- a/app/views/user_roles/revoke.html.erb
+++ b/app/views/user_roles/revoke.html.erb
@@ -1,4 +1,4 @@
-<% form_tag request.request_uri do %>
+<%= form_tag request.request_uri do %>
 <%= hidden_field_tag 'nonce', @nonce %>
 <% @title = t('user_role.revoke.heading') %>
 <h1><%= t('user_role.revoke.heading') %></h1>