giorgio/read.markets
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Revert "i18n: add diagnostic logging to localizer + lang-toggle click path"

Browse source 
This reverts commit 74b61a59ed.
This commit is contained in:
Giorgio Gilestro 2026-05-27 23:55:59 +02:00
parent 74b61a59ed
commit f9d448d57b
2 changed files with 8 additions and 28 deletions
Download patch file Download diff file Expand all files Collapse all files

22
app/routers/api.py
View file

@ -21,10 +21,6 @@ from app.auth import require_token, maybe_current_user, CurrentUser
from app.services.i18n import ACTIVE_LANGUAGES
from app.config import get_settings
from app.db import get_session, utcnow
from app.logging import get_logger
log = get_logger("api_router")
from app.templates_env import templates
from app.models import (
Headline,
@ -316,14 +312,8 @@ async def _localized_content(
Returns None to signal 'use row.content as-is' (the default English
path)."""
if row is None or principal is None or principal.user is None:
log.info("i18n.log.skip", reason="no_row_or_principal",
row_id=getattr(row, "id", None),
has_principal=principal is not None,
has_user=(principal.user is not None) if principal else False)
return None
lang = (principal.user.lang or "en")
log.info("i18n.log.lookup", row_id=row.id, lang=lang,
user_id=principal.user.id)
if lang == "en":
return None
t = (await session.execute(
@ -331,9 +321,6 @@ async def _localized_content(
.where(StrategicLogTranslation.log_id == row.id)
.where(StrategicLogTranslation.lang == lang)
)).scalar_one_or_none()
log.info("i18n.log.result", row_id=row.id, lang=lang,
found=(t is not None),
content_preview=(t.content[:60] if t is not None else None))
return t.content if t is not None else None
@ -348,14 +335,8 @@ async def _apply_localized_summary(
for the lifetime of this GET request.
"""
if row is None or principal is None or principal.user is None:
log.info("i18n.summary.skip", reason="no_row_or_principal",
row_id=getattr(row, "id", None),
has_principal=principal is not None,
has_user=(principal.user is not None) if principal else False)
return
lang = (principal.user.lang or "en")
log.info("i18n.summary.lookup", row_id=row.id, lang=lang,
user_id=principal.user.id)
if lang == "en":
return
t = (await session.execute(
@ -363,9 +344,6 @@ async def _apply_localized_summary(
.where(IndicatorSummaryTranslation.summary_id == row.id)
.where(IndicatorSummaryTranslation.lang == lang)
)).scalar_one_or_none()
log.info("i18n.summary.result", row_id=row.id, lang=lang,
found=(t is not None),
content_preview=(t.content[:60] if t is not None else None))
if t is not None:
row.content = t.content

14
app/templates/base.html
View file

@ -143,11 +143,12 @@
};
window.cassandraSetLang = async function (newLang) {
console.log('[lang] click', newLang);
var pill = document.getElementById('lang-toggle');
if (!pill) { console.warn('[lang] no pill element'); return; }
if (!pill) return;
var prev = pill.dataset.lang;
if (prev === newLang) { console.log('[lang] already', newLang); return; }
if (prev === newLang) return;
// Optimistic update — flip the pill immediately so the click feels
// responsive. Revert on PATCH failure.
pill.dataset.lang = newLang;
try {
var r = await fetch('/api/settings/language', {
@ -156,17 +157,18 @@
credentials: 'same-origin',
body: JSON.stringify({lang: newLang}),
});
console.log('[lang] PATCH', r.status);
if (!r.ok) throw new Error('HTTP ' + r.status);
// Trigger HTMX-driven panels to re-fetch in the new language.
// Same shape as cassandraSetTone — every panel that listens to
// tone-changed also listens to lang-changed.
['#dash-header-container', '#log-panel .panel-body',
'#indicators-body', '#log-content'].forEach(function (sel) {
var el = document.querySelector(sel);
console.log('[lang] trigger', sel, 'found:', !!el, 'htmx:', !!window.htmx);
if (el && window.htmx) window.htmx.trigger(el, 'lang-changed');
});
} catch (e) {
pill.dataset.lang = prev;
console.warn('[lang] switch failed:', e);
console.warn('language switch failed:', e);
}
};
</script>