Merge pull request #1103 from nationalarchives/chore/remove-delete

Remove unused deletion code
nationalarchives · Aug 7, 2023 · 3fb7dee · 3fb7dee
2 parents 7501e06 + fdecea2
commit 3fb7dee
Show file tree

Hide file tree

Showing 6 changed files with 1 addition and 62 deletions.
diff --git a/ds_caselaw_editor_ui/sass/includes/_judgment_text_toolbar.scss b/ds_caselaw_editor_ui/sass/includes/_judgment_text_toolbar.scss
@@ -10,17 +10,6 @@
     padding: calc($spacer__unit / 2) calc($spacer__unit * 1.5) !important;
   }
 
-  &__delete {
-    border-color: $color__red !important;
-    color: $color__red !important;
-    &:focus,
-    &:hover {
-      background-color: $color__red !important;
-      color: $color__white !important;
-      outline-color: $color__red !important;
-    }
-  }
-
   &__return-link {
     display: inline-block;
 

diff --git a/ds_caselaw_editor_ui/templates/includes/judgment/toolbar.html b/ds_caselaw_editor_ui/templates/includes/judgment/toolbar.html
@@ -40,16 +40,6 @@
         <span class="judgment-toolbar__button button-secondary"
               aria-disabled="true">{% translate "judgment.toolbar.download_docx" %}</span>
       {% endif %}
-      {% if judgment.is_failure %}
-        <form action="{% url "delete" %}" method="post">
-          {% csrf_token %}
-          <input type="hidden" name="judgment_uri" value="{{ judgment.uri }}" />
-          <input type="submit"
-                 name="assign"
-                 class="button-secondary judgment-toolbar__delete"
-                 value="{% translate "judgment.toolbar.delete" %}" />
-        </form>
-      {% endif %}
     </div>
     {% if version %}
       <div class="judgment-toolbar__version">

diff --git a/judgments/urls.py b/judgments/urls.py
@@ -5,7 +5,6 @@
     hold_judgment_button,
     prioritise_judgment_button,
 )
-from .views.delete import delete
 from .views.full_text import (
     html_view,
     html_view_redirect,
@@ -53,7 +52,6 @@
     path("unpublish", unpublish, name="unpublish"),
     path("hold", hold, name="hold"),
     path("unhold", unhold, name="unhold"),
-    path("delete", delete, name="delete"),
     path("unlock", unlock, name="unlock"),
     path("assign", assign_judgment_button, name="assign"),
     path("prioritise", prioritise_judgment_button, name="prioritise"),

diff --git a/judgments/utils/aws.py b/judgments/utils/aws.py
@@ -104,11 +104,6 @@ def delete_from_bucket(uri: str, bucket: str) -> None:
         )
 
 
-def delete_documents(uri: str) -> None:
-    unpublish_documents(uri)
-    delete_from_bucket(uri, env("PRIVATE_ASSET_BUCKET"))
-
-
 def publish_documents(uri: str) -> None:
     client = create_s3_client()
 

diff --git a/judgments/views/delete.py b/judgments/views/delete.py
diff --git a/locale/en_GB/LC_MESSAGES/django.po b/locale/en_GB/LC_MESSAGES/django.po
@@ -7,7 +7,7 @@ msgid ""
 msgstr ""
 "Project-Id-Version: PACKAGE VERSION\n"
 "Report-Msgid-Bugs-To: \n"
-"POT-Creation-Date: 2023-08-03 15:39+0000\n"
+"POT-Creation-Date: 2023-08-07 16:14+0000\n"
 "PO-Revision-Date: YEAR-MO-DA HO:MI+ZONE\n"
 "Last-Translator: FULL NAME <EMAIL@ADDRESS>\n"
 "Language-Team: LANGUAGE <[email protected]>\n"
@@ -175,10 +175,6 @@ msgstr "Publish"
 msgid "judgment.toolbar.download_docx"
 msgstr "Download .docx"
 
-#: ds_caselaw_editor_ui/templates/includes/judgment/toolbar.html
-msgid "judgment.toolbar.delete"
-msgstr "Delete"
-
 #: ds_caselaw_editor_ui/templates/includes/judgments_list_header.html
 msgid "judgments.details"
 msgstr "Document details:"
@@ -334,10 +330,6 @@ msgstr ""
 msgid "judgments.submitteremail"
 msgstr "Contact email"
 
-#: judgments/views/delete.py
-msgid "judgment.delete_a_judgment"
-msgstr "Delete a document"
-
 #: judgments/views/judgment_hold.py
 msgid "judgment.hold.hold_success_flash_message"
 msgstr "Document on hold"