🐛 archaeological sites: remove "add operation" shortcuts (refs #6329)

3 files changed, 0 insertions, 94 deletions

diff --git a/archaeological_operations/models.py b/archaeological_operations/models.py
index d6a0d45e3..72838e815 100644
--- a/archaeological_operations/models.py
+++ b/archaeological_operations/models.py
@@ -612,14 +612,6 @@ class ArchaeologicalSite(
     QUICK_ACTIONS = [
         QA_EDIT,
         QuickAction(
-            url="site-add-operation",
-            icon_class="fa fa-plus",
-            text=_("Create associated operation"),
-            target="many",
-            rights=["archaeological_operations.add_operation"],
-            is_popup=False
-        ),
-        QuickAction(
             url="site-qa-duplicate",
             icon_class="fa fa-clone",
             text=_("Duplicate"),
@@ -826,33 +818,6 @@ class ArchaeologicalSite(
                 ),
             ]
 
-        can_create_operation = self.can_do(
-            request, "archaeological_operations.change_operation"
-        )
-        if can_create_operation and not self.operations.count():
-            actions.append(
-                (
-                    reverse("site-add-operation", args=[self.pk]),
-                    _("Create an operation associated to this site"),
-                    "fa fa-plus",
-                    _("ope."),
-                    "",
-                    False
-                )
-            )
-
-        if can_create_operation and self.operations.count():
-            actions.append(
-                (
-                    reverse("site-add-top-operation", args=[self.pk]),
-                    _("Create a cluster operation for site associated to many operations"),
-                    "fa fa-plus",
-                    _("top ope."),
-                    "",
-                    False
-                )
-            )
-
         if can_edit_site:
             actions += [
                 (
diff --git a/archaeological_operations/urls.py b/archaeological_operations/urls.py
index 675d6d13c..46461a2bd 100644
--- a/archaeological_operations/urls.py
+++ b/archaeological_operations/urls.py
@@ -443,20 +443,6 @@ urlpatterns = [
         kwargs={"confirm": True},
     ),
     url(
-        r"^site-add-operation/(?P<pks>[0-9-]+)?/$",
-        check_permissions(["archaeological_operations.change_operation"])(
-            views.site_add_operation
-        ),
-        name="site-add-operation",
-    ),
-    url(
-        r"^site-add-top-operation/(?P<pks>\d+)?/$",
-        check_permissions(["archaeological_operations.change_operation"])(
-            views.site_add_top_operation
-        ),
-        name="site-add-top-operation",
-    ),
-    url(
         r"generate-stats-operation/(?P<pk>.+)/",
         views.GenerateStatsOperation.as_view(),
         name="generate-stats-operation",
diff --git a/archaeological_operations/views.py b/archaeological_operations/views.py
index bca85ef44..191e5c0f4 100644
--- a/archaeological_operations/views.py
+++ b/archaeological_operations/views.py
@@ -749,51 +749,6 @@ def site_delete(request, pk):
     return redirect(reverse(wizard_url, kwargs={"step": "final-" + wizard_url}))
 
 
-def site_add_operation(request, pks, current_right=None):
-    pks = [int(p) for p in pks.split("-")]
-    for pk in pks:
-        q = models.ArchaeologicalSite.objects.filter(pk=pk)
-        if not q.count():
-            raise Http404()
-        site = q.all()[0]
-        if not site.can_do(request, "view", app="archaeological_operations",
-                           model_name="archaeologicalsite") \
-                or site.operations.count():
-            raise Http404()
-    # operation add permission checked on view call
-    operation_creation_wizard(request)
-    wizards.OperationWizard.session_set_value(
-            request, "general-operation_creation", "archaeological_site", pks,
-            reset=True
-    )
-    return redirect(
-        reverse("operation_creation", kwargs={"step": "general-operation_creation"})
-    )
-
-
-def site_add_top_operation(request, pks, current_right=None):
-    q = models.ArchaeologicalSite.objects.filter(pk=pks)
-    if not q.count():
-        raise Http404()
-    site = q.all()[0]
-    if not site.can_do(request, "view", app="archaeological_operations",
-                       model_name="archaeologicalsite") \
-            or not site.operations.count():
-        raise Http404()
-    # operation add permission checked on view call
-    top_operation = site.create_or_update_top_operation(create=True)
-    if not wizard_is_available(
-            operation_modification_wizard, request, models.Operation, top_operation.pk
-    ):
-        return HttpResponseRedirect("/")
-
-    wizard_url = "operation_modification"
-    wizards.OperationModificationWizard.session_set_value(
-        request, "selec-" + wizard_url, "pk", top_operation.pk, reset=True
-    )
-    return redirect(reverse(wizard_url, kwargs={"step": "general-" + wizard_url}))
-
-
 operation_administrativeactop_search_wizard = wizards.SearchWizard.as_view(
     [
         (