diff --git a/taiga/projects/api.py b/taiga/projects/api.py index 114b78a0..c51673c0 100644 --- a/taiga/projects/api.py +++ b/taiga/projects/api.py @@ -66,20 +66,6 @@ class ProjectViewSet(ModelCrudViewSet): self.check_permissions(request, 'stats', project) return Response(services.get_stats_for_project(project)) - @detail_route(methods=['post']) - def star(self, request, pk=None): - project = self.get_object() - self.check_permissions(request, 'star', project) - votes_service.add_vote(project, user=request.user) - return Response(status=status.HTTP_200_OK) - - @detail_route(methods=['post']) - def unstar(self, request, pk=None): - project = self.get_object() - self.check_permissions(request, 'unstar', project) - votes_service.remove_vote(project, user=request.user) - return Response(status=status.HTTP_200_OK) - @detail_route(methods=['get']) def issues_stats(self, request, pk=None): project = self.get_object() @@ -92,18 +78,26 @@ class ProjectViewSet(ModelCrudViewSet): self.check_permissions(request, 'issues_filters_data', project) return Response(services.get_issues_filters_data(project)) - @detail_route(methods=['get']) - def tags(self, request, pk=None): - project = self.get_object() - self.check_permissions(request, 'tags', project) - return Response(services.get_all_tags(project)) - @detail_route(methods=['get']) def tags_colors(self, request, pk=None): project = self.get_object() self.check_permissions(request, 'tags_colors', project) return Response(dict(project.tags_colors)) + @detail_route(methods=['post']) + def star(self, request, pk=None): + project = self.get_object() + self.check_permissions(request, 'star', project) + votes_service.add_vote(project, user=request.user) + return Response(status=status.HTTP_200_OK) + + @detail_route(methods=['post']) + def unstar(self, request, pk=None): + project = self.get_object() + self.check_permissions(request, 'unstar', project) + votes_service.remove_vote(project, user=request.user) + return Response(status=status.HTTP_200_OK) + @detail_route(methods=['get']) def fans(self, request, pk=None): project = self.get_object() diff --git a/tests/integration/resources_permissions/test_projects_resource.py b/tests/integration/resources_permissions/test_projects_resource.py index 18517a57..58ef3eae 100644 --- a/tests/integration/resources_permissions/test_projects_resource.py +++ b/tests/integration/resources_permissions/test_projects_resource.py @@ -262,25 +262,6 @@ def test_project_action_issues_filters_data(client, data): assert results == [404, 404, 200, 200] -def test_project_action_tags(client, data): - public_url = reverse('projects-tags', kwargs={"pk": data.public_project.pk}) - private1_url = reverse('projects-tags', kwargs={"pk": data.private_project1.pk}) - private2_url = reverse('projects-tags', kwargs={"pk": data.private_project2.pk}) - - users = [ - None, - data.registered_user, - data.project_member_with_perms, - data.project_owner - ] - results = helper_test_http_method(client, 'get', public_url, None, users) - assert results == [200, 200, 200, 200] - results = helper_test_http_method(client, 'get', private1_url, None, users) - assert results == [200, 200, 200, 200] - results = helper_test_http_method(client, 'get', private2_url, None, users) - assert results == [404, 404, 200, 200] - - def test_project_action_fans(client, data): public_url = reverse('projects-fans', kwargs={"pk": data.public_project.pk}) private1_url = reverse('projects-fans', kwargs={"pk": data.private_project1.pk})