repos/website
1
Fork 0
Code Issues 1 Pull requests 27 Activity

Ensure request is passed into the singleton URL cache

Browse source 
This removes needing extra queries
This commit is contained in:
Jake Howard 2022-08-31 19:06:02 +01:00
parent 25e49de081
commit df09ec0f38
Signed by: jake
GPG key ID: 57AFB45680EDD477
8 changed files with 16 additions and 14 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
website/common/templates/common/navbar.html
View file

@ -2,7 +2,7 @@
<nav class="navbar is-family-code" role="navigation" aria-label="main navigation">
<div class="navbar-brand">
<a class="navbar-item home-link" href="{% pageurl homepage %}">
<a class="navbar-item home-link" href="{{ homepage_url }}">
/home/theorangeone
</a>

7
website/common/templatetags/footer_tags.py
View file

@ -6,8 +6,9 @@ from website.home.models import HomePage
register = Library()
@register.inclusion_tag("common/footer.html")
def footer() -> dict:
@register.inclusion_tag("common/footer.html", takes_context=True)
def footer(context: dict) -> dict:
request = context["request"]
return {
"homepage_url": SingletonPageCache.get_url(HomePage),
"homepage_url": SingletonPageCache.get_url(HomePage, request),
}

11
website/common/templatetags/navbar_tags.py
View file

@ -7,15 +7,16 @@ from website.search.models import SearchPage
register = Library()
@register.inclusion_tag("common/navbar.html")
def navbar() -> dict:
homepage = HomePage.objects.live().get()
@register.inclusion_tag("common/navbar.html", takes_context=True)
def navbar(context: dict) -> dict:
request = context["request"]
homepage = HomePage.objects.get()
return {
"homepage": homepage,
"homepage_url": SingletonPageCache.get_url(HomePage, request),
"nav_pages": homepage.get_children()
.live()
.public()
.filter(show_in_menus=True)
.order_by("title"),
"search_page_url": SingletonPageCache.get_url(SearchPage),
"search_page_url": SingletonPageCache.get_url(SearchPage, request),
}

2
website/common/tests/test_pages.py
View file

@ -52,7 +52,7 @@ class ListingPageTestCase(TestCase):
ContentPageFactory(parent=cls.page)
def test_accessible(self) -> None:
with self.assertNumQueries(33):
with self.assertNumQueries(32):
response = self.client.get(self.page.url)
self.assertEqual(response.status_code, 200)
self.assertEqual(len(response.context["listing_pages"]), 2)

2
website/common/tests/test_views.py
View file

@ -16,7 +16,7 @@ class Error404PageTestCase(TestCase):
)
def test_queries(self) -> None:
with self.assertNumQueries(17):
with self.assertNumQueries(16):
self.client.get(self.url)

2
website/home/models.py
View file

@ -49,5 +49,5 @@ class HomePage(BasePage, WagtailImageMetadataMixin):
.order_by("-date")
.first()
)
context["search_page_url"] = SingletonPageCache.get_url(SearchPage)
context["search_page_url"] = SingletonPageCache.get_url(SearchPage, request)
return context

2
website/search/tests.py
View file

@ -89,7 +89,7 @@ class SearchPageResultsTestCase(TestCase):
)
def test_too_high_page(self) -> None:
with self.assertNumQueries(48):
with self.assertNumQueries(46):
response = self.client.get(
self.url, {"q": "post", "page": 3}, HTTP_HX_REQUEST="true"
)

2
website/well_known/views.py
View file

@ -24,7 +24,7 @@ class SecurityView(TemplateView):
def get_context_data(self, **kwargs: dict) -> dict:
context = super().get_context_data(**kwargs)
context["security_txt"] = self.request.build_absolute_uri(self.request.path)
contact_page_path = SingletonPageCache.get_url(ContactPage)
contact_page_path = SingletonPageCache.get_url(ContactPage, self.request)
context["contact_page_url"] = (
self.request.build_absolute_uri(contact_page_path)
if contact_page_path