diff options
| author | Douglas Barbosa Alexandre <dbalexandre@gmail.com> | 2016-02-17 19:33:30 +0300 |
|---|---|---|
| committer | Douglas Barbosa Alexandre <dbalexandre@gmail.com> | 2016-02-20 17:10:26 +0300 |
| commit | 56a5fc0cd331143c9a2b4215db3dba6cefd6cc3e (patch) | |
| tree | 0fd7d0de89d1712d26e63b344776f76ff6fd17b5 /app/services/issuable_base_service.rb | |
| parent | 372cdf4ce5fa124e03e2c93e9d1ed586d489b1d8 (diff) | |
Move common behavior to to IssuableBaseService
Diffstat (limited to 'app/services/issuable_base_service.rb')
| -rw-r--r-- | app/services/issuable_base_service.rb | 13 |
1 files changed, 13 insertions, 0 deletions
diff --git a/app/services/issuable_base_service.rb b/app/services/issuable_base_service.rb index 63025c10fb0..fef96639ace 100644 --- a/app/services/issuable_base_service.rb +++ b/app/services/issuable_base_service.rb @@ -71,6 +71,19 @@ class IssuableBaseService < BaseService end end + def have_changes?(issuable, options = {}) + valid_attrs = [:title, :description, :assignee_id, :milestone_id, :target_branch] + + attrs_changed = valid_attrs.any? do |attr| + issuable.previous_changes.include?(attr.to_s) + end + + old_labels = options[:old_labels] + labels_changed = old_labels && issuable.labels != old_labels + + attrs_changed || labels_changed + end + def handle_common_system_notes(issuable, options = {}) if issuable.previous_changes.include?('title') create_title_change_note(issuable, issuable.previous_changes['title'].first) |