Commit 11848feb authored by Kirill Zaitsev's avatar Kirill Zaitsev

Add issueable actor

parent 43be3fcb
...@@ -131,9 +131,10 @@ module Issuable ...@@ -131,9 +131,10 @@ module Issuable
users.concat(mentions.reduce([], :|)).uniq users.concat(mentions.reduce([], :|)).uniq
end end
def to_hook_data def to_hook_data(user)
{ {
object_kind: self.class.name.underscore, object_kind: self.class.name.underscore,
user: user.hook_attrs,
object_attributes: hook_attrs object_attributes: hook_attrs
} }
end end
......
...@@ -488,6 +488,14 @@ class User < ActiveRecord::Base ...@@ -488,6 +488,14 @@ class User < ActiveRecord::Base
end end
end end
def hook_attrs
{
name: name,
username: username,
avatar_url: avatar_url
}
end
def ensure_namespace_correct def ensure_namespace_correct
# Ensure user has namespace # Ensure user has namespace
self.create_namespace!(path: self.username, name: self.username) unless self.namespace self.create_namespace!(path: self.username, name: self.username) unless self.namespace
......
...@@ -8,7 +8,7 @@ module Issues ...@@ -8,7 +8,7 @@ module Issues
end end
def execute_hooks(issue, action = 'open') def execute_hooks(issue, action = 'open')
issue_data = issue.to_hook_data issue_data = issue.to_hook_data(current_user)
issue_url = Gitlab::UrlBuilder.new(:issue).build(issue.id) issue_url = Gitlab::UrlBuilder.new(:issue).build(issue.id)
issue_data[:object_attributes].merge!(url: issue_url, action: action) issue_data[:object_attributes].merge!(url: issue_url, action: action)
issue.project.execute_hooks(issue_data, :issue_hooks) issue.project.execute_hooks(issue_data, :issue_hooks)
......
...@@ -13,7 +13,8 @@ module MergeRequests ...@@ -13,7 +13,8 @@ module MergeRequests
def execute_project_hooks(merge_request) def execute_project_hooks(merge_request)
if merge_request.project if merge_request.project
merge_request.project.execute_hooks(merge_request.to_hook_data, :merge_request_hooks) hook_data = merge_request.to_hook_data(current_user)
merge_request.project.execute_hooks(hook_data, :merge_request_hooks)
end end
end end
end end
......
...@@ -13,7 +13,8 @@ module MergeRequests ...@@ -13,7 +13,8 @@ module MergeRequests
def execute_hooks(merge_request) def execute_hooks(merge_request)
if merge_request.project if merge_request.project
merge_request.project.execute_hooks(merge_request.to_hook_data, :merge_request_hooks) hook_data = merge_request.to_hook_data(current_user)
merge_request.project.execute_hooks(hook_data, :merge_request_hooks)
end end
end end
......
...@@ -63,6 +63,11 @@ Triggered when a new issue is created or an existing issue was updated/closed/re ...@@ -63,6 +63,11 @@ Triggered when a new issue is created or an existing issue was updated/closed/re
```json ```json
{ {
"object_kind": "issue", "object_kind": "issue",
"user": {
"name": "Administrator",
"username": "root",
"avatar_url": "http://www.gravatar.com/avatar/e64c7d89f26bd1972efa854d13d7dd61?s=40\u0026d=identicon"
},
"object_attributes": { "object_attributes": {
"id": 301, "id": 301,
"title": "New API: create/update/delete file", "title": "New API: create/update/delete file",
...@@ -92,6 +97,11 @@ Triggered when a new merge request is created or an existing merge request was u ...@@ -92,6 +97,11 @@ Triggered when a new merge request is created or an existing merge request was u
```json ```json
{ {
"object_kind": "merge_request", "object_kind": "merge_request",
"user": {
"name": "Administrator",
"username": "root",
"avatar_url": "http://www.gravatar.com/avatar/e64c7d89f26bd1972efa854d13d7dd61?s=40\u0026d=identicon"
},
"object_attributes": { "object_attributes": {
"id": 99, "id": 99,
"target_branch": "master", "target_branch": "master",
......
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