#42454 closed defect (bug) (fixed)
Unable to translate codex URL in theme-editor.php
Reported by: | odysseygate | Owned by: | dd32 |
---|---|---|---|
Milestone: | 4.9.1 | Priority: | normal |
Severity: | normal | Version: | 4.9 |
Component: | Text Changes | Keywords: | has-patch |
Focuses: | Cc: |
Description
the string "https://codex.wordpress.org/Child_Themes" exists, but it cant be translated 'cause of missing translate function.
Attachments (2)
Change History (11)
#1
@
7 years ago
- Keywords has-patch needs-refresh i18n-change added
- Milestone changed from Awaiting Review to 4.9.1
Thanks for reporting @odysseygate!
You're right, this URL should probably be translated, however the patch should probably be esc_url( __() )
instead (That can be changed upon commit if needed)
This would be an additional string, the Child_Themes
URL isn't translated anywhere else.
Marking as 4.9.1, although this is small enough to slip into 4.9 if so desired ( @melchoyce ) -
#2
@
7 years ago
@dd32 OK, i understand 4.9 is about to release soon, so ill leave it up to you.
The Child_Themes
URL has already translated on help tab in the same theme-editor.php. see https://core.trac.wordpress.org/browser/trunk/src/wp-admin/theme-editor.php?marks=39#L39.
#7
@
7 years ago
- Owner set to dd32
- Resolution set to fixed
- Status changed from new to closed
In 42247:
Fixed the function from escape to translate.