Merge pull request openstreetmap#5689 from tomhughes/settings-fixes

User settings fixes

Author: gravitystorm

app/controllers/accounts_controller.rb

@@ -31,7 +31,7 @@ def update
        (params[:user][:auth_provider] == current_user.auth_provider &&
         params[:user][:auth_uid] == current_user.auth_uid)
       update_user(current_user, user_params)
-      if current_user.errors.count.zero?
+      if current_user.errors.empty?
         redirect_to edit_account_path
       else
         render :edit

app/controllers/concerns/user_methods.rb

@@ -59,8 +59,9 @@ def update_user(user, params)
             # Ignore errors sending email
           end
         else
-          current_user.errors.add(:new_email, current_user.errors[:email])
-          current_user.errors.add(:email, [])
+          current_user.errors.delete(:email).each do |error|
+            current_user.errors.add(:new_email, error)
+          end
         end
 
         user.restore_email!