copilot-swe-agent[bot]
df4c91a586
- Fix detect_language() to safely handle unhashable session values and requests missing cookies/headers attributes (TypeError + AttributeError) - Add default English `_()` translation function to Jinja2 template environment globals so error pages always have it available - Fix app/main.py exception handlers to use a dedicated error templates instance with `_` registered, keeping it separate from view templates to avoid test patches breaking error rendering - Fix app/views/plans.py to import shared templates from app.views.base instead of creating its own Jinja2Templates instance - Make migration 029_add_user_language_preference idempotent: skip ALTER TABLE if user_profiles table does not exist - Update test_i18n.py expectations to reflect 31 supported languages - Create 21 missing translation files (nb, da, sv, fi, is, ga, lb, ca, cs, sk, hu, sl, hr, ro, bg, el, et, lv, lt, tr, uk) with English placeholder translations - Update de.json with 117 missing translation keys including proper German translations - Update es, fr, it, nl, pl, pt, ru, zh translation files with missing keys using English fallbacks Co-authored-by: christianlouis <361235+christianlouis@users.noreply.github.com>
113 lines
4.7 KiB
Python
113 lines
4.7 KiB
Python
"""
|
|
Base setup for views, containing shared functionality and imports.
|
|
"""
|
|
|
|
import logging
|
|
from pathlib import Path
|
|
|
|
from fastapi import APIRouter, Depends, HTTPException, Request # noqa: F401
|
|
from fastapi.templating import Jinja2Templates
|
|
from sqlalchemy.orm import Session # noqa: F401
|
|
|
|
from app.auth import require_login # noqa: F401
|
|
from app.config import settings
|
|
from app.database import get_db # noqa: F401
|
|
from app.utils.i18n import (
|
|
SUPPORTED_LANGUAGES,
|
|
detect_language,
|
|
format_date,
|
|
format_datetime,
|
|
format_number,
|
|
translate,
|
|
)
|
|
|
|
# Set up Jinja2 templates
|
|
templates_dir = Path(__file__).parent.parent.parent / "frontend" / "templates"
|
|
templates = Jinja2Templates(directory=str(templates_dir))
|
|
|
|
# Add Python built-in functions to Jinja2 template globals
|
|
templates.env.globals["min"] = min
|
|
templates.env.globals["max"] = max
|
|
|
|
# ---------------------------------------------------------------------------
|
|
# i18n Jinja2 integration
|
|
# ---------------------------------------------------------------------------
|
|
# The _() function is available in every template to translate UI strings.
|
|
# Usage: {{ _("nav.dashboard") }} or {{ _("upload.max_size", size="10 MB") }}
|
|
# The locale is automatically resolved from the request context.
|
|
# A default English implementation is registered as a global so error handlers
|
|
# that don't go through _inject_global_context still have the function available.
|
|
# ---------------------------------------------------------------------------
|
|
|
|
templates.env.globals["supported_languages"] = SUPPORTED_LANGUAGES
|
|
templates.env.globals["_"] = lambda key, **kwargs: translate(key, "en", **kwargs)
|
|
|
|
# Customize Jinja2Templates to include app_version in all templates
|
|
original_template_response = templates.TemplateResponse
|
|
|
|
|
|
def _inject_global_context(ctx: dict) -> None:
|
|
"""Inject shared global variables into every template context dict."""
|
|
ctx.setdefault("version", settings.version)
|
|
ctx.setdefault("release_name", getattr(settings, "release_name", None))
|
|
ctx.setdefault("ui_default_color_scheme", getattr(settings, "ui_default_color_scheme", "system"))
|
|
ctx.setdefault("multi_user_enabled", getattr(settings, "multi_user_enabled", False))
|
|
ctx.setdefault("auth_enabled", getattr(settings, "auth_enabled", True))
|
|
ctx.setdefault(
|
|
"allow_signup",
|
|
getattr(settings, "multi_user_enabled", False) and getattr(settings, "allow_local_signup", False),
|
|
)
|
|
|
|
req = ctx.get("request")
|
|
if req is not None:
|
|
# CSRF token
|
|
if hasattr(req, "state") and hasattr(req.state, "csrf_token"):
|
|
ctx.setdefault("csrf_token", req.state.csrf_token)
|
|
# Determine whether the current visitor is authenticated
|
|
session_user = None
|
|
if hasattr(req, "session"):
|
|
session_user = req.session.get("user")
|
|
# When auth is disabled every visitor is effectively "logged in"
|
|
ctx.setdefault("is_logged_in", not getattr(settings, "auth_enabled", True) or session_user is not None)
|
|
|
|
# --- i18n: detect language and register template helpers ---
|
|
current_locale = detect_language(req)
|
|
ctx.setdefault("current_locale", current_locale)
|
|
|
|
def _translate(key: str, **kwargs: object) -> str:
|
|
return translate(key, current_locale, **kwargs)
|
|
|
|
def _format_date(value: object, short: bool = False) -> str:
|
|
return format_date(value, current_locale, short=short) # type: ignore[arg-type]
|
|
|
|
def _format_datetime(value: object) -> str:
|
|
return format_datetime(value, current_locale) # type: ignore[arg-type]
|
|
|
|
def _format_number(value: object) -> str:
|
|
return format_number(value, current_locale) # type: ignore[arg-type]
|
|
|
|
ctx.setdefault("_", _translate)
|
|
ctx.setdefault("format_date_l10n", _format_date)
|
|
ctx.setdefault("format_datetime_l10n", _format_datetime)
|
|
ctx.setdefault("format_number_l10n", _format_number)
|
|
else:
|
|
ctx.setdefault("is_logged_in", not getattr(settings, "auth_enabled", True))
|
|
ctx.setdefault("current_locale", "en")
|
|
ctx.setdefault("_", lambda key, **kw: translate(key, "en", **kw))
|
|
|
|
|
|
def template_response_with_version(*args, **kwargs):
|
|
"""Wrapper for TemplateResponse to include version and CSRF token in all templates"""
|
|
# If context dict is provided, add version to it
|
|
if len(args) >= 2 and isinstance(args[1], dict):
|
|
_inject_global_context(args[1])
|
|
elif "context" in kwargs and isinstance(kwargs["context"], dict):
|
|
_inject_global_context(kwargs["context"])
|
|
return original_template_response(*args, **kwargs)
|
|
|
|
|
|
templates.TemplateResponse = template_response_with_version
|
|
|
|
# Set up logging
|
|
logger = logging.getLogger(__name__)
|