Merge branch 'master' into cd-v3-parity
This commit is contained in:
commit
2974dccd3b
|
@ -50,7 +50,7 @@ GIT
|
||||||
|
|
||||||
GIT
|
GIT
|
||||||
remote: git://github.com/travis-ci/travis-core.git
|
remote: git://github.com/travis-ci/travis-core.git
|
||||||
revision: 3c20e66a20d9d540e2a6c5fb70c542fe97c62e25
|
revision: 9a9f4caa2187734d34e98aa61ba5f4a61bc8a9f5
|
||||||
specs:
|
specs:
|
||||||
travis-core (0.0.1)
|
travis-core (0.0.1)
|
||||||
actionmailer (~> 3.2.19)
|
actionmailer (~> 3.2.19)
|
||||||
|
|
|
@ -8,7 +8,6 @@ module Travis::API::V3
|
||||||
user = Models::User.find(user.id) if user.is_a? ::User
|
user = Models::User.find(user.id) if user.is_a? ::User
|
||||||
@user = user
|
@user = user
|
||||||
@access_permissions = user.permissions.where(user_id: user.id)
|
@access_permissions = user.permissions.where(user_id: user.id)
|
||||||
@got_request = false
|
|
||||||
super()
|
super()
|
||||||
end
|
end
|
||||||
|
|
||||||
|
@ -21,7 +20,6 @@ module Travis::API::V3
|
||||||
end
|
end
|
||||||
|
|
||||||
def visible_repositories(list)
|
def visible_repositories(list)
|
||||||
load_permissions
|
|
||||||
list.where('repositories.private = false OR repositories.id IN (?)'.freeze, access_permissions.map(&:repository_id))
|
list.where('repositories.private = false OR repositories.id IN (?)'.freeze, access_permissions.map(&:repository_id))
|
||||||
end
|
end
|
||||||
|
|
||||||
|
@ -49,19 +47,7 @@ module Travis::API::V3
|
||||||
|
|
||||||
def permission?(type, id)
|
def permission?(type, id)
|
||||||
id = id.id if id.is_a? ::Repository
|
id = id.id if id.is_a? ::Repository
|
||||||
|
access_permissions.where(type => true, :repository_id => id).any?
|
||||||
load_permissions if @got_request
|
|
||||||
@got_request = true
|
|
||||||
|
|
||||||
if access_permissions.respond_to? :where
|
|
||||||
access_permissions.where(type => true, :repository_id => id).any?
|
|
||||||
else
|
|
||||||
access_permissions.any? { |p| p[type] == true and p.repository_id == id }
|
|
||||||
end
|
|
||||||
end
|
|
||||||
|
|
||||||
def load_permissions
|
|
||||||
@access_permissions = @access_permissions.to_a
|
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
|
|
@ -50,5 +50,17 @@ module Travis::API::V3
|
||||||
rescue ActiveRecord::RecordNotUnique
|
rescue ActiveRecord::RecordNotUnique
|
||||||
branches.where(name: name).first
|
branches.where(name: name).first
|
||||||
end
|
end
|
||||||
|
|
||||||
|
def id_default_branch
|
||||||
|
[id, default_branch_name]
|
||||||
|
end
|
||||||
|
|
||||||
|
def send(name, *args, &block)
|
||||||
|
if name == [:id, :default_branch]
|
||||||
|
name = :id_default_branch
|
||||||
|
end
|
||||||
|
|
||||||
|
__send__(name, *args, &block)
|
||||||
|
end
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
|
Loading…
Reference in New Issue
Block a user