summaryrefslogtreecommitdiff
path: root/app
diff options
context:
space:
mode:
authorFelipe Artur <fcardozo@gitlab.com>2019-07-09 15:21:10 +0000
committerDmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>2019-07-09 15:21:10 +0000
commitbdbaebf87e95d513baaf9c3b735ba7420a1aa6aa (patch)
treeb3d27c131a0c3025571aba3c4bf7b41c39231641 /app
parent54820ac996f29c4c6af084def7d5efdaf70b3f05 (diff)
downloadgitlab-ce-bdbaebf87e95d513baaf9c3b735ba7420a1aa6aa.tar.gz
Skip spam check for task list updates
Task list updates should not mark users action as spam on akismet.
Diffstat (limited to 'app')
-rw-r--r--app/services/issuable_base_service.rb4
-rw-r--r--app/services/issues/update_service.rb4
2 files changed, 4 insertions, 4 deletions
diff --git a/app/services/issuable_base_service.rb b/app/services/issuable_base_service.rb
index 02de080e0ba..db673cace81 100644
--- a/app/services/issuable_base_service.rb
+++ b/app/services/issuable_base_service.rb
@@ -182,7 +182,7 @@ class IssuableBaseService < BaseService
# To be overridden by subclasses
end
- def before_update(issuable)
+ def before_update(issuable, skip_spam_check: false)
# To be overridden by subclasses
end
@@ -257,7 +257,7 @@ class IssuableBaseService < BaseService
last_edited_at: Time.now,
last_edited_by: current_user))
- before_update(issuable)
+ before_update(issuable, skip_spam_check: true)
if issuable.with_transaction_returning_status { issuable.save }
# We do not touch as it will affect a update on updated_at field
diff --git a/app/services/issues/update_service.rb b/app/services/issues/update_service.rb
index 6b9f23f24cd..7cd825aa967 100644
--- a/app/services/issues/update_service.rb
+++ b/app/services/issues/update_service.rb
@@ -17,8 +17,8 @@ module Issues
super
end
- def before_update(issue)
- spam_check(issue, current_user)
+ def before_update(issue, skip_spam_check: false)
+ spam_check(issue, current_user) unless skip_spam_check
end
def handle_changes(issue, options)