From 786060d85cc4739792e759449475e27c502a52ff Mon Sep 17 00:00:00 2001
From: actlikewill <wilsongaturu@gmail.com>
Date: Fri, 30 Aug 2024 07:03:57 +0300
Subject: [PATCH] updates saved document template

---
 peachjam/forms.py                             | 11 ++++--
 .../peachjam/saved_document_create.html       | 29 ++++++++++++++
 ...cument.html => saved_document_update.html} | 39 ++++++++-----------
 peachjam/views/save_document.py               |  3 +-
 4 files changed, 54 insertions(+), 28 deletions(-)
 create mode 100644 peachjam/templates/peachjam/saved_document_create.html
 rename peachjam/templates/peachjam/{saved_document.html => saved_document_update.html} (71%)

diff --git a/peachjam/forms.py b/peachjam/forms.py
index a58f06a5f..d6318f400 100644
--- a/peachjam/forms.py
+++ b/peachjam/forms.py
@@ -327,15 +327,18 @@ class Meta:
     def __init__(self, *args, **kwargs):
         super().__init__(*args, **kwargs)
         self.fields["folder"].queryset = self.instance.user.folders.all()
+        self.fields["document"].required = False
 
     def clean(self):
-        if self.cleaned_data.get("new_folder"):
+        cleaned_data = super().clean()
+        cleaned_data["document"] = self.instance.document
+        if cleaned_data.get("new_folder"):
             folder, _ = Folder.objects.get_or_create(
-                name=self.cleaned_data["new_folder"],
+                name=cleaned_data["new_folder"],
                 user=self.instance.user,
             )
-            self.cleaned_data["folder"] = folder
-        return self.cleaned_data
+            cleaned_data["folder"] = folder
+        return cleaned_data
 
 
 class PeachjamSignupForm(SignupForm):
diff --git a/peachjam/templates/peachjam/saved_document_create.html b/peachjam/templates/peachjam/saved_document_create.html
new file mode 100644
index 000000000..d177cf04f
--- /dev/null
+++ b/peachjam/templates/peachjam/saved_document_create.html
@@ -0,0 +1,29 @@
+{% load i18n %}
+<div id="saveDocumentWrapper">
+  <button id="saveDocumentButton"
+          class="btn btn-outline-primary btn-shrink-sm ms-2"
+          type="button"
+          data-bs-toggle="modal"
+          data-bs-target="#saveDocumentModal"
+          aria-expanded="false"
+          hx-post="{% url 'saved_document_create' %}?doc_id={{ form.document.value }}"
+          hx-target="#saveDocumentWrapper">
+    {% trans 'Save document' %}
+  </button>
+  <div hx-swap-oob="true" id="saveDocumentModalDialog" class="modal-dialog">
+    <div class="modal-content">
+      <div class="modal-header">
+        <p class="h5 modal-title">{% trans 'Save document' %}</p>
+        <button type="button"
+                class="btn-close"
+                data-bs-dismiss="modal"
+                aria-label="Close"></button>
+      </div>
+      <div class="modal-body">
+        Saving...
+        {{ form.errors }}
+      </div>
+      <div class="modal-footer d-flex"></div>
+    </div>
+  </div>
+</div>
diff --git a/peachjam/templates/peachjam/saved_document.html b/peachjam/templates/peachjam/saved_document_update.html
similarity index 71%
rename from peachjam/templates/peachjam/saved_document.html
rename to peachjam/templates/peachjam/saved_document_update.html
index a80206211..e2b04ba13 100644
--- a/peachjam/templates/peachjam/saved_document.html
+++ b/peachjam/templates/peachjam/saved_document_update.html
@@ -5,17 +5,12 @@
           type="button"
           data-bs-toggle="modal"
           data-bs-target="#saveDocumentModal"
-          aria-expanded="false"
-          {% if not saved_document %} hx-post="{% url 'saved_document_create' %}" hx-target="#saveDocumentWrapper" hx-include="#save-document-form" {% endif %}>
-    {% if saved_document %}
-      <i class="bi bi-star-fill"></i>
-      {% if saved_document.folder %}
-        {% trans 'Saved to' %} {{ saved_document.folder }}
-      {% else %}
-        {% trans 'Saved' %}
-      {% endif %}
+          aria-expanded="false">
+    <i class="bi bi-star-fill"></i>
+    {% if saved_document.folder %}
+      {% trans 'Saved to' %} {{ saved_document.folder }}
     {% else %}
-      {% trans 'Save document' %}
+      {% trans 'Saved' %}
     {% endif %}
   </button>
   <div hx-swap-oob="true" id="saveDocumentModalDialog" class="modal-dialog">
@@ -29,7 +24,7 @@
       </div>
       <div class="modal-body">
         <form id="save-document-form"
-              {% if saved_document %} hx-post="{% url 'saved_document_update' saved_document.id %}" {% else %} hx-post="{% url 'saved_document_create' %}" {% endif %}
+              hx-post="{% url 'saved_document_update' saved_document.id %}"
               hx-target="#saveDocumentWrapper">
           {{ form.document }}
           <p class="h5">{{ saved_document.document }}</p>
@@ -66,18 +61,16 @@
         </form>
       </div>
       <div class="modal-footer d-flex justify-content-between">
-        {% if saved_document %}
-          <button class="btn btn-danger"
-                  type="button"
-                  data-bs-dismiss="modal"
-                  hx-target="#saveDocumentWrapper"
-                  hx-include="#save-document-form"
-                  hx-headers='{"X-CSRFToken": "{{ csrf_token }}"}'
-                  hx-post="{% url 'saved_document_delete' saved_document.id %}"
-                  hx-confirm="Are you sure you want to unsave this document?">
-            {% trans 'Unsave' %}
-          </button>
-        {% endif %}
+        <button class="btn btn-danger"
+                type="button"
+                data-bs-dismiss="modal"
+                hx-target="#saveDocumentWrapper"
+                hx-include="#save-document-form"
+                hx-headers='{"X-CSRFToken": "{{ csrf_token }}"}'
+                hx-post="{% url 'saved_document_delete' saved_document.id %}"
+                hx-confirm="Are you sure you want to unsave this document?">
+          {% trans 'Unsave' %}
+        </button>
         <div>
           <button type="button"
                   class="btn btn-outline-secondary btn-shrink-sm ms-2"
diff --git a/peachjam/views/save_document.py b/peachjam/views/save_document.py
index 2a33e231f..424a5962e 100644
--- a/peachjam/views/save_document.py
+++ b/peachjam/views/save_document.py
@@ -106,7 +106,7 @@ def get(self, *args, **kwargs):
 
 
 class BaseSavedDocumentFormView(AllowSavedDocumentMixin, LoginRequiredMixin):
-    template_name = "peachjam/saved_document.html"
+    template_name = "peachjam/saved_document_update.html"
     form_class = SaveDocumentForm
     context_object_name = "saved_document"
     model = SavedDocument
@@ -124,6 +124,7 @@ def get_success_url(self):
 
 
 class SavedDocumentCreateView(BaseSavedDocumentFormView, CreateView):
+    template_name = "peachjam/saved_document_create.html"
     permission_required = "peachjam.add_saveddocument"
 
     def get_form_kwargs(self):