Ticket #10971: l10n-translation-noop-performance.patch

wp-includes/l10n.php

@@ -55 +55 @@
  * @return string Translated text
  */
 function translate( $text, $domain = 'default' ) {
-        $translations = &get_translations_for_domain( $domain );
-        return apply_filters('gettext', $translations->translate($text), $text, $domain);
+        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 before_last_bar( $string ) {
@@ -84 +88 @@
 }
 
 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);
+        global $l10n;
+
+        if (isset($l10n[$domain]))
+                return apply_filters('gettext_with_context', $l10n[$domain]->translate($text, $context), $text, $context, $domain);
+        else
+                return apply_filters('gettext_with_context', $text, $text, $context, $domain);
 }
 
 /**
@@ -236 +244 @@
  * @return string Either $single or $plural translated text
  */
 function _n($single, $plural, $number, $domain = 'default') {
-        $translations = &get_translations_for_domain( $domain );
-        $translation = $translations->translate_plural( $single, $plural, $number );
+        global $l10n;
+
+        if ( isset($l10n[$domain]) )
+                $translation = $l10n[$domain]->translate_plural( $single, $plural, $number );
+        else
+                $translation = 1 == $number ? $single : $plural;
+
         return apply_filters( 'ngettext', $translation, $single, $plural, $number, $domain );
 }
 
@@ -251 +264 @@
 }
 
 function _nx($single, $plural, $number, $context, $domain = 'default') {
-        $translations = &get_translations_for_domain( $domain );
-        $translation = $translations->translate_plural( $single, $plural, $number, $context );
+        global $l10n;
+
+        if ( isset($l10n[$domain]) )
+                $translation = $l10n[$domain]->translate_plural( $single, $plural, $number, $context );
+        else
+                $translation = 1 == $number ? $single : $plural;
+
         return apply_filters( 'ngettext_with_context', $translation, $single, $plural, $number, $context, $domain );
 }
 
@@ -396 +414 @@
 }
 
 /**
- * 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;
-        if ( isset($l10n[$domain]) )
-                return $l10n[$domain];
-        else
-                return $empty;
-}
-
-/**
  * Translates role name. Since the role names are in the database and
  * not in the source there are dummy gettext calls to get them into the POT
  * file and this function properly translates them back.