diff options
author | George Andrinopoulos <geoandri@gmail.com> | 2017-11-09 22:34:21 +0300 |
---|---|---|
committer | George Andrinopoulos <geoandri@gmail.com> | 2017-11-11 22:06:28 +0300 |
commit | 3963f91ee355e26778dc6a6ccfd844af3cee194f (patch) | |
tree | 85001816be7b8c629e74ebf8118cd83dc0fa1e06 /app/services/issuable_base_service.rb | |
parent | d180affa210bd6e5474746bcfed006cee9d3730e (diff) |
Move update_project_counter_caches? out of issue and merge request
Diffstat (limited to 'app/services/issuable_base_service.rb')
-rw-r--r-- | app/services/issuable_base_service.rb | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/app/services/issuable_base_service.rb b/app/services/issuable_base_service.rb index 68b49d880f7..90865867ff0 100644 --- a/app/services/issuable_base_service.rb +++ b/app/services/issuable_base_service.rb @@ -187,7 +187,7 @@ class IssuableBaseService < BaseService # We have to perform this check before saving the issuable as Rails resets # the changed fields upon calling #save. - update_project_counters = issuable.project && issuable.update_project_counter_caches? + update_project_counters = issuable.project && update_project_counter_caches?(issuable) if issuable.with_transaction_returning_status { issuable.save } # We do not touch as it will affect a update on updated_at field @@ -288,4 +288,8 @@ class IssuableBaseService < BaseService # override if needed def execute_hooks(issuable, action = 'open', params = {}) end + + def update_project_counter_caches?(issuable) + issuable.state_changed? + end end |