From 3652568e8c95e55a3a96baedf46272bdf17e85f7 Mon Sep 17 00:00:00 2001
From: nickmwangemi <nickmwangemi@gmail.com>
Date: Tue, 15 Aug 2023 18:44:46 +0300
Subject: [PATCH] By-laws on LawLibrary

---
 lawlibrary/constants.py                       |  1 +
 lawlibrary/context_processors.py              | 10 +++++
 lawlibrary/settings.py                        |  5 +++
 .../liiweb/locality_legislation_list.html     | 10 +++++
 lawlibrary/templates/peachjam/_header.html    |  6 +++
 .../peachjam/legislation_detail.html          | 12 ++++--
 lawlibrary/urls.py                            | 16 +++++---
 lawlibrary/views/legislation.py               | 39 +++++++++++++++++--
 .../liiweb/locality_legislation.html          |  2 +-
 .../liiweb/locality_legislation_list.html     | 11 ++++--
 liiweb/views/legislation.py                   |  2 +-
 11 files changed, 96 insertions(+), 18 deletions(-)
 create mode 100644 lawlibrary/constants.py
 create mode 100644 lawlibrary/context_processors.py
 create mode 100644 lawlibrary/templates/liiweb/locality_legislation_list.html

diff --git a/lawlibrary/constants.py b/lawlibrary/constants.py
new file mode 100644
index 000000000..8dc7106c1
--- /dev/null
+++ b/lawlibrary/constants.py
@@ -0,0 +1 @@
+PROVINCIAL_CODES = ["ec", "fs", "gp", "lp", "mp", "nc", "nw", "wc", "kzn", "transvaal"]
diff --git a/lawlibrary/context_processors.py b/lawlibrary/context_processors.py
new file mode 100644
index 000000000..6e3bbf4cf
--- /dev/null
+++ b/lawlibrary/context_processors.py
@@ -0,0 +1,10 @@
+from lawlibrary.constants import PROVINCIAL_CODES
+
+
+def lawlibrary(request):
+    """
+    Add some useful context to templates.
+    """
+    return {
+        "PROVINCIAL_CODES": PROVINCIAL_CODES,
+    }
diff --git a/lawlibrary/settings.py b/lawlibrary/settings.py
index a2a3e2b45..39923b304 100644
--- a/lawlibrary/settings.py
+++ b/lawlibrary/settings.py
@@ -7,3 +7,8 @@
 JAZZMIN_SETTINGS["site_title"] = "Lawlibrary"  # noqa
 JAZZMIN_SETTINGS["site_header"] = "Lawlibrary"  # noqa
 JAZZMIN_SETTINGS["site_brand"] = "Lawlibrary.org.za"  # noqa
+
+
+TEMPLATES[0]["OPTIONS"]["context_processors"].append(  # noqa
+    "lawlibrary.context_processors.lawlibrary"
+)
diff --git a/lawlibrary/templates/liiweb/locality_legislation_list.html b/lawlibrary/templates/liiweb/locality_legislation_list.html
new file mode 100644
index 000000000..b0f398912
--- /dev/null
+++ b/lawlibrary/templates/liiweb/locality_legislation_list.html
@@ -0,0 +1,10 @@
+{% extends "liiweb/locality_legislation_list.html" %}
+{% load i18n %}
+{% block page-heading %}
+  {% if locality.code in PROVINCIAL_CODES %}
+    <h1 class="my-4">{% blocktrans with locality=locality.name %}{{ locality }} Legislation{% endblocktrans %}</h1>
+  {% else %}
+    <h1 class="my-4">{% blocktrans with locality=locality.name %}{{ locality }} By-laws{% endblocktrans %}</h1>
+  {% endif %}
+  {% include 'lawlibrary/_legislation_alert.html' %}
+{% endblock %}
diff --git a/lawlibrary/templates/peachjam/_header.html b/lawlibrary/templates/peachjam/_header.html
index 14de65d0e..e4ca84bd8 100644
--- a/lawlibrary/templates/peachjam/_header.html
+++ b/lawlibrary/templates/peachjam/_header.html
@@ -27,6 +27,12 @@
       {% trans 'Provincial Legislation' %}
     </a>
   </li>
+  <li class="nav-item">
+    <a class="nav-link {% if view.navbar_link == 'legislation/municipal' %}active{% endif %}"
+       href="{% url 'municipal_legislation' %}">
+      {% trans 'Municipal By-laws' %}
+    </a>
+  </li>
   <li class="nav-item">
     <a class="nav-link {% if view.navbar_link == 'gazettes' %}active{% endif %}"
        href="{% url 'gazettes' %}">
diff --git a/lawlibrary/templates/peachjam/legislation_detail.html b/lawlibrary/templates/peachjam/legislation_detail.html
index 5a43633c8..14a97e65f 100644
--- a/lawlibrary/templates/peachjam/legislation_detail.html
+++ b/lawlibrary/templates/peachjam/legislation_detail.html
@@ -4,9 +4,15 @@
   <nav aria-label="breadcrumb">
     {% if document.locality %}
       <ol class="breadcrumb">
-        <li class="breadcrumb-item">
-          <a href="{% url 'locality_legislation' %}">{% trans 'Provincial Legislation' %}</a>
-        </li>
+        {% if document.locality.code in PROVINCIAL_CODES %}
+          <li class="breadcrumb-item">
+            <a href="{% url 'locality_legislation' %}">{% trans 'Provincial Legislation' %}</a>
+          </li>
+        {% else %}
+          <li class="breadcrumb-item">
+            <a href="{% url 'municipal_legislation' %}">{% trans 'Municipal By-laws' %}</a>
+          </li>
+        {% endif %}
         <li class="breadcrumb-item">
           <a href="{% url 'locality_legislation_list' document.locality.code %}">{{ document.locality.name }}</a>
         </li>
diff --git a/lawlibrary/urls.py b/lawlibrary/urls.py
index 949d5375e..ca0156c02 100644
--- a/lawlibrary/urls.py
+++ b/lawlibrary/urls.py
@@ -1,14 +1,13 @@
 from django.urls import include, path
 
 from lawlibrary import views
-from liiweb.views import LocalityLegislationListView
 
 urlpatterns = [
     path("", views.HomePageView.as_view(), name="home_page"),
     path("legislation/", views.LegislationListView.as_view(), name="legislation_list"),
     path(
         "legislation/provincial",
-        views.LocalityLegislationView.as_view(),
+        views.LocalityLegislationView.as_view(variant="provincial"),
         name="locality_legislation",
     ),
     path(
@@ -23,23 +22,28 @@
     ),
     path(
         "legislation/<str:code>/",
-        LocalityLegislationListView.as_view(),
+        views.LocalityLegislationListView.as_view(),
         name="locality_legislation_list",
     ),
     path(
         "legislation/<str:code>/repealed",
-        LocalityLegislationListView.as_view(variant="repealed"),
+        views.LocalityLegislationListView.as_view(variant="repealed"),
         name="locality_legislation_list_repealed",
     ),
     path(
         "legislation/<str:code>/subsidiary",
-        LocalityLegislationListView.as_view(variant="subleg"),
+        views.LocalityLegislationListView.as_view(variant="subleg"),
         name="locality_legislation_list_subsidiary",
     ),
     path(
         "legislation/<str:code>/all",
-        LocalityLegislationListView.as_view(variant="all"),
+        views.LocalityLegislationListView.as_view(variant="all"),
         name="locality_legislation_list_all",
     ),
+    path(
+        "legislation/municipal",
+        views.LocalityLegislationView.as_view(variant="municipal"),
+        name="municipal_legislation",
+    ),
     path("", include("liiweb.urls")),
 ]
diff --git a/lawlibrary/views/legislation.py b/lawlibrary/views/legislation.py
index eb4a0e3ed..b458a83c4 100644
--- a/lawlibrary/views/legislation.py
+++ b/lawlibrary/views/legislation.py
@@ -1,5 +1,9 @@
+from lawlibrary.constants import PROVINCIAL_CODES
 from liiweb.views import LegislationListView as BaseLegislationListView
+from liiweb.views import LocalityLegislationListView as BaseLocalityLegislationListView
 from liiweb.views import LocalityLegislationView as BaseLocalityLegislationView
+from peachjam.helpers import chunks
+from peachjam.models import Locality
 
 
 class LegislationListView(BaseLegislationListView):
@@ -8,7 +12,36 @@ def get_queryset(self):
 
 
 class LocalityLegislationView(BaseLocalityLegislationView):
+    variant = None
+
+    def get_context_data(self, **kwargs):
+        context = super().get_context_data(**kwargs)
+
+        if self.variant == "provincial":
+            context["locality_legislation_title"] = "Provincial Legislation"
+            localities = Locality.objects.filter(code__in=PROVINCIAL_CODES)
+            context["locality_groups"] = list(chunks(localities, 2))
+
+        if self.variant == "municipal":
+            context["locality_legislation_title"] = "Municipal By-laws"
+            localities = Locality.objects.exclude(code__in=PROVINCIAL_CODES)
+            context["locality_groups"] = list(chunks(localities, 2))
+            self.navbar_link = "legislation/municipal"
+
+        return context
+
+
+class LocalityLegislationListView(BaseLocalityLegislationListView):
     def get_context_data(self, **kwargs):
-        return super().get_context_data(
-            locality_legislation_title="Provincial Legislation", **kwargs
-        )
+        context = super().get_context_data(**kwargs)
+
+        context["locality"] = self.locality
+
+        if self.locality.code in PROVINCIAL_CODES:
+            context["locality_legislation_title"] = "Provincial Legislation"
+            context["show_subleg"] = True
+        else:
+            context["locality_legislation_title"] = "Municipal By-laws"
+            self.navbar_link = "legislation/municipal"
+
+        return context
diff --git a/liiweb/templates/liiweb/locality_legislation.html b/liiweb/templates/liiweb/locality_legislation.html
index 179984410..bf3d60eab 100644
--- a/liiweb/templates/liiweb/locality_legislation.html
+++ b/liiweb/templates/liiweb/locality_legislation.html
@@ -6,7 +6,7 @@
     <div class="container">
       <h1 class="my-4">{{ locality_legislation_title }}</h1>
       <div class="row">
-        {% for group in province_groups %}
+        {% for group in locality_groups %}
           <div class="col-lg">
             <ul class="list-unstyled mb-0">
               {% for locality in group %}
diff --git a/liiweb/templates/liiweb/locality_legislation_list.html b/liiweb/templates/liiweb/locality_legislation_list.html
index 1946f3c8a..3903f78dc 100644
--- a/liiweb/templates/liiweb/locality_legislation_list.html
+++ b/liiweb/templates/liiweb/locality_legislation_list.html
@@ -7,6 +7,7 @@
       <li class="breadcrumb-item">
         <a href="{% url 'locality_legislation' %}">{{ locality_legislation_title }}</a>
       </li>
+      <li class="breadcrumb-item active" aria-current="page">{{ locality.name }}</li>
     </ol>
   </nav>
 {% endblock %}
@@ -24,10 +25,12 @@ <h1 class="my-4">{% blocktrans with locality=locality.name %}{{ locality }} Legi
       <a class="nav-link {% if view.variant == 'repealed' %}active{% endif %}"
          href="{% url 'locality_legislation_list_repealed' locality.code %}">Repealed legislation</a>
     </li>
-    <li class="nav-item">
-      <a class="nav-link {% if view.variant == 'subleg' %}active{% endif %}"
-         href="{% url 'locality_legislation_list_subsidiary' locality.code %}">{{ PEACHJAM_SETTINGS.subleg_label }}</a>
-    </li>
+    {% if show_subleg %}
+      <li class="nav-item">
+        <a class="nav-link {% if view.variant == 'subleg' %}active{% endif %}"
+           href="{% url 'locality_legislation_list_subsidiary' locality.code %}">{{ PEACHJAM_SETTINGS.subleg_label }}</a>
+      </li>
+    {% endif %}
     <li class="nav-item">
       <a class="nav-link {% if view.variant == 'all' %}active{% endif %}"
          href="{% url 'locality_legislation_list_all' locality.code %}">All legislation</a>
diff --git a/liiweb/views/legislation.py b/liiweb/views/legislation.py
index a51872330..168622ad1 100644
--- a/liiweb/views/legislation.py
+++ b/liiweb/views/legislation.py
@@ -76,7 +76,7 @@ class LocalityLegislationView(TemplateView):
     def get_context_data(self, **kwargs):
         context = super().get_context_data(**kwargs)
         localities = Locality.objects.all()
-        context["province_groups"] = list(chunks(localities, 2))
+        context["locality_groups"] = list(chunks(localities, 2))
         return context