diff options
author | Lin Jen-Shin <godfat@godfat.org> | 2017-06-15 16:36:23 +0300 |
---|---|---|
committer | Lin Jen-Shin <godfat@godfat.org> | 2017-06-15 16:36:23 +0300 |
commit | c791ca7bc23ebc26a5a39c57f36f9e9b8434fa1f (patch) | |
tree | 1650e54ca038acc95fffade9685506aac1a01dc0 /app/services/issuable_base_service.rb | |
parent | 039c375372f76050c48d6e5c39bb8322c8bf7dc5 (diff) |
Use the same way to invalidate cache in EE
Because it doesn't really hurt and could reduce conflicts
Discussion:
https://gitlab.com/gitlab-org/gitlab-ee/commit/5020e1a779cecbf16fffa3b37726b9167093126c#note_32403646
Diffstat (limited to 'app/services/issuable_base_service.rb')
-rw-r--r-- | app/services/issuable_base_service.rb | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/app/services/issuable_base_service.rb b/app/services/issuable_base_service.rb index a65d6e11c47..cd4d180824f 100644 --- a/app/services/issuable_base_service.rb +++ b/app/services/issuable_base_service.rb @@ -236,8 +236,9 @@ class IssuableBaseService < BaseService ) if old_assignees != issuable.assignees - assignees = old_assignees + issuable.assignees.to_a - invalidate_cache_counts(assignees.compact, issuable) + new_assignees = issuable.assignees.to_a + affected_assignees = (old_assignees + new_assignees) - (old_assignees & new_assignees) + invalidate_cache_counts(affected_assignees.compact, issuable) end after_update(issuable) |