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

fix syntax of db queries, add output

Browse Source
This commit is contained in:
carlad 2015-11-19 17:26:30 +01:00
parent 16687f16d7
commit 2420784a87
2 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
lib/travis/api/v3/services/repository/star.rb
View File

@ -4,7 +4,7 @@ module Travis::API::V3
raise LoginRequired unless access_control.logged_in? or access_control.full_access? raise LoginRequired unless access_control.logged_in? or access_control.full_access?
raise NotFound unless repository = find(:repository) raise NotFound unless repository = find(:repository)
starred = Models::StarredRepository.where(repository_id: repository.id, user_id: access_control.user.id).first starred = Models::StarredRepository.where(repository_id: repository.id, user_id: access_control.user.id).first
raise AlreadyStarred unless starred.nil? raise AlreadyStarred unless starred == nil
Models::StarredRepository.create(repository_id: repository.id, user_id: access_control.user.id) Models::StarredRepository.create(repository_id: repository.id, user_id: access_control.user.id)
repository #TODO what do we want to return??? repository #TODO what do we want to return???

6
lib/travis/api/v3/services/repository/unstar.rb
View File

@ -3,9 +3,9 @@ module Travis::API::V3
def run! def run!
raise LoginRequired unless access_control.logged_in? or access_control.full_access? raise LoginRequired unless access_control.logged_in? or access_control.full_access?
raise NotFound unless repository = find(:repository) raise NotFound unless repository = find(:repository)
starred_repo = Models::StarredRepository.where(repository_id: repository.id, user_id: access_control.user.id) starred = Models::StarredRepository.where(repository_id: repository.id, user_id: access_control.user.id).first
raise NotStarred unless !starred_repo.nil? raise NotStarred if starred == nil
starred_repo.first.delete starred.delete
repository #TODO what do we want to return??? repository #TODO what do we want to return???
end end