Rename IssuableBaseService#have_changes? to has_changes?

parent a7dee0ab
...@@ -71,7 +71,7 @@ class IssuableBaseService < BaseService ...@@ -71,7 +71,7 @@ class IssuableBaseService < BaseService
end end
end end
def have_changes?(issuable, options = {}) def has_changes?(issuable, options = {})
valid_attrs = [:title, :description, :assignee_id, :milestone_id, :target_branch] valid_attrs = [:title, :description, :assignee_id, :milestone_id, :target_branch]
attrs_changed = valid_attrs.any? do |attr| attrs_changed = valid_attrs.any? do |attr|
......
...@@ -5,7 +5,7 @@ module Issues ...@@ -5,7 +5,7 @@ module Issues
end end
def handle_changes(issue, options = {}) def handle_changes(issue, options = {})
if have_changes?(issue, options) if has_changes?(issue, options)
task_service.mark_pending_tasks_as_done(issue, current_user) task_service.mark_pending_tasks_as_done(issue, current_user)
end end
......
...@@ -15,7 +15,7 @@ module MergeRequests ...@@ -15,7 +15,7 @@ module MergeRequests
end end
def handle_changes(merge_request, options = {}) def handle_changes(merge_request, options = {})
if have_changes?(merge_request, options) if has_changes?(merge_request, options)
task_service.mark_pending_tasks_as_done(merge_request, current_user) task_service.mark_pending_tasks_as_done(merge_request, current_user)
end end
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment