[3.0] Use the keys these language lookups meant - #9399
Merged
Sesquipedalian merged 1 commit intoAug 14, 2026
Conversation
Four lookups name a key no language file defines, so getTxt() returns an
empty string.
The theme file browser asks for size_kilobytes and size_bytes. Everywhere
else in SMF the keys are singular, size_kilobyte and size_byte, so the size
column in Admin -> Themes -> Modify -> browse files was blank for every
file. The arguments went through Lang::numberFormat() first as well, which
hands an already formatted string to a {0, number} placeholder; the other
callers pass the number and let the string format it.
The installer and upgrader ask for done and failed while reporting whether
chmod through FTP worked. Maintenance.php calls those log_done and
log_failed.
Signed-off-by: Mathias Papenbrock <mathiaspapealbert@hotmail.com>
Signed-off-by: albertlast <mathiaspapealbert@hotmail.com>
This was referenced Aug 8, 2026
Closed
Sesquipedalian
approved these changes
Aug 14, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
Four lookups name a key that no language file defines, so
Lang::getTxt()returns an empty string.The theme file browser has no size column
Sources/Actions/Admin/Themes.phpasks forsize_kilobytesandsize_bytes. Everywhere else in SMF —Attachment.php,Feed.php,Actions/Admin/Search.php,ManageAttachments.template.php, and about a dozen more — the keys are singular:size_kilobyteandsize_byte. So Admin → Themes → Modify → browse files showed nothing in the size column, for every file. Straight off the running forum:The arguments needed a second look too. Both went through
Lang::numberFormat()first, which hands an already formatted string like32.76to a{0, number}placeholder. The other callers pass the number and let the string format it, so that is what these do now.The installer's FTP log says nothing
Sources/Maintenance/Tools/ToolsBase.phpreports whether achmodthrough FTP worked withgetTxt('done')andgetTxt('failed').Languages/en_US/Maintenance.phpcalls thoselog_done('done.') andlog_failed('failed.'), and has since the file was written. I have not staged a failing FTP chmod to watch it, so this one is by inspection.One I left alone
Themes/default/ReportedContent.template.phpasks formc_recent_member_reports, which is also undefined, intemplate_reported_members_block(). That function is dead: the moderation home registersreported_users_block, whichModerationCenter.template.phpprovides, andReportedContent.template.phpalso carries a secondtemplate_reported_posts_block()that duplicates the one inModerationCenter.template.php. Deleting dead template functions is a bigger call than this PR, so it is only flagged here.Found while splitting #7933, with the same sweep as #9397 (which fixes calls that name the wrong language file) and #9398 (which fixes fallbacks that cannot fire). Not part of the theme, so it goes straight to
release-3.0.Issues References (Fixes|Related|Closes)
Related to #7933