[wp-trac] Re: [WordPress Trac] #3474: misc i18n improvements

WordPress Trac wp-trac at lists.automattic.com
Thu Dec 21 22:23:01 GMT 2006


#3474: misc i18n improvements
--------------------------+-------------------------------------------------
 Reporter:  nbachiyski    |        Owner:  anonymous
     Type:  defect        |       Status:  reopened 
 Priority:  low           |    Milestone:  2.1      
Component:  General       |      Version:           
 Severity:  normal        |   Resolution:           
 Keywords:  bg|has-patch  |  
--------------------------+-------------------------------------------------
Comment (by markjaquith):

 The error fix looks good, so long as it doesn't reduce translators'
 freedom.  Are there any cases where that portion would be placed AFTER the
 error message?

 Line 254 of the patch needs to be:

 {{{
 wp_die( __('You are not allowed to edit posts as this user.' ));
 }}}

 (forgot the "to")

 I don't think we should change the name of {{{__ngettext()}}}:

  * plugins may be using {{{__ngettext()}}}
  * translators are used to setting their {{{.pot}}} creation software to
 look for {{{__ngettext}}}
  * It matches PHP's {{{ngettext()}}} function

 Not sure about removing wrapping parenthesis in translations.

 I like the "you cannot FOO" » "you are not allowed to FOO" change.
 Makes it clear that it's a permissions issue, not an error.

 I'm going to try to commit some portions of this.

-- 
Ticket URL: <http://trac.wordpress.org/ticket/3474#comment:3>
WordPress Trac <http://trac.wordpress.org/>
WordPress blogging software


More information about the wp-trac mailing list