From 885e4485ee84f1c0584db0aba308ec879f2f8883 Mon Sep 17 00:00:00 2001
From: Dave Miller <justdave@thunderbird.net>
Date: Sun, 13 Sep 2020 11:58:52 -0400
Subject: [PATCH] use localized link for thunderbird/all. fixes #198

---
 builder.py |  4 ++--
 helper.py  | 10 +++++++---
 2 files changed, 9 insertions(+), 5 deletions(-)

diff --git a/builder.py b/builder.py
index 9422f1a2a..ed096405f 100644
--- a/builder.py
+++ b/builder.py
@@ -215,8 +215,8 @@ def build_notes(self):
         # Build htaccess files for sysreq and release notes redirects.
         sysreq_path = os.path.join(self.renderpath, 'system-requirements')
         notes_path = os.path.join(self.renderpath, 'notes')
-        write_htaccess(sysreq_path, settings.CANONICAL_URL + helper.thunderbird_url('system-requirements'))
-        write_htaccess(notes_path, settings.CANONICAL_URL + helper.thunderbird_url('releasenotes'))
+        write_htaccess(sysreq_path, settings.CANONICAL_URL + helper.thunderbird_url({'LANG':'en-US'},'system-requirements'))
+        write_htaccess(notes_path, settings.CANONICAL_URL + helper.thunderbird_url({'LANG':'en-US'},'releasenotes'))
 
     def build_assets(self):
         """Build assets, that is, bundle and compile the LESS and JS files in `settings.ASSETS`."""
diff --git a/helper.py b/helper.py
index c55741691..0371b836d 100644
--- a/helper.py
+++ b/helper.py
@@ -290,7 +290,8 @@ def download_thunderbird(ctx, channel='release', dom_id=None,
     return jinja2.Markup(html)
 
 
-def thunderbird_url(page, channel='None'):
+@jinja2.contextfunction
+def thunderbird_url(ctx, page, channel='None'):
     """
     Return a product-related URL like /thunderbird/all/ or /thunderbird/beta/60.0/releasenotes/.
     page = ('system-requirements', 'all', 'releasenotes')
@@ -300,18 +301,21 @@ def thunderbird_url(page, channel='None'):
         {{ thunderbird_url('system-requirements', channel) }}
     """
 
+    lang = ctx['LANG']
     channel = channel or 'release'
     version = thunderbird_desktop.latest_version(channel)
     # replace 'b1', 'b2' etc in beta version with just 'beta', since we don't generate
     # new notes for each beta iteration.
     version = re.sub(r"b[1-9][0-9]?", "beta", version)
 
+    # system reqs and release notes don't get localized
     url = '/en-US/thunderbird/{0}/{1}/'.format(version, page)
 
+    # but the 'all' page does
     if page == 'all':
-        url = '/en-US/thunderbird/{0}/{1}/'.format(channel, page)
+        url = '/{0}/thunderbird/{1}/{2}/'.format(lang, channel, page)
         if channel == 'release':
-            url = '/en-US/thunderbird/{0}/'.format(page)
+            url = '/{0}/thunderbird/{1}/'.format(lang, page)
 
     return url