From 3a89537a0c8d3b722b9779bd6623822d3f14801c Mon Sep 17 00:00:00 2001
From: shagi <shagi@gisa-elkartea.org>
Date: Thu, 1 Sep 2022 11:24:01 +0200
Subject: [PATCH] Allow overriding home url

---
 gas/templates/gas/base.html | 4 ++--
 gas/tests/test_views.py     | 6 +++++-
 gas/views.py                | 5 +++++
 runtests.py                 | 1 +
 4 files changed, 13 insertions(+), 3 deletions(-)

diff --git a/gas/templates/gas/base.html b/gas/templates/gas/base.html
index 26677be..efb7021 100644
--- a/gas/templates/gas/base.html
+++ b/gas/templates/gas/base.html
@@ -15,7 +15,7 @@
     {% if not is_popup %}
     <nav id="navbar-main" role="navigation">
         <div class="navbar-header">
-            <a class="navbar-brand" href="{% url "gas:index" %}"><img id="logo" src="{% static logo_static_url %}"></a>
+            <a class="navbar-brand" href="{{ home_url }}"><img id="logo" src="{% static logo_static_url %}"></a>
         </div>
         <ul id="main-navigation">
             <li>{{ user }}</li>
@@ -41,7 +41,7 @@
             {% if not is_popup %}
             <nav class="breadcrumb">
                 <ol>
-                    <li><a href="{% url "gas:index" %}" title="{% trans "Home" %}"><i class="fas fa-home fa-fw"></i></a></li>
+                    <li><a href="{{ home_url }}" title="{% trans "Home" %}"><i class="fas fa-home fa-fw"></i></a></li>
                     {% for bc_url, bc_label in breadcrumbs %}
                         {% if bc_url %}
                             <li><a href="{{ bc_url }}">{{ bc_label }}</a></li>
diff --git a/gas/tests/test_views.py b/gas/tests/test_views.py
index 068e68a..eb0864f 100644
--- a/gas/tests/test_views.py
+++ b/gas/tests/test_views.py
@@ -228,10 +228,14 @@ class GASMixinTestCase(TestCase):
             ]
             template_name = 'gas/base.html'
 
+            def get_home_url(self):
+                return 'home_url'
+
         view = SampleView.as_view()
         request = self.request_factory.get('some_url')
         request.user = self.admin_user
         response = view(request)
+        self.assertEqual(response.context_data['home_url'], 'home_url')
         self.assertEqual(response.context_data['title'], SampleView.title)
         self.assertEqual(response.context_data['header_title'], SampleView.header_title)
         self.assertEqual(response.context_data['help_text'], SampleView.help_text)
@@ -428,7 +432,7 @@ class GASDeleteTestCase(TestCase):
             success_url = 'success'
             confirmation_text = 'delete {object.username}'
             deleted_text = '{object.username} deleted'
-        
+
         view = SampleDeleteView.as_view()
 
         request = request_factory.get('some_url')
diff --git a/gas/views.py b/gas/views.py
index 938bf7b..64c22f6 100644
--- a/gas/views.py
+++ b/gas/views.py
@@ -110,6 +110,10 @@ class GASMixin:
     def get_success_message(self):
         return self.success_message
 
+    def get_home_url(self):
+        """ Url for the home of the control panel """
+        return reverse('gas:index')
+
     def get_cancel_url(self):
         if self.cancel_url:
             # Forcing possible reverse_lazy evaluation
@@ -154,6 +158,7 @@ class GASMixin:
             js = js + gas_settings.EXTRA_MEDIA.get('js', [])
         ctx.update({
             'base_template': self.base_template,
+            'home_url': self.get_home_url(),
             'header_title': self.get_header_title(),
             'title': self.get_title(),
             'help_text': self.get_help_text(),
diff --git a/runtests.py b/runtests.py
index 6a4e1c9..f697bf4 100755
--- a/runtests.py
+++ b/runtests.py
@@ -10,6 +10,7 @@ def runtests():
     if not settings.configured:
         # Configure test environment
         settings.configure(
+            SECRET_KEY="test-secret",
             DATABASES={
                 'default': {
                     'ENGINE': 'django.db.backends.sqlite3',