diff options
| author | Alexis Reigel <alexis.reigel.ext@siemens.com> | 2017-12-04 12:51:39 +0100 |
|---|---|---|
| committer | Alexis Reigel <alexis.reigel.ext@siemens.com> | 2018-01-17 09:55:00 +0100 |
| commit | a78abf3a0071d1705adc562f23be1f3a347c6db5 (patch) | |
| tree | 553bac3b43e07507aabe3d833b988015e8749f85 /app/models/project.rb | |
| parent | ac92d70d9025e1c90bffa99c08bfc4cdb2fc36c9 (diff) | |
| download | gitlab-ce-a78abf3a0071d1705adc562f23be1f3a347c6db5.tar.gz | |
use safer .hooks_for instead of .public_send
with .public_send we can't make sure that the scope on the model
actually exists.
Diffstat (limited to 'app/models/project.rb')
| -rw-r--r-- | app/models/project.rb | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/app/models/project.rb b/app/models/project.rb index a8c634200ce..602c6a2795f 100644 --- a/app/models/project.rb +++ b/app/models/project.rb @@ -970,7 +970,7 @@ class Project < ActiveRecord::Base def execute_hooks(data, hooks_scope = :push_hooks) run_after_commit_or_now do - hooks.public_send(hooks_scope).each do |hook| # rubocop:disable GitlabSecurity/PublicSend + hooks.hooks_for(hooks_scope).each do |hook| hook.async_execute(data, hooks_scope.to_s) end end |