Merge pull request #3647 from marpo60/3636-adds-password-confirmation-field

adds password_confirmation field to registration page

Changelog.md

@@ -1,5 +1,6 @@
 # 0.0.2.0pre
 
+Add password_confirmation field to registration page
 Fix error with open/close registrations.
 Fix javascripts error in invitations facebox. 
 Fix css overflow problem in aspect dropdown on welcome page.

app/assets/stylesheets/new_styles/_registration.scss

@@ -39,4 +39,16 @@
 
     max-width : 95%;
   }
-}
+
+  form {
+    .control-label,
+    .controls {
+      margin-left : auto;
+      width : auto;
+    }
+
+    .controls {
+      float : right;
+    }
+  }
+}

app/controllers/registrations_controller.rb

@@ -18,10 +18,10 @@ class RegistrationsController < Devise::RegistrationsController
       Rails.logger.info("event=registration status=successful user=#{@user.diaspora_handle}")
     else
       @user.errors.delete(:person)
-      
+
       flash[:error] = @user.errors.full_messages.join(" - ")
       Rails.logger.info("event=registration status=failure errors='#{@user.errors.full_messages.join(', ')}'")
-      render :new
+      redirect_to :back
     end
   end
 
@@ -50,6 +50,6 @@ class RegistrationsController < Devise::RegistrationsController
       @invite ||= InvitationCode.find_by_token(params[:invite][:token])
     end
   end
-  
+
   helper_method :invite
 end

app/views/registrations/new.html.erb

@@ -50,6 +50,16 @@
                         <%= f.password_field :password, :placeholder => "••••••••", :title => t('registrations.new.enter_password'), :required => true, :pattern => "......+" %>
                     </div>
                   </div>
+
+                  <div class="control-group">
+                    <label class="control-label" for="user_password_confirmation">
+                        <%= t('.password_confirmation') %>
+                    </label>
+
+                    <div class="controls">
+                        <%= f.password_field :password_confirmation, :placeholder => "••••••••", :title => t('registrations.new.enter_password_again'), :required => true, :pattern => "......+" %>
+                    </div>
+                  </div>
                   <%= invite_hidden_tag(invite) %>
                 </fieldset>
 

config/locales/diaspora/en.yml

@@ -687,6 +687,7 @@ en:
       email: "EMAIL"
       username: "USERNAME"
       password: "PASSWORD"
+      password_confirmation: "PASSWORD CONFIRMATION"
       continue: "Continue"
     create:
       success: "You've joined Diaspora!"

features/accepts_invitation.feature

@@ -7,6 +7,7 @@ Feature: invitation acceptance
         | user_username              | ohai           |
         | user_email                 | woot@sweet.com |
         | user_password              | secret         |
+        | user_password_confirmation | secret         |
       And I press "Continue"
       Then I should be on the getting started page
       And I should see "Well, hello there!"
@@ -24,6 +25,7 @@ Feature: invitation acceptance
         | user_username              | ohai           |
         | user_email                 | woot@sweet.com |
         | user_password              | secret         |
+        | user_password_confirmation | secret         |
       And I press "Continue"
       Then I should be on the getting started page
       And I should see "Well, hello there!"

features/signs_up.feature

@@ -4,9 +4,10 @@ Feature: new user registration
   Background:
     When I go to the new user registration page
     And I fill in the following:
-        | user_username     |     ohai              |
-        | user_email        |   ohai@example.com    |
-        | user_password     |     secret            |
+        | user_username              |     ohai              |
+        | user_email                 |   ohai@example.com    |
+        | user_password              |     secret            |
+        | user_password_confirmation |     secret            |
     And I press "Continue"
     Then I should be on the getting started page
     And I should see "Well, hello there!" and "Who are you?" and "What are you into?"

features/step_definitions/user_steps.rb

@@ -209,6 +209,7 @@ When /^I fill in the new user form$/ do
   step 'I fill in "user_username" with "ohai"'
   step 'I fill in "user_email" with "ohai@example.com"'
   step 'I fill in "user_password" with "secret"'
+  step 'I fill in "user_password_confirmation" with "secret"'
 end
 
 And /^I should be able to friend Alice$/ do

spec/controllers/registrations_controller_spec.rb

@@ -112,9 +112,9 @@ describe RegistrationsController do
         flash[:error].should_not be_blank
       end
 
-      it "re-renders the form" do
+      it "redirects back" do
         get :create, @invalid_params
-        response.should render_template("registrations/new")
+        response.should be_redirect
       end
     end
   end