diff options
| author | James Lopez <james@jameslopez.es> | 2017-07-06 15:19:14 +0200 |
|---|---|---|
| committer | James Lopez <james@jameslopez.es> | 2017-07-06 15:27:49 +0200 |
| commit | 2aa95aa0185c94aefd6e51a35c62650d75d3c8b6 (patch) | |
| tree | b1f7811bab8da709de2f5d6f26fb6c47e1840813 /lib/api | |
| parent | 47e17fe12ad009656f0b20469f930d1aa0e742e6 (diff) | |
| download | gitlab-ce-2aa95aa0185c94aefd6e51a35c62650d75d3c8b6.tar.gz | |
refactor code based on feedback
Diffstat (limited to 'lib/api')
| -rw-r--r-- | lib/api/entities.rb | 2 | ||||
| -rw-r--r-- | lib/api/issues.rb | 3 | ||||
| -rw-r--r-- | lib/api/snippets.rb | 3 |
3 files changed, 3 insertions, 5 deletions
diff --git a/lib/api/entities.rb b/lib/api/entities.rb index 33620473175..44b8c6e010d 100644 --- a/lib/api/entities.rb +++ b/lib/api/entities.rb @@ -879,7 +879,7 @@ module API class UserAgentDetail < Grape::Entity expose :user_agent expose :ip_address - expose :submitted + expose :submitted, as: :akismet_submitted end end end diff --git a/lib/api/issues.rb b/lib/api/issues.rb index 88dd31739da..64be08094ed 100644 --- a/lib/api/issues.rb +++ b/lib/api/issues.rb @@ -253,10 +253,9 @@ module API issue = find_project_issue(params[:issue_iid]) - return not_found!('Issue') unless issue return not_found!('UserAgentDetail') unless issue.user_agent_detail - present issue.user_agent_detail, with: Entities::UserAgentDetail, current_user: current_user, project: user_project + present issue.user_agent_detail, with: Entities::UserAgentDetail end end end diff --git a/lib/api/snippets.rb b/lib/api/snippets.rb index db4e0b0b013..fd634037a77 100644 --- a/lib/api/snippets.rb +++ b/lib/api/snippets.rb @@ -150,9 +150,8 @@ module API get ":id/user_agent_detail" do authenticated_as_admin! - snippet = Snippet.find_by(id: params[:id]) + snippet = Snippet.find_by!(id: params[:id]) - return not_found!('Snippet') unless snippet return not_found!('UserAgentDetail') unless snippet.user_agent_detail present snippet.user_agent_detail, with: Entities::UserAgentDetail |