Commit 84a3f8fca4e57422267bdd7e07d4f1b90717fbcc
1 parent
7b5c3cc8
Exists in
master
and in
4 other branches
show only current user projects
Showing
2 changed files
with
4 additions
and
5 deletions
Show diff stats
lib/api.rb
@@ -41,19 +41,19 @@ module Gitlab | @@ -41,19 +41,19 @@ module Gitlab | ||
41 | 41 | ||
42 | # GET /projects/:id | 42 | # GET /projects/:id |
43 | get ":id" do | 43 | get ":id" do |
44 | - @project = Project.find_by_code(params[:id]) | 44 | + @project = current_user.projects.find_by_code(params[:id]) |
45 | present @project, :with => Entities::Project | 45 | present @project, :with => Entities::Project |
46 | end | 46 | end |
47 | 47 | ||
48 | # GET /projects/:id/repository/branches | 48 | # GET /projects/:id/repository/branches |
49 | get ":id/repository/branches" do | 49 | get ":id/repository/branches" do |
50 | - @project = Project.find_by_code(params[:id]) | 50 | + @project = current_user.projects.find_by_code(params[:id]) |
51 | present @project.repo.heads.sort_by(&:name), :with => Entities::ProjectRepositoryBranches | 51 | present @project.repo.heads.sort_by(&:name), :with => Entities::ProjectRepositoryBranches |
52 | end | 52 | end |
53 | 53 | ||
54 | # GET /projects/:id/repository/tags | 54 | # GET /projects/:id/repository/tags |
55 | get ":id/repository/tags" do | 55 | get ":id/repository/tags" do |
56 | - @project = Project.find_by_code(params[:id]) | 56 | + @project = current_user.projects.find_by_code(params[:id]) |
57 | present @project.repo.tags.sort_by(&:name).reverse, :with => Entities::ProjectRepositoryTags | 57 | present @project.repo.tags.sort_by(&:name).reverse, :with => Entities::ProjectRepositoryTags |
58 | end | 58 | end |
59 | end | 59 | end |
spec/api/projects_spec.rb
@@ -3,10 +3,9 @@ require 'spec_helper' | @@ -3,10 +3,9 @@ require 'spec_helper' | ||
3 | describe Gitlab::API do | 3 | describe Gitlab::API do |
4 | let(:user) { Factory :user } | 4 | let(:user) { Factory :user } |
5 | let!(:project) { Factory :project, :owner => user } | 5 | let!(:project) { Factory :project, :owner => user } |
6 | + before { project.add_access(user, :read) } | ||
6 | 7 | ||
7 | describe "GET /projects" do | 8 | describe "GET /projects" do |
8 | - before { project.add_access(user, :read) } | ||
9 | - | ||
10 | it "should return authentication error" do | 9 | it "should return authentication error" do |
11 | get "/api/projects" | 10 | get "/api/projects" |
12 | response.status.should == 401 | 11 | response.status.should == 401 |