diff options
| author | elijah <elijah@riseup.net> | 2013-07-04 01:45:41 -0700 | 
|---|---|---|
| committer | elijah <elijah@riseup.net> | 2013-07-04 01:45:41 -0700 | 
| commit | ab2aafa74ea98c4a7af65b87ee32487883e702c1 (patch) | |
| tree | 304b369591d08a6e3a90bf187dc3fc4ee4e43502 /users/app/controllers/email_settings_controller.rb | |
| parent | 4b6c22acb24c140860eac3ca567673ec14c3c308 (diff) | |
allow forms with blank email forward.
Diffstat (limited to 'users/app/controllers/email_settings_controller.rb')
| -rw-r--r-- | users/app/controllers/email_settings_controller.rb | 9 | 
1 files changed, 8 insertions, 1 deletions
| diff --git a/users/app/controllers/email_settings_controller.rb b/users/app/controllers/email_settings_controller.rb index 0261b47..f7d85be 100644 --- a/users/app/controllers/email_settings_controller.rb +++ b/users/app/controllers/email_settings_controller.rb @@ -8,7 +8,7 @@ class EmailSettingsController < UsersBaseController    end    def update -    @user.attributes = params[:user] +    @user.attributes = cleanup_params(params[:user])      if @user.changed?        if @user.save          flash[:notice] = t(:changes_saved) @@ -31,4 +31,11 @@ class EmailSettingsController < UsersBaseController      redirect_to edit_user_email_settings_url(@user)    end +  def cleanup_params(user) +    if !user['email_forward'].nil? && user['email_forward'].empty? +      user.delete('email_forward') # don't allow "" as an email forward +    end +    user +  end +  end | 
