Skip to content

Show the do-not-translate and broken XML statuses - #68

Open
lacatoire wants to merge 1 commit into
php:masterfrom
lacatoire:revcheck/dnt-broken-sections
Open

Show the do-not-translate and broken XML statuses#68
lacatoire wants to merge 1 commit into
php:masterfrom
lacatoire:revcheck/dnt-broken-sections

Conversation

@lacatoire

@lacatoire lacatoire commented Aug 4, 2026

Copy link
Copy Markdown
Member

Front end for the two statuses added in php/doc-base#334.

Once that one is deployed, files carrying the new statuses land in the files table and count in the denominator of get_lang_stats(), but show up in no row of the file summary. The percentages then stop adding up to 100, with nothing on the page saying why. Until it is deployed this PR is inert: the queries just return nothing.

It adds two pages, p=brokenfiles and p=donottranslate, built on the same shape as the existing misstags and oldfiles pages, plus their menu entries and the two statuses in $TRANSLATION_STATUSES.

Files marked do not translate are listed but left out of the completion totals, to match scripts/revcheck.php and genrevdb.php in doc-base: a translation should not see its rate drop for files it is not expected to have. Their share of a total they are not part of shows as n/a. Other statuses are displayed as before. If that is the wrong call on this side, it is a small change to make.

Checked locally with the built-in server, against a database built from doc-fr and doc-ru.

The file summary gains the two rows:

summary-ru

Broken XML lists chmonly/search.xml with its parse error:

broken-ru

Do not translate lists the marked files:

dnt-fr

Two pages following the shape of the existing ones, their menu
entries, and the two statuses in $TRANSLATION_STATUSES.

Files marked do not translate are kept out of the completion totals,
matching doc-base, so no translation sees its rate drop for files it
is not expected to have. Their share of a total they are not part of
shows as n/a. Other statuses are displayed as before.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant