Ticket #9112: new-gettext-lib.diff

wp-includes/l10n.php

@@ -44 +44 @@
 }
 
 /**
- * Retrieve the translated text.
+ * Retrieves the translation of $text. If there is no translation, or
+ * the domain isn't loaded the original text is returned.
  *
- * If the domain is set in the $l10n global, then the text is run through the
- * domain's translate method. After it is passed to the 'gettext' filter hook,
- * along with the untranslated text as the second parameter.
- *
- * If the domain is not set, the $text is just returned.
- *
+ * @see __() Don't use translate() directly, use __()
  * @since 2.2.0
- * @uses $l10n Gets list of domain translated string (gettext_reader) objects.
  * @uses apply_filters() Calls 'gettext' on domain translated text
  *              with the untranslated text as second parameter.
  *
@@ -61 +56 @@
  * @param string $domain Domain to retrieve the translated text.
  * @return string Translated text
  */
-function translate($text, $domain = 'default') {
-        global $l10n;
-
-        if (isset($l10n[$domain]))
-                return apply_filters('gettext', $l10n[$domain]->translate($text), $text, $domain);
-        else
-                return apply_filters('gettext', $text, $text, $domain);
+function translate( $text, $domain = 'default' ) {
+        $translations = &get_translations_for_domain( $domain );
+        return apply_filters('gettext', $translations->translate($text), $text, $domain);
 }
 
 function before_last_bar( $string ) {
@@ -79 +70 @@
 }
 
 /**
- * Retrieve the translated text and strip context.
+ * Translate $text like translate(), but assumes that the text
+ * contains a context after its last vertical bar.
  *
- * If the domain is set in the $l10n global, then the text is run through the
- * domain's translate method. After it is passed to the 'gettext' filter hook,
- * along with the untranslated text as the second parameter.
- *
- * If the domain is not set, the $text is just returned.
- *
  * @since 2.5
  * @uses translate()
  *
@@ -99 +85 @@
 
 }
 
+function translate_with_gettext_context( $text, $context, $domain = 'default' ) {
+        $translations = &get_translations_for_domain( $domain );
+        return apply_filters( 'gettext_with_context', $translations->translate( $text, $context ), $text, $context, $domain);
+}
+
 /**
- * Retrieves the translated string from the translate().
+ * Retrieves the translation of $text. If there is no translation, or
+ * the domain isn't loaded the original text is returned.
  *
  * @see translate() An alias of translate()
  * @since 2.1.0
@@ -109 +101 @@
  * @param string $domain Optional. Domain to retrieve the translated text
  * @return string Translated text
  */
-function __($text, $domain = 'default') {
-        return translate($text, $domain);
+function __( $text, $domain = 'default' ) {
+        return translate( $text, $domain );
 }
 
 /**
@@ -122 +114 @@
  * @param string $text Text to translate
  * @param string $domain Optional. Domain to retrieve the translated text
  */
-function _e($text, $domain = 'default') {
-        echo translate($text, $domain);
+function _e( $text, $domain = 'default' ) {
+        echo translate( $text, $domain );
 }
 
 /**
- * Retrieve context translated string.
+ * Retrieve translated string with vertical bar context
  *
  * Quite a few times, there will be collisions with similar translatable text
  * found in more than two places but with different translated context.
@@ -149 +141 @@
         return translate_with_context($text, $domain);
 }
 
+function _x( $single, $context, $domain = 'default' ) {
+        return translate_with_gettext_context( $single, $context, $domain );
+}
+
+function __ngettext() {
+        _deprecated_function( __FUNCTION__, '2.8', '_n()' );
+        $args = func_get_args();
+        return call_user_func_array('_n', $args);
+}
+
 /**
  * Retrieve the plural or single form based on the amount.
  *
@@ -171 +173 @@
  * @param string $domain Optional. The domain identifier the text should be retrieved in
  * @return string Either $single or $plural translated text
  */
-function __ngettext($single, $plural, $number, $domain = 'default') {
-        global $l10n;
-
-        if (isset($l10n[$domain])) {
-                return apply_filters('ngettext', $l10n[$domain]->ngettext($single, $plural, $number), $single, $plural, $number);
-        } else {
-                if ($number != 1)
-                        return $plural;
-                else
-                        return $single;
-        }
+function _n($single, $plural, $number, $domain = 'default') {
+        $translations = &get_translations_for_domain( $domain );
+        $translation = $translations->translate_plural( $single, $plural, $number );
+        return apply_filters( 'ngettext', $translation, $single, $plural, $number );
 }
 
 /**
- * @see __ngettext() An alias of __ngettext
+ * @see _n() A version of _n(), which supports contexts --
+ * strips everything from the translation after the last bar
  *
  */
-function _n() {
-        $args = func_get_args();
-        return call_user_func_array('__ngettext', $args);
+function _nc( $single, $plural, $number, $domain = 'default' ) {
+        return before_last_bar( _n( $single, $plural, $number, $domain ) );
 }
 
+function _nx($single, $plural, $number, $context, $domain = 'default') {
+        $translations = &get_translations_for_domain( $domain );
+        $translation = $translations->translate_plural( $single, $plural, $number, $context );
+        return apply_filters( 'ngettext_with_context ', $translation, $single, $plural, $number, $context );
+}
+
 /**
- * @see _n() A version of _n(), which supports contexts --
- * strips everything from the translation after the last bar
- *
+ * @deprecated Use _n_noop()
  */
-function _nc( $single, $plural, $number, $domain = 'default' ) {
-        return before_last_bar( __ngettext( $single, $plural, $number, $domain ) );
+function __ngettext_noop() {
+        _deprecated_function( __FUNCTION__, '2.8', '_n_noop()' );
+        $args = func_get_args();
+        return call_user_func_array('_n_noop', $args);
+
 }
 
 /**
@@ -210 +212 @@
  *
  * Example:
  *  $messages = array(
- *      'post' => ngettext_noop('%s post', '%s posts'),
- *      'page' => ngettext_noop('%s pages', '%s pages')
+ *      'post' => _n_noop('%s post', '%s posts'),
+ *      'page' => _n_noop('%s pages', '%s pages')
  *  );
  *  ...
  *  $message = $messages[$type];
- *  $usable_text = sprintf(__ngettext($message[0], $message[1], $count), $count);
+ *  $usable_text = sprintf(_n($message[0], $message[1], $count), $count);
  *
  * @since 2.5
  * @param $single Single form to be i18ned
  * @param $plural Plural form to be i18ned
- * @param $number Not used, here for compatibility with __ngettext, optional
- * @param $domain Not used, here for compatibility with __ngettext, optional
+ * @param $number Not used, here for compatibility with _n, optional
+ * @param $domain Not used, here for compatibility with _n, optional
  * @return array array($single, $plural)
  */
-function __ngettext_noop($single, $plural, $number=1, $domain = 'default') {
-        return array($single, $plural);
+function _n_noop( $single, $plural, $number = 1, $domain = 'default' ) {
+        return array( $single, $plural );
 }
 
 /**
- * @see __ngettext_noop() An alias of __ngettext_noop()
- *
- */
-function _n_noop() {
-        $args = func_get_args();
-        return call_user_func_array('__ngettext_noop', $args);
-}
-
-/**
  * Loads MO file into the list of domains.
  *
  * If the domain already exists, the inclusion will fail. If the MO file is not
@@ -258 +251 @@
 function load_textdomain($domain, $mofile) {
         global $l10n;
 
-        if ( is_readable($mofile))
-                $input = new CachedFileReader($mofile);
-        else
-                return;
+        if ( !is_readable($mofile)) return;
+        
+        $mo = new MO();
+        $mo->import_from_file( $mofile );
 
-        $gettext = new gettext_reader($input);
-
-        if (isset($l10n[$domain])) {
-                $l10n[$domain]->load_tables();
-                $gettext->load_tables();
-                $l10n[$domain]->cache_translations = array_merge($gettext->cache_translations, $l10n[$domain]->cache_translations);
-        } else
-                $l10n[$domain] = $gettext;
-
-        unset($input, $gettext);
+        if (isset($l10n[$domain]))
+                $mo->merge_with( $l10n[$domain] );
+                
+        $l10n[$domain] = &$mo;
 }
 
 /**
@@ -340 +327 @@
         load_textdomain($domain, $mofile);
 }
 
+/**
+ * Returns the Translations instance for a domain. If there isn't one,
+ * returns empty Translations instance.
+ * 
+ * @param string $domain
+ * @return object A Translation instance
+ */
+function get_translations_for_domain( $domain ) {
+        global $l10n;
+        $empty = &new Translations;
+        return isset($l10n[$domain])? $l10n[$domain] : $empty;
+}
+
 ?>

wp-settings.php

@@ -273 +273 @@
 
 require (ABSPATH . WPINC . '/plugin.php');
 require (ABSPATH . WPINC . '/default-filters.php');
-include_once(ABSPATH . WPINC . '/streams.php');
-include_once(ABSPATH . WPINC . '/gettext.php');
+include_once(ABSPATH . WPINC . '/pomo/mo.php');
 require_once (ABSPATH . WPINC . '/l10n.php');
 
 if ( !is_blog_installed() && (strpos($_SERVER['PHP_SELF'], 'install.php') === false && !defined('WP_INSTALLING')) ) {