suzanne.soy/travis-api
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

v3: minimal represenations should not use data from other tables

Browse Source
This commit is contained in:
Konstantin Haase 2015-10-08 18:45:18 +02:00
parent d7c9a367e9
commit 2d0bb58e35
3 changed files with 5 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
lib/travis/api/v3/renderer/build.rb
View File

@ -2,8 +2,8 @@ require 'travis/api/v3/renderer/model_renderer'
module Travis::API::V3 module Travis::API::V3
class Renderer::Build < Renderer::ModelRenderer class Renderer::Build < Renderer::ModelRenderer
representation(:minimal, :id, :number, :state, :duration, :event_type, :previous_state, :started_at, :finished_at, :jobs) representation(:minimal, :id, :number, :state, :duration, :event_type, :previous_state, :started_at, :finished_at)
representation(:standard, *representations[:minimal], :repository, :branch, :commit) representation(:standard, *representations[:minimal], :repository, :branch, :commit, :jobs)
def jobs def jobs
return model.jobs if include_full_jobs? return model.jobs if include_full_jobs?

2
lib/travis/api/v3/renderer/repository.rb
View File

@ -2,7 +2,7 @@ require 'travis/api/v3/renderer/model_renderer'
module Travis::API::V3 module Travis::API::V3
class Renderer::Repository < Renderer::ModelRenderer class Renderer::Repository < Renderer::ModelRenderer
representation(:minimal, :id, :slug, :default_branch) representation(:minimal, :id, :name, :slug)
representation(:standard, :id, :name, :slug, :description, :github_language, :active, :private, :owner, :last_build, :default_branch) representation(:standard, :id, :name, :slug, :description, :github_language, :active, :private, :owner, :last_build, :default_branch)
def active def active

2
lib/travis/api/v3/services/broadcasts/for_current_user.rb
View File

@ -1,5 +1,7 @@
module Travis::API::V3 module Travis::API::V3
class Services::Broadcasts::ForCurrentUser < Service class Services::Broadcasts::ForCurrentUser < Service
params :active, prefix: :broadcast
def run! def run!
raise LoginRequired unless access_control.logged_in? raise LoginRequired unless access_control.logged_in?
query.for_user(access_control.user) query.for_user(access_control.user)