Opened 12 years ago
Closed 12 years ago
#30012 closed defect (bug) (fixed)
Replace escaped translations with proper function
| Reported by: | obenland | Owned by: | SergeyBiryukov |
|---|---|---|---|
| Priority: | normal | Milestone: | 4.1 |
| Component: | I18N | Version: | |
| Severity: | normal | Keywords: | good-first-bug has-patch |
| Cc: | Focuses: |
Description
There are ten instances of esc_attr( __() ) across the WordPress codebase, as of revision 29927.
Replacing them with esc_attr__() would be a great opportunity for a beginning contributor to get their feet wet.
There is also one instance of esc_html( _x() ) that could be replaced with esc_html_x().
Attachments (2)
Change History (7)
#3
@
12 years ago
- Keywords has-patch added
- Owner set to
- Status new → accepted
wp-includes/script-loader.php can be included when esc_attr__() is not yet available, but it does not run it until did_action( 'init' ), so 30012.2.diff should work.
Note:
See TracTickets
for help on using tickets.
![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png)
Found a little more than 10 running trunk.