From 04e428fcf6cd5b16a4388612d349bd934e124b18 Mon Sep 17 00:00:00 2001
From: bobbyvish <bobbyvishwakarma114@gmail.com>
Date: Tue, 2 Jul 2024 18:54:18 +0530
Subject: [PATCH] fix(module_1):event form validation, venue validation

---
 manage_events/forms.py                        |  9 ++++++--
 manage_subscriptions/views.py                 |  2 +-
 .../accounts/customer/customer_detail.html    | 14 ++++++-------
 .../accounts/customer/customer_edit.html      |  2 +-
 .../accounts/customer/customer_list.html      |  2 +-
 templates/manage_events/event_add.html        | 21 ++-----------------
 .../subscription_list.html                    | 10 +++++++--
 templates/manage_venues/venue_add.html        | 19 +++++++++++------
 8 files changed, 40 insertions(+), 39 deletions(-)

diff --git a/manage_events/forms.py b/manage_events/forms.py
index 9b68fb5..dc5777d 100644
--- a/manage_events/forms.py
+++ b/manage_events/forms.py
@@ -141,16 +141,21 @@ class VenueForm(forms.ModelForm):
         label="Non-transfer user list",
         required=True
     )
+    image = forms.ImageField(required=True)
+    latitude = forms.DecimalField(
+        widget=forms.NumberInput()
+    )
+    longitude = forms.DecimalField(
+        widget=forms.NumberInput()
+    )
 
     class Meta:
         model = Venue
         fields = [
             "principal",
             "title",
-            # "description",
             "address",
             "image",
-            # "url",
             "latitude",
             "longitude",
         ]
diff --git a/manage_subscriptions/views.py b/manage_subscriptions/views.py
index 0050df9..bf7dda6 100644
--- a/manage_subscriptions/views.py
+++ b/manage_subscriptions/views.py
@@ -100,7 +100,7 @@ class SubscriptionCreateOrUpdateView(LoginRequiredMixin, generic.View):
 
         # This code ensures that only one free plan can be created by checking for existing free plans before saving a new one.
         if form.cleaned_data.get("is_free"):
-            if self.model.objects.filter(Q(is_free=True) & Q(active=True)).exists:
+            if self.model.objects.filter(Q(is_free=True) & Q(active=True)).exists():
                 messages.error(self.request, "A free plan is already available. Please deactivate the existing one before creating a new one.")
                 context = self.get_context_data(form=form)
                 return render(request, self.template_name, context=context)
diff --git a/templates/accounts/customer/customer_detail.html b/templates/accounts/customer/customer_detail.html
index 438b117..1cec652 100644
--- a/templates/accounts/customer/customer_detail.html
+++ b/templates/accounts/customer/customer_detail.html
@@ -61,13 +61,13 @@
                             <div class="col-md-3">End Date</div>
                             <div class="col-md-9">{% if principal_subscription %}{{ principal_subscription.end_date }}{% else %}No subscription found{% endif %}</div>
                         </div>
-                            {% if principal_obj.extended_data and not principal_obj.extended_data.transferred and principal_obj.extended_data.onboarded and principal_obj.principal_type.name == 'event_manager' %}
-                                <div class="col text-end">
-                                    <a class="btn btn-dark mb-2 me-4" href="{% url 'accounts:customer_transfer' principal_obj.id %}">
-                                        Transfer Account
-                                    </a>
-                                </div>
-                            {% endif %}
+                        {% if principal_obj.extended_data and not principal_obj.extended_data.is_transferred and principal_obj.extended_data.is_onboarded and principal_obj.principal_type.name == 'event_manager' %}
+                            <div class="col text-end">
+                                <a class="btn btn-dark mb-2 me-4" href="{% url 'accounts:customer_transfer' principal_obj.id %}">
+                                    Transfer Account
+                                </a>
+                            </div>
+                        {% endif %}
                         </div>
                     </div>
                 </div>
diff --git a/templates/accounts/customer/customer_edit.html b/templates/accounts/customer/customer_edit.html
index d5b3009..57c6774 100644
--- a/templates/accounts/customer/customer_edit.html
+++ b/templates/accounts/customer/customer_edit.html
@@ -19,7 +19,7 @@
                 </a>
             </div>
             
-            {% if principal_obj.extended_data and not principal_obj.extended_data.transferred and principal_obj.extended_data.onboarded and principal_obj.principal_type.name == 'event_manager' %}
+            {% if principal_obj.extended_data and not principal_obj.extended_data.is_transferred and principal_obj.extended_data.is_onboarded and principal_obj.principal_type.name == 'event_manager' %}
                 <div class="col text-end">
                     <a class="btn btn-dark mb-2 me-4" href="{% url 'accounts:customer_transfer' principal_obj.id %}">
                         Transfer Account
diff --git a/templates/accounts/customer/customer_list.html b/templates/accounts/customer/customer_list.html
index ad1825f..caf4463 100644
--- a/templates/accounts/customer/customer_list.html
+++ b/templates/accounts/customer/customer_list.html
@@ -21,7 +21,7 @@
                 </a>
                 
                 <a class="btn btn-dark mb-2 me-md-4" href="{% url 'accounts:import_customer_data' %}">
-                    import
+                    Import
                 </a>
 
                 <a class="btn btn-dark mb-2 me-md-4" href="{% url 'accounts:export_customer_data' %}">
diff --git a/templates/manage_events/event_add.html b/templates/manage_events/event_add.html
index 2a598cb..60c74c1 100644
--- a/templates/manage_events/event_add.html
+++ b/templates/manage_events/event_add.html
@@ -5,7 +5,6 @@
 
     {% include "cdn_through_html/filepond_cdn_css.html" %}
     {% include "cdn_through_html/flatpicker_cdn_css.html" %}
-    {% include "cdn_through_html/quill_cdn_css.html" %}
     {% include "cdn_through_html/tagify_cdn_css.html" %}
     {% include "cdn_through_html/sweetalert2_cdn_css.html" %}
     {{form.media}}
@@ -60,20 +59,7 @@
 
 
     <script>
-        // Function to initialize Quill editor
-        function initializeQuillEditor(selector, placeholder) {
-            return new Quill(selector, {
-                modules: {
-                    toolbar: [
-                        [{ header: [1, 2, false] }],
-                        ['bold', 'italic', 'underline'],
-                        ['image', 'code-block']
-                    ]
-                },
-                placeholder: placeholder,
-                theme: 'snow'  // or 'bubble'
-            });
-        }
+        
     
         // Function to initialize FilePond
         function initializeFilePond(selector, allowMultiple = false) {
@@ -140,8 +126,6 @@
             // Set multiple attribute for event images
             document.getElementById('id_event_images').setAttribute('multiple', '');
 
-            // Initialize Quill editor
-            var quill = initializeQuillEditor('#description', 'Write description...');
     
             // Initialize FilePond
             var thumbnail = initializeFilePond('id_image');
@@ -347,8 +331,7 @@
                     
                     // Disable the submit button to prevent multiple submissions
                     $('button[type="submit"]').prop('disabled', true);
-                    event.prevent()
-                    //form.submit();
+                    form.submit();
                 }
             });
         });
diff --git a/templates/manage_subscriptions/subscription_list.html b/templates/manage_subscriptions/subscription_list.html
index 1639afb..6e6896d 100644
--- a/templates/manage_subscriptions/subscription_list.html
+++ b/templates/manage_subscriptions/subscription_list.html
@@ -50,6 +50,9 @@
                                                         <th class="checkbox-column sorting_asc" tabindex="0"
                                                             aria-controls="style-3" aria-sort="ascending"
                                                             style="width: 69.2656px;"> Amount </th>
+                                                        {% comment %} <th class="checkbox-column sorting_asc" tabindex="0"
+                                                            aria-controls="style-3" aria-sort="ascending"
+                                                            style="width: 69.2656px;"> Customer Type</th> {% endcomment %}
                                                         <th class="checkbox-column sorting_asc" tabindex="0"
                                                             aria-controls="style-3" aria-sort="ascending"
                                                             style="width: 69.2656px;"> Free for Admin </th>
@@ -67,6 +70,8 @@
                                                         <td>{{data_obj.title}}</td>
                                                         <td>{{data_obj.plan.days}}</td>
                                                         <td>{{data_obj.amount}}</td>
+                                                        {% comment %} <td>{{data_obj.principal_types.name}}</td> {% endcomment %}
+                                                        
                                                         <td class="text-center">
                                                             <span class="shadow-none badge {% if data_obj.is_free %}badge-primary{% else %}badge-danger{% endif %}">{{data_obj.is_free}}</span>
                                                         </td>
@@ -86,8 +91,9 @@
                                                                             <path
                                                                                 d="M17 3a2.828 2.828 0 1 1 4 4L7.5 20.5 2 22l1.5-5.5L17 3z">
                                                                             </path>
-                                                                        </svg></a></li>
-                                                                
+                                                                        </svg>
+                                                                    </a>
+                                                                </li>
                                                             </ul>
                                                         </td>
                                                     </tr>
diff --git a/templates/manage_venues/venue_add.html b/templates/manage_venues/venue_add.html
index deb9b5c..922cafe 100644
--- a/templates/manage_venues/venue_add.html
+++ b/templates/manage_venues/venue_add.html
@@ -3,6 +3,19 @@
 {% block stylesheet %}
 <!-- include required css cdn link through html here -->
 
+<style>
+    /* Inline CSS for removing number input spinners */
+    input[type="number"]::-webkit-outer-spin-button,
+    input[type="number"]::-webkit-inner-spin-button {
+        -webkit-appearance: none;
+        margin: 0;
+    }
+
+    input[type="number"] {
+        -moz-appearance: textfield;
+    }
+</style>
+
 {% include "cdn_through_html/filepond_cdn_css.html" %}
 {{form.media}}
 
@@ -47,14 +60,9 @@
 
 {% block javascript %}
 
-
-
-
-
 <!-- include required css cdn link through html here -->
 
 {% include "cdn_through_html/filepond_cdn_js.html" %}
-
 {% include "cdn_through_html/jquery_validate_cdn_js.html" %}
 
 
@@ -76,7 +84,6 @@
                 },
                 title: {
                     required: true,
-                    noSpace: true
                 },
                 address: {
                     required: true