diff options
| author | Timothy Andrew <mail@timothyandrew.net> | 2017-04-06 13:06:36 +0300 |
|---|---|---|
| committer | Timothy Andrew <mail@timothyandrew.net> | 2017-04-06 16:29:17 +0300 |
| commit | 3e1a1242c67781fb52940433c5ad1bbefd346216 (patch) | |
| tree | 5f45e1f2dfee8ac69c0ee6340e2cc01ac5273817 /app/models/award_emoji.rb | |
| parent | 682987547a932c011f84c6455f0fd32bb500b308 (diff) | |
Move a user's award emoji to the ghost user
... when the user is destroyed.
1. Normally, for a given awardable and award emoji name, a user is only allowed
to create a single award emoji.
2. This validation needs to be removed for ghost users, since:
- User A and User B have created award emoji - with the same name and against
the same awardable
- User A is deleted. Their award emoji is moved to the ghost user
- User B is deleted. Their award emoji needs to be moved to the ghost user.
However, this breaks the uniqueness validation, since the ghost user is
only allowed to have one award emoji of a given name for a given awardable
Diffstat (limited to 'app/models/award_emoji.rb')
| -rw-r--r-- | app/models/award_emoji.rb | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/app/models/award_emoji.rb b/app/models/award_emoji.rb index 6937ad3bdd9..6ada6fae4eb 100644 --- a/app/models/award_emoji.rb +++ b/app/models/award_emoji.rb @@ -3,13 +3,14 @@ class AwardEmoji < ActiveRecord::Base UPVOTE_NAME = "thumbsup".freeze include Participable + include GhostUser belongs_to :awardable, polymorphic: true belongs_to :user validates :awardable, :user, presence: true validates :name, presence: true, inclusion: { in: Gitlab::Emoji.emojis_names } - validates :name, uniqueness: { scope: [:user, :awardable_type, :awardable_id] } + validates :name, uniqueness: { scope: [:user, :awardable_type, :awardable_id] }, unless: :ghost_user? participant :user |