conntrackt Changeset - 4d15241bfa82

Changeset - 4d15241bfa82

Parent rev.

Child rev.

[Not reviewed]

default

5 1 1

Branko Majic (branko) - 12 years ago 2013-07-25 20:02:43
branko@majic.rs

CONNT-15: Reduced repetition for the delete form by using a mixin and single template.

7 files changed with 79 insertions and 140 deletions:

conntrackt/templates/conntrackt/communication_confirm_delete.html

conntrackt/templates/conntrackt/delete_form.html

conntrackt/templates/conntrackt/entity_confirm_delete.html

conntrackt/templates/conntrackt/interface_confirm_delete.html

conntrackt/templates/conntrackt/location_confirm_delete.html

conntrackt/templates/conntrackt/project_confirm_delete.html

conntrackt/views.py

0 comments (0 inline, 0 general)

conntrackt/templates/conntrackt/communication_confirm_delete.html

➞

Show inline comments

deleted file

conntrackt/templates/conntrackt/delete_form.html

➞

Show inline comments

@@ new file 100644 @@
 {% extends "conntrackt/base.html" %}
 {# For html_link #}
 {% load conntrackt_tags %}
 {# For Bootstrapped forms #}
 {% load crispy_forms_tags %}
 {% block content %}
 <div class="row">
   <h1 class="span12">{{headline}}</h1>
 </div>
 <div class="row">
   <div class="span12">
     <form action="" method="post">
       <div class="controls controls-row">
         {% csrf_token %}
         {{ form }}
         Are you sure you want to remove this project?
       </div>
       <hr>
       <div class="controls">
         <button type="submit" class="btn btn-primary">Remove</button>
       </div>
     </form>
   </div>
 </div>
 {% endblock content %}

conntrackt/templates/conntrackt/entity_confirm_delete.html

➞

Show inline comments

deleted file

conntrackt/templates/conntrackt/interface_confirm_delete.html

➞

Show inline comments

deleted file

conntrackt/templates/conntrackt/location_confirm_delete.html

➞

Show inline comments

deleted file

conntrackt/templates/conntrackt/project_confirm_delete.html

➞

Show inline comments

deleted file

conntrackt/views.py

➞

Show inline comments

@@ @@ -291,12 +291,13 @@ class ProjectUpdateView(SetHeadlineMixin @@
         return "Update project %s" % self.object.name
 class ProjectDeleteView(MultiplePermissionsRequiredMixin, DeleteView):
+class ProjectDeleteView(SetHeadlineMixin, MultiplePermissionsRequiredMixin, DeleteView):
     """
     View for deleting a project.
     """
     model = Project
     template_name = "conntrackt/delete_form.html"
     # Required permissions.
     permissions = {
@@ @@ -318,6 +319,13 @@ class ProjectDeleteView(MultiplePermissi @@
         return super(ProjectDeleteView, self).post(*args, **kwargs)
     def get_headline(self):
         """
         Set headline based on project name.
         """
         return "Delete project %s" % self.object.name
 class LocationCreateView(SetHeadlineMixin, MultiplePermissionsRequiredMixin, CreateView):
     """
@@ @@ -367,12 +375,13 @@ class LocationUpdateView(SetHeadlineMixi @@
         return "Update location %s" % self.object.name
 class LocationDeleteView(MultiplePermissionsRequiredMixin, DeleteView):
+class LocationDeleteView(SetHeadlineMixin, MultiplePermissionsRequiredMixin, DeleteView):
     """
     View for deleting a location.
     """
     model = Location
     template_name = "conntrackt/delete_form.html"
     # Required permissions.
     permissions = {
@@ @@ -394,6 +403,13 @@ class LocationDeleteView(MultiplePermiss @@
         return super(LocationDeleteView, self).post(*args, **kwargs)
     def get_headline(self):
         """
         Set headline based on location name.
         """
         return "Delete location %s" % self.object.name
 class EntityCreateView(SetHeadlineMixin, MultiplePermissionsRequiredMixin, CreateView):
     """
@@ @@ -476,12 +492,13 @@ class EntityUpdateView(SetHeadlineMixin, @@
         return "Update entity %s" % self.object.name
 class EntityDeleteView(MultiplePermissionsRequiredMixin, DeleteView):
+class EntityDeleteView(SetHeadlineMixin, MultiplePermissionsRequiredMixin, DeleteView):
     """
     View for deleting an entity.
     """
     model = Entity
     template_name = "conntrackt/delete_form.html"
     # Required permissions.
     permissions = {
@@ @@ -513,6 +530,13 @@ class EntityDeleteView(MultiplePermissio @@
         return super(EntityDeleteView, self).delete(*args, **kwargs)
     def get_headline(self):
         """
         Set headline based on entity name.
         """
         return "Delete entity %s" % self.object.name
 class InterfaceCreateView(SetHeadlineMixin, MultiplePermissionsRequiredMixin, CreateView):
     """
@@ @@ -624,12 +648,13 @@ class InterfaceUpdateView(SetHeadlineMix @@
         return "Update interface %s" % self.object.name
 class InterfaceDeleteView(MultiplePermissionsRequiredMixin, DeleteView):
+class InterfaceDeleteView(SetHeadlineMixin, MultiplePermissionsRequiredMixin, DeleteView):
     """
     View for deleting an interface.
     """
     model = Interface
     template_name = "conntrackt/delete_form.html"
     # Required permissions.
     permissions = {
@@ @@ -661,6 +686,13 @@ class InterfaceDeleteView(MultiplePermis @@
         return super(InterfaceDeleteView, self).delete(*args, **kwargs)
     def get_headline(self):
         """
         Set headline based on interface name.
         """
         return "Delete interface %s" % self.object.name
 class CommunicationCreateView(SetHeadlineMixin, MultiplePermissionsRequiredMixin, CreateView):
     """
@@ @@ -746,6 +778,13 @@ class CommunicationCreateView(SetHeadlin @@
         return self.request.GET.get("next", reverse("project", args=(self.object.source.entity.project.pk,)))
     def get_headline(self):
         """
         Set headline based on interface name.
         """
         return "Delete interface %s" % self.object.name
 class CommunicationUpdateView(SetHeadlineMixin, MultiplePermissionsRequiredMixin, UpdateView):
     """
@@ @@ -802,12 +841,13 @@ class CommunicationUpdateView(SetHeadlin @@
         return "Update communication %s" % self.object
 class CommunicationDeleteView(MultiplePermissionsRequiredMixin, DeleteView):
+class CommunicationDeleteView(SetHeadlineMixin, MultiplePermissionsRequiredMixin, DeleteView):
     """
     View for deleting an communication.
     """
     model = Communication
     template_name = "conntrackt/delete_form.html"
     # Required permissions.
     permissions = {
@@ @@ -840,3 +880,10 @@ class CommunicationDeleteView(MultiplePe @@
         self.success_url = reverse("entity", args=(entity_id,))
         return super(CommunicationDeleteView, self).delete(*args, **kwargs)
     def get_headline(self):
         """
         Set headline based on communication.
         """
         return "Delete communication %s" % self.object

0 comments (0 inline, 0 general)