Context Navigation

← Previous Ticket
Next Ticket →

#45191 closed defect (bug) (fixed)

Remove the 'default' textdomain on __() translation functions introduced into script-loader.php for 5.0

Reported by:	garrett-eclipse	Owned by:	pento
Priority:	normal	Milestone:	5.0
Component:	I18N	Version:	5.0
Severity:	normal	Keywords:	has-patch
Cc:		Focuses:	coding-standards

Description

Found in the 5.0 branch in the /wp-includes/script-loader.php is the use of 'default' as the textdomain for the () translation functions;
https://core.trac.wordpress.org/browser/branches/5.0/src/wp-includes/script-loader.php#L405-414

<?php
                                                        'future' => __( '%s from now', 'default' ),
                                                        /* translators: %s: duration */
                                                        'past'   => __( '%s ago', 'default' ),
                                                ),
                                        ),
                                        'formats'  => array(
                                                'time'                => get_option( 'time_format', __( 'g:i a', 'default' ) ),
                                                'date'                => get_option( 'date_format', __( 'F j, Y', 'default' ) ),
                                                'datetime'            => __( 'F j, Y g:i a', 'default' ),
                                                'datetimeAbbreviated' => __( 'M j, Y g:i a', 'default' ),

These should be removed as unnecessary.

Attachments (1)

45191.patch (1.3 KB ) - added by mukesh27 8 years ago.: Working Patch.

Download all attachments as: .zip

Change History (6)

@mukesh27
8 years ago

Attachment 45191.patch added

Working Patch.

#1 @mukesh27
8 years ago

Keywords has-patch added

#2 @garrett-eclipse
8 years ago

Thanks @mukesh27

#3 @SergeyBiryukov
8 years ago

Milestone Awaiting Review → 5.0

Related: #45162

#4 @pento
8 years ago

Owner set to pento
Resolution → fixed
Status new → closed

In 43831:

Script Loader: Add translator comments and remove textdomains.

The inline script for wp-date incorrectly included a textdomain, and lacked translator comments.

Props mukesh27.
Fixes #45191, #45162.

#5 @pento
8 years ago

In 44171:

Script Loader: Add translator comments and remove textdomains.

The inline script for wp-date incorrectly included a textdomain, and lacked translator comments.

Merges [43831] from the 5.0 branch to trunk.

Props mukesh27.
Fixes #45191, #45162.

Note: See TracTickets for help on using tickets.

Download in other formats:

Make WordPress Core

Context Navigation

#45191 closed defect (bug) (fixed)

Remove the 'default' textdomain on __() translation functions introduced into script-loader.php for 5.0

Description

Attachments (1)

Change History (6)

@mukesh27 8 years ago

#1 @mukesh27 8 years ago

#2 @garrett-eclipse 8 years ago

#3 @SergeyBiryukov 8 years ago

#4 @pento 8 years ago

#5 @pento 8 years ago