diff options
| author | Douwe Maan <douwe@gitlab.com> | 2016-02-12 18:11:01 +0000 |
|---|---|---|
| committer | Douwe Maan <douwe@gitlab.com> | 2016-02-12 18:11:01 +0000 |
| commit | 31c8dbc96fae558a794de18dbb5f06c2dc1a6286 (patch) | |
| tree | 403b680d15e85c2eb08bc816576dccb1c84ec01f /lib/api/files.rb | |
| parent | c142bde51d05ad6fe2137c1b6bb0cd2485b4f1d9 (diff) | |
| parent | 4603e5714705da75fda4ef4857c327ac4b087848 (diff) | |
| download | gitlab-ce-31c8dbc96fae558a794de18dbb5f06c2dc1a6286.tar.gz | |
Merge branch 'fix-duplicate-hook-notifications' into 'master'
Fix duplicated branch creation/deletion Web hooks/service notifications when using Web UI
Similar to 423d2d6, except duplicates occurred only if a Web service (e.g. Slack) were
configured.
When deleting a branch, this is what was happening:
1. DeleteBranchService calls execute_hooks and execute_services
2. The call to repository.rm_branch triggers the GitHooksService.
3. This, in turn, calls GitPushService and then calls the same hooks/services again.
5145706 now makes it no longer necessary for DeleteBranchService and CreateBranchService to execute
the branch hooks/services. Note that tags behave differently in GitTagPushService and GitPushService
is not called.
Closes #10330
See merge request !2763
Diffstat (limited to 'lib/api/files.rb')
0 files changed, 0 insertions, 0 deletions