[wp-trac] [WordPress Trac] #18180: Missing i18n in setup-config.php

WordPress Trac wp-trac at lists.automattic.com
Tue Aug 30 03:49:46 UTC 2011


#18180: Missing i18n in setup-config.php
-------------------------+-----------------------
 Reporter:  CoenJacobs   |       Owner:  nacin
     Type:  enhancement  |      Status:  reopened
 Priority:  normal       |   Milestone:  3.3
Component:  I18N         |     Version:  3.2.1
 Severity:  normal       |  Resolution:
 Keywords:  has-patch    |
-------------------------+-----------------------

Comment (by dd32):

 Duplicated some of your efforts SergeyBiryukov, but took all that on
 board, Patch still contains parts of #13839 and some dev stuff (as it's a
 work in progress).

 I've removed some translations of things that shouldn't be translated
 (wordpress & wp_ (the table prefix).

 The latest patch doesn't require any of setup-config.php to be translated
 manually at all, which is always a win.

 I investigated including is_rtl(), but that ultimately leads to requiring
 additional files which require !WordPress to be installed, the manual
 method is easier, I've also sync'd it all to work the same way as the
 !WordPress admin/wp-settings.php does when including and/or defining.

 The only other thing which has come to mind, Is do we want to have the
 installer (both setup config, and install.php) in the default language
 files? Or do we want to separate it like the MultiSite strings? But that
 can be changed later if need be.

 @nacin: Got any thoughts on this? Sound like the way forward to you? Am I
 missing anything here which affects transations?

-- 
Ticket URL: <http://core.trac.wordpress.org/ticket/18180#comment:11>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software


More information about the wp-trac mailing list