forked from github/plane
chore: state delete validations endpoint (#880)
This commit is contained in:
parent
73a8bbb31f
commit
5b6caadd6f
@ -79,6 +79,7 @@ from plane.api.views import (
|
|||||||
## End Issues
|
## End Issues
|
||||||
# States
|
# States
|
||||||
StateViewSet,
|
StateViewSet,
|
||||||
|
StateDeleteIssueCheckEndpoint,
|
||||||
## End States
|
## End States
|
||||||
# Estimates
|
# Estimates
|
||||||
EstimateViewSet,
|
EstimateViewSet,
|
||||||
@ -509,6 +510,11 @@ urlpatterns = [
|
|||||||
),
|
),
|
||||||
name="project-state",
|
name="project-state",
|
||||||
),
|
),
|
||||||
|
path(
|
||||||
|
"workspaces/<str:slug>/projects/<uuid:project_id>/states/<uuid:pk>/",
|
||||||
|
StateDeleteIssueCheckEndpoint.as_view(),
|
||||||
|
name="state-delete-check",
|
||||||
|
),
|
||||||
# End States ##
|
# End States ##
|
||||||
# States
|
# States
|
||||||
path(
|
path(
|
||||||
|
@ -42,7 +42,7 @@ from .workspace import (
|
|||||||
UserWorkspaceDashboardEndpoint,
|
UserWorkspaceDashboardEndpoint,
|
||||||
WorkspaceThemeViewSet,
|
WorkspaceThemeViewSet,
|
||||||
)
|
)
|
||||||
from .state import StateViewSet
|
from .state import StateViewSet, StateDeleteIssueCheckEndpoint
|
||||||
from .shortcut import ShortCutViewSet
|
from .shortcut import ShortCutViewSet
|
||||||
from .view import IssueViewViewSet, ViewIssuesEndpoint, IssueViewFavoriteViewSet
|
from .view import IssueViewViewSet, ViewIssuesEndpoint, IssueViewFavoriteViewSet
|
||||||
from .cycle import (
|
from .cycle import (
|
||||||
|
@ -11,10 +11,10 @@ from sentry_sdk import capture_exception
|
|||||||
|
|
||||||
|
|
||||||
# Module imports
|
# Module imports
|
||||||
from . import BaseViewSet
|
from . import BaseViewSet, BaseAPIView
|
||||||
from plane.api.serializers import StateSerializer
|
from plane.api.serializers import StateSerializer
|
||||||
from plane.api.permissions import ProjectEntityPermission
|
from plane.api.permissions import ProjectEntityPermission
|
||||||
from plane.db.models import State
|
from plane.db.models import State, Issue
|
||||||
|
|
||||||
|
|
||||||
class StateViewSet(BaseViewSet):
|
class StateViewSet(BaseViewSet):
|
||||||
@ -53,7 +53,10 @@ class StateViewSet(BaseViewSet):
|
|||||||
)
|
)
|
||||||
except Exception as e:
|
except Exception as e:
|
||||||
capture_exception(e)
|
capture_exception(e)
|
||||||
return Response({"error": "Something went wrong please try again later"}, status=status.HTTP_400_BAD_REQUEST)
|
return Response(
|
||||||
|
{"error": "Something went wrong please try again later"},
|
||||||
|
status=status.HTTP_400_BAD_REQUEST,
|
||||||
|
)
|
||||||
|
|
||||||
def list(self, request, slug, project_id):
|
def list(self, request, slug, project_id):
|
||||||
try:
|
try:
|
||||||
@ -85,7 +88,37 @@ class StateViewSet(BaseViewSet):
|
|||||||
{"error": "Default state cannot be deleted"}, status=False
|
{"error": "Default state cannot be deleted"}, status=False
|
||||||
)
|
)
|
||||||
|
|
||||||
|
# Check for any issues in the state
|
||||||
|
issue_exist = Issue.objects.filter(state=pk).exists()
|
||||||
|
|
||||||
|
if issue_exist:
|
||||||
|
return Response(
|
||||||
|
{
|
||||||
|
"error": "The state is not empty, only empty states can be deleted"
|
||||||
|
},
|
||||||
|
status=status.HTTP_400_BAD_REQUEST,
|
||||||
|
)
|
||||||
|
|
||||||
state.delete()
|
state.delete()
|
||||||
return Response(status=status.HTTP_204_NO_CONTENT)
|
return Response(status=status.HTTP_204_NO_CONTENT)
|
||||||
except State.DoesNotExist:
|
except State.DoesNotExist:
|
||||||
return Response({"error": "State does not exists"}, status=status.HTTP_404)
|
return Response({"error": "State does not exists"}, status=status.HTTP_404)
|
||||||
|
|
||||||
|
|
||||||
|
class StateDeleteIssueCheckEndpoint(BaseAPIView):
|
||||||
|
permission_classes = [
|
||||||
|
ProjectEntityPermission,
|
||||||
|
]
|
||||||
|
|
||||||
|
def get(self, request, slug, project_id, pk):
|
||||||
|
try:
|
||||||
|
issue_count = Issue.objects.filter(
|
||||||
|
state=pk, workspace__slug=slug, project_id=project_id
|
||||||
|
).count()
|
||||||
|
return Response({"issue_count": issue_count}, status=status.HTTP_200_OK)
|
||||||
|
except Exception as e:
|
||||||
|
capture_exception(e)
|
||||||
|
return Response(
|
||||||
|
{"error": "Something went wrong please try again later"},
|
||||||
|
status=status.HTTP_400_BAD_REQUEST,
|
||||||
|
)
|
||||||
|
Loading…
Reference in New Issue
Block a user