From 5838e1bc07b68c72c7d7f5e8a4eeda636c1e8c38 Mon Sep 17 00:00:00 2001 From: yackob03 Date: Mon, 4 Nov 2013 19:01:58 -0500 Subject: [PATCH 1/3] Fix a merge error in create_repo_api. --- endpoints/api.py | 5 ----- 1 file changed, 5 deletions(-) diff --git a/endpoints/api.py b/endpoints/api.py index f2bf7e769..c4c14fa87 100644 --- a/endpoints/api.py +++ b/endpoints/api.py @@ -407,11 +407,6 @@ def create_repo_api(): repo.description = json['description'] repo.save() - repo = model.create_repository(namespace_name, repository_name, owner, - visibility) - repo.description = json['description'] - repo.save() - return jsonify({ 'namespace': namespace_name, 'name': repository_name From 6aeac247fa85e1b3c756b343375ab8e16486883d Mon Sep 17 00:00:00 2001 From: yackob03 Date: Mon, 4 Nov 2013 19:06:56 -0500 Subject: [PATCH 2/3] Teams are part of organizations. --- endpoints/api.py | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/endpoints/api.py b/endpoints/api.py index c4c14fa87..3e188f765 100644 --- a/endpoints/api.py +++ b/endpoints/api.py @@ -216,7 +216,8 @@ def get_matching_entities(prefix): def team_view(team): return { 'name': team.name, - 'kind': 'team' + 'kind': 'team', + 'is_org_member': True, } def user_view(user): From c1b246c3b59945ea92c4e321f07f9f3532b868f8 Mon Sep 17 00:00:00 2001 From: yackob03 Date: Mon, 4 Nov 2013 19:11:01 -0500 Subject: [PATCH 3/3] Remove a TODO and make sure a user is part of an org to get access to typeahead information for an org. --- endpoints/api.py | 17 ++++++++--------- 1 file changed, 8 insertions(+), 9 deletions(-) diff --git a/endpoints/api.py b/endpoints/api.py index 3e188f765..7f901262c 100644 --- a/endpoints/api.py +++ b/endpoints/api.py @@ -202,14 +202,15 @@ def get_matching_entities(prefix): organization_name = request.args.get('organization', None) organization = None if organization_name: - try: - organization = model.get_organization(organization_name) - except: - pass + permission = OrganizationMemberPermission(organization_name) + if permission.can(): + try: + organization = model.get_organization(organization_name) + except: + pass - if organization: - # TODO: ensure that the user has access to the organization - teams = model.get_matching_teams(prefix, organization) + if organization: + teams = model.get_matching_teams(prefix, organization) users = model.get_matching_users(prefix, organization) @@ -221,8 +222,6 @@ def get_matching_entities(prefix): } def user_view(user): - # TODO: Return whether the user is outside the organization (if one is - # specified) return { 'name': user.username, 'kind': 'user',