Changeset 10584

Timestamp:

02/17/2009 05:03:29 AM (16 years ago)

Author:

ryan

Message:

Switch to pomo lib. Support gettext contexts. Deprecate long form functions. Props nbachiyski. fixes #9112 #9111

Location:

trunk

Files:

• wp-includes/l10n.php (modified) (12 diffs)
• wp-includes/pomo (added)
• wp-includes/pomo/entry.php (added)
• wp-includes/pomo/mo.php (added)
• wp-includes/pomo/po.php (added)
• wp-includes/pomo/streams.php (added)
• wp-includes/pomo/translations.php (added)
• wp-settings.php (modified) (1 diff)

trunk/wp-includes/l10n.php

@@ -45 +45 @@
 
 /**
- * Retrieve the translated text.
- *
- * 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.
- *
+ * Retrieves the translation of $text. If there is no translation, or
+ * the domain isn't loaded the original text is 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.
@@ -62 +57 @@
  * @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);
 }
 
@@ -80 +71 @@
 
 /**
- * Retrieve the translated text and strip context.
- *
- * 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.
+ * Translate $text like translate(), but assumes that the text
+ * contains a context after its last vertical bar.
  *
  * @since 2.5
@@ -100 +86 @@
 }
 
-/**
- * Retrieves the translated string from the translate().
+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 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()
@@ -110 +102 @@
  * @return string Translated text
  */
-function __($text, $domain = 'default') {
-    return translate($text, $domain);
+function __( $text, $domain = 'default' ) {
+    return translate( $text, $domain );
 }
 
@@ -123 +115 @@
  * @param string $domain Optional. Domain to retrieve the translated text
  */
-function _e($text, $domain = 'default') {
-    echo translate($text, $domain);
-}
-
-/**
- * Retrieve context translated string.
+function _e( $text, $domain = 'default' ) {
+    echo translate( $text, $domain );
+}
+
+/**
+ * Retrieve translated string with vertical bar context
  *
  * Quite a few times, there will be collisions with similar translatable text
@@ -148 +140 @@
 function _c($text, $domain = 'default') {
     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);
 }
 
@@ -172 +174 @@
  * @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;
-    }
-}
-
-/**
- * @see __ngettext() An alias of __ngettext
- *
- */
-function _n() {
-    $args = func_get_args();
-    return call_user_func_array('__ngettext', $args);
+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 );
 }
 
@@ -200 +186 @@
  */
 function _nc( $single, $plural, $number, $domain = 'default' ) {
-    return before_last_bar( __ngettext( $single, $plural, $number, $domain ) );
+    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 );
+}
+
+/**
+ * @deprecated Use _n_noop()
+ */
+function __ngettext_noop() {
+    _deprecated_function( __FUNCTION__, '2.8', '_n_noop()' );
+    $args = func_get_args();
+    return call_user_func_array('_n_noop', $args);
+
 }
 
@@ -211 +213 @@
  * 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);
-}
-
-/**
- * @see __ngettext_noop() An alias of __ngettext_noop()
- *
- */
-function _n_noop() {
-    $args = func_get_args();
-    return call_user_func_array('__ngettext_noop', $args);
+function _n_noop( $single, $plural, $number = 1, $domain = 'default' ) {
+    return array( $single, $plural );
 }
 
@@ -259 +252 @@
     global $l10n;
 
-    if ( is_readable($mofile))
-        $input = new CachedFileReader($mofile);
-    else
-        return;
-
-    $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 ( !is_readable($mofile)) return;
+    
+    $mo = new MO();
+    $mo->import_from_file( $mofile );
+
+    if (isset($l10n[$domain]))
+        $mo->merge_with( $l10n[$domain] );
+        
+    $l10n[$domain] = &$mo;
 }
 
@@ -341 +328 @@
 }
 
+/**
+ * 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;
+}
+
 ?>

trunk/wp-settings.php

@@ -274 +274 @@
 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');