diff options
| author | Hannes Rosenögger <123haynes@gmail.com> | 2015-05-12 14:46:58 +0000 |
|---|---|---|
| committer | Hannes Rosenögger <123haynes@gmail.com> | 2015-05-12 14:46:58 +0000 |
| commit | 1dd0caf4d95c6d09fe54041fd7aff9f8661bfbef (patch) | |
| tree | 9af0408d3129cc4bf59abff98bdd1af069afa892 /app/models/user.rb | |
| parent | 8b9e3af8bf7a6089a11ef3f5eb91dfef4940a3f4 (diff) | |
| parent | bf4b4384590c271d1dfadf874d185c2f6130ad0e (diff) | |
| download | gitlab-ce-1dd0caf4d95c6d09fe54041fd7aff9f8661bfbef.tar.gz | |
Merge branch 'fix-avatar-removal' into 'master'
Fix bug where avatar filenames were not actually deleted from the database during removal
This would result in a 404 error in certain views.
The `save` call was being rolled back due to an error in the validation step.
Relax the validation step so that this works.
Closes #1570
See merge request !620
Diffstat (limited to 'app/models/user.rb')
| -rw-r--r-- | app/models/user.rb | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/app/models/user.rb b/app/models/user.rb index d088d2d8630..4dd37e73564 100644 --- a/app/models/user.rb +++ b/app/models/user.rb @@ -148,7 +148,7 @@ class User < ActiveRecord::Base validates :notification_level, inclusion: { in: Notification.notification_levels }, presence: true validate :namespace_uniq, if: ->(user) { user.username_changed? } - validate :avatar_type, if: ->(user) { user.avatar_changed? } + validate :avatar_type, if: ->(user) { user.avatar.present? && user.avatar_changed? } validate :unique_email, if: ->(user) { user.email_changed? } validate :owns_notification_email, if: ->(user) { user.notification_email_changed? } validate :owns_public_email, if: ->(user) { user.public_email_changed? } @@ -309,7 +309,7 @@ class User < ActiveRecord::Base if primary_email_record primary_email_record.destroy self.emails.create(email: self.email_was) - + self.update_secondary_emails! end end |