portugalfuturista/elixir
2
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

web: Make "Go to home page" link to current version

Browse source 
If project and version is valid. Otherwise, redirect to latest or /.
This commit is contained in:
Franciszek Stachura 2025-02-06 11:23:29 +01:00 committed by Théo Lebrun
parent 6edaafb2f3
commit 531529a182
2 changed files with 15 additions and 15 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

28
elixir/web.py
View file

@ -94,6 +94,7 @@ def get_project_error_page(req, resp, exception: ElixirProjectError):
'referer': req.referer if req.referer != req.uri else None,
'bug_report_link': get_github_issue_link(report_error_details),
'home_page_url': '/',
'report_error_details': report_error_details,
'error_title': exception.title,
@ -109,25 +110,24 @@ def get_project_error_page(req, resp, exception: ElixirProjectError):
get_url_with_new_version = lambda v: stringify_source_path(project, v, '/')
versions, current_version_path = get_versions(versions_raw, get_url_with_new_version, version)
template_ctx = {
**template_ctx,
'current_project': project,
'versions': versions,
'current_version_path': current_version_path,
}
if version is None:
if current_version_path[2] is None:
# If details about current version are not available, make base links
# point to latest.
# current_tag is not set to latest to avoid latest being highlighted in the sidebar
version = query.query('latest')
else:
template_ctx['current_tag'] = version
template_ctx['source_base_url'] = get_source_base_url(project, version)
template_ctx['ident_base_url'] = get_ident_base_url(project, version)
template_ctx = {
**template_ctx,
'current_project': project,
'current_tag': version,
'versions': versions,
'current_version_path': current_version_path,
'home_page_url': get_source_base_url(project, version),
'source_base_url': get_source_base_url(project, version),
'ident_base_url': get_ident_base_url(project, version),
}
if exception.description is not None:
template_ctx['error_details'] = exception.description

2
templates/error.html
View file

@ -16,7 +16,7 @@
{% if referer is not none -%}
<li><a class="link" href="{{ referer }}">Go back</a></li>
{%- endif %}
<li><a class="link" href="/">Go to home page</a></li>
<li><a class="link" href="{{ home_page_link }}">Go to home page</a></li>
{% if bug_report_link is not none -%}
<li><a class="link" href="{{ bug_report_link }}">Report a bug</a></li>
{%- endif %}