fix: return _handle_no_page when page is None

sakhawy · sakhawy · commit 3e159229b500 · 2024-01-27T00:37:54.000+02:00
diff --git a/cms/tests/test_views.py b/cms/tests/test_views.py
@@ -74,6 +74,14 @@ def test_handle_no_page_for_root_url(self):
         self.assertEqual(response.status_code, 302)
         self.assertEqual(response.url, reverse('admin:cms_pagecontent_changelist'))
 
+    def test_handle_no_page_for_rool_url_no_homepage(self):
+        """
+        Test details view when visiting root and homepage doesn't exist
+        """
+        create_page("one", "nav_playground.html", "en")
+        response = self.client.get("/en/")
+        self.assertEqual(response.status_code, 302)
+
     def test_apphook_not_hooked(self):
         """
         Test details view when apphook pool has apphooks, but they're not
diff --git a/cms/views.py b/cms/views.py
@@ -115,8 +115,10 @@ def details(request, slug):
             return HttpResponseRedirect(redirect_url)
 
     if not page:
-        # raise 404
-        _handle_no_page(request)
+        # raise 404 or redirect to PageContent's
+        # changelist in the admin if this is a
+        # request to the root URL
+        return _handle_no_page(request)
 
     request.current_page = page
 
