diff options
author | Patricio Cano <suprnova32@gmail.com> | 2016-07-21 04:15:17 +0300 |
---|---|---|
committer | Patricio Cano <suprnova32@gmail.com> | 2016-07-26 23:18:07 +0300 |
commit | 8f04cf0eadbcde7fc5d1c970741e30ca8b97967d (patch) | |
tree | 4eac04baa5093bc4fd97691e2d6219d09810b55f /app/services/spam_check_service.rb | |
parent | f7807c5b68b59f6a5b984ee64a6c82a3bd993d92 (diff) |
Refactor `SpamCheckService` to make it cleaner and clearer.
Diffstat (limited to 'app/services/spam_check_service.rb')
-rw-r--r-- | app/services/spam_check_service.rb | 28 |
1 files changed, 28 insertions, 0 deletions
diff --git a/app/services/spam_check_service.rb b/app/services/spam_check_service.rb new file mode 100644 index 00000000000..6768047aa63 --- /dev/null +++ b/app/services/spam_check_service.rb @@ -0,0 +1,28 @@ +class SpamCheckService + include Gitlab::AkismetHelper + + attr_accessor :subject, :current_user, :params + + def initialize(subject, user, params = {}) + @subject, @current_user, @params = subject, user, params.dup + end + + def spam_detected? + request = params[:request] + return false unless request || check_for_spam?(subject) + + text = [params[:title], params[:description]].reject(&:blank?).join("\n") + + return false unless is_spam?(request.env, current_user, text) + + attrs = { + user_id: current_user.id, + project_id: subject.id, + title: params[:title], + description: params[:description] + } + create_spam_log(subject, current_user, attrs, request.env, api: false) + + true + end +end |