Code cleanup.

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

diff --git a/app/views/account/password_recovery.html.erb b/app/views/account/password_recovery.html.erb
index 53c6350505f4f3d8dc388068862ed93f8afb1e7b..24da8223b5cb8ae97d464323c402b29cd85ee76b 100644 (file)
--- a/app/views/account/password_recovery.html.erb
+++ b/app/views/account/password_recovery.html.erb
@@ -2,14 +2,19 @@
 
 <%= error_messages_for 'user' %>
 
-<%= form_tag({:token => @token.value}) do %>
-<div class="box tabular">
-<p><label for="new_password"><%=l(:field_new_password)%> <span class="required">*</span></label>
-<%= password_field_tag 'new_password', nil, :size => 25 %>
-<em class="info"><%= l(:text_caracters_minimum, :count => Setting.password_min_length) %></em></p>
+<%= form_tag(lost_password_path) do %>
+  <%= hidden_field_tag 'token', @token.value %>
+  <div class="box tabular">
+    <p>
+      <label for="new_password"><%=l(:field_new_password)%> <span class="required">*</span></label>
+      <%= password_field_tag 'new_password', nil, :size => 25 %>
+      <em class="info"><%= l(:text_caracters_minimum, :count => Setting.password_min_length) %></em>
+    </p>
 
-<p><label for="new_password_confirmation"><%=l(:field_password_confirmation)%> <span class="required">*</span></label>
-<%= password_field_tag 'new_password_confirmation', nil, :size => 25 %></p>
-</div>
-<p><%= submit_tag l(:button_save) %></p>
+    <p>
+      <label for="new_password_confirmation"><%=l(:field_password_confirmation)%> <span class="required">*</span></label>
+      <%= password_field_tag 'new_password_confirmation', nil, :size => 25 %>
+    </p>
+  </div>
+  <p><%= submit_tag l(:button_save) %></p>
 <% end %>

diff --git a/test/integration/account_test.rb b/test/integration/account_test.rb
index 9f73be2c2b0993d6005136ba28f20c96f7aeac07..d50bb894e518fa34723623d1f1914cc2c1e19150 100644 (file)
--- a/test/integration/account_test.rb
+++ b/test/integration/account_test.rb
@@ -76,6 +76,7 @@ class AccountTest < ActionController::IntegrationTest
     get "account/lost_password"
     assert_response :success
     assert_template "account/lost_password"
+    assert_select 'input[name=mail]'
 
     post "account/lost_password", :mail => 'jSmith@somenet.foo'
     assert_redirected_to "/login"
@@ -88,6 +89,9 @@ class AccountTest < ActionController::IntegrationTest
     get "account/lost_password", :token => token.value
     assert_response :success
     assert_template "account/password_recovery"
+    assert_select 'input[type=hidden][name=token][value=?]', token.value
+    assert_select 'input[name=new_password]'
+    assert_select 'input[name=new_password_confirmation]'
 
     post "account/lost_password", :token => token.value, :new_password => 'newpass', :new_password_confirmation => 'newpass'
     assert_redirected_to "/login"