feat: add import taxonomy endpoint to content_tagging (#33663)

openedx/core/djangoapps/content_tagging/rest_api/v1/views.py

@@ -1,14 +1,17 @@
 """
 Tagging Org API Views
 """
-
+from openedx_tagging.core.tagging import rules as oel_tagging_rules
 from openedx_tagging.core.tagging.rest_api.v1.views import ObjectTagView, TaxonomyView
+from rest_framework import status
 from rest_framework.decorators import action
 from rest_framework.exceptions import PermissionDenied
+from rest_framework.request import Request
 from rest_framework.response import Response
 
 from ...api import (
     create_taxonomy,
+    get_taxonomy,
     get_taxonomies,
     get_taxonomies_for_org,
     set_taxonomy_orgs,
@@ -20,7 +23,7 @@
 
 class TaxonomyOrgView(TaxonomyView):
     """
-    View to list, create, retrieve, update, or delete Taxonomies.
+    View to list, create, retrieve, update, delete, export or import Taxonomies.
     This view extends the TaxonomyView to add Organization filters.
 
     Refer to TaxonomyView docstring for usage details.
@@ -69,6 +72,32 @@ def perform_create(self, serializer):
         user_admin_orgs = get_admin_orgs(self.request.user)
         serializer.instance = create_taxonomy(**serializer.validated_data, orgs=user_admin_orgs)
 
+    @action(detail=False, url_path="import", methods=["post"])
+    def create_import(self, request: Request, **kwargs) -> Response:
+        """
+        Creates a new taxonomy with the given orgs and imports the tags from the uploaded file.
+        """
+        response = super().create_import(request, **kwargs)
+
+        # If creation was successful, set the orgs for the new taxonomy
+        if status.is_success(response.status_code):
+            # ToDo: This code is temporary
+            # In the future, the orgs parameter will be defined in the request body from the frontend
+            # See: https://github.com/openedx/modular-learning/issues/116
+            if oel_tagging_rules.is_taxonomy_admin(request.user):
+                orgs = None
+            else:
+                orgs = get_admin_orgs(request.user)
+
+            taxonomy = get_taxonomy(response.data["id"])
+            assert taxonomy
+            set_taxonomy_orgs(taxonomy, all_orgs=False, orgs=orgs)
+
+            serializer = self.get_serializer(taxonomy)
+            return Response(serializer.data, status=status.HTTP_201_CREATED)
+
+        return response
+
     @action(detail=True, methods=["put"])
     def orgs(self, request, **_kwargs) -> Response:
         """

openedx/core/djangoapps/content_tagging/rules.py

@@ -281,7 +281,6 @@ def can_change_taxonomy_tag(user: UserType, tag: oel_tagging.Tag | None = None)
 rules.set_perm("oel_tagging.change_taxonomy", can_change_taxonomy)
 rules.set_perm("oel_tagging.delete_taxonomy", can_change_taxonomy)
 rules.set_perm("oel_tagging.view_taxonomy", can_view_taxonomy)
-rules.set_perm("oel_tagging.export_taxonomy", can_view_taxonomy)
 rules.add_perm("oel_tagging.update_orgs", oel_tagging.is_taxonomy_admin)
 
 # Tag

requirements/constraints.txt

@@ -125,7 +125,7 @@ libsass==0.10.0
 click==8.1.6
 
 # pinning this version to avoid updates while the library is being developed
-openedx-learning==0.3.2
+openedx-learning==0.3.3
 
 # lti-consumer-xblock 9.6.2 contains a breaking change that makes
 # existing custom parameter configurations unusable.

requirements/edx/base.txt

@@ -803,7 +803,7 @@ openedx-filters==1.6.0
     # via
     #   -r requirements/edx/kernel.in
     #   lti-consumer-xblock
-openedx-learning==0.3.2
+openedx-learning==0.3.3
     # via
     #   -c requirements/edx/../constraints.txt
     #   -r requirements/edx/kernel.in

requirements/edx/development.txt

@@ -1331,7 +1331,7 @@ openedx-filters==1.6.0
     #   -r requirements/edx/doc.txt
     #   -r requirements/edx/testing.txt
     #   lti-consumer-xblock
-openedx-learning==0.3.2
+openedx-learning==0.3.3
     # via
     #   -c requirements/edx/../constraints.txt
     #   -r requirements/edx/doc.txt

requirements/edx/doc.txt

@@ -961,7 +961,7 @@ openedx-filters==1.6.0
     # via
     #   -r requirements/edx/base.txt
     #   lti-consumer-xblock
-openedx-learning==0.3.2
+openedx-learning==0.3.3
     # via
     #   -c requirements/edx/../constraints.txt
     #   -r requirements/edx/base.txt

requirements/edx/testing.txt

@@ -1006,7 +1006,7 @@ openedx-filters==1.6.0
     # via
     #   -r requirements/edx/base.txt
     #   lti-consumer-xblock
-openedx-learning==0.3.2
+openedx-learning==0.3.3
     # via
     #   -c requirements/edx/../constraints.txt
     #   -r requirements/edx/base.txt