Changeset 24988 for trunk

Timestamp:

Oct 19, 2013, 11:21:01 AM (11 years ago)

Author:

mistic100

Message:

feature 2978: l10n() and {translate} with additional arguments
TODO: apply in core

Location:

trunk/include

Files:

• functions.inc.php (modified) (2 diffs)
• template.class.php (modified) (4 diffs)

trunk/include/functions.inc.php

@@ -901 +901 @@
 
 /**
- * returns the corresponding value from $lang if existing. Else, the key is
- * returned
- *
- * @param string key
+ * translation function
+ * returns the corresponding value from $lang if existing, else the key is returned
+ * if more than one parameter is provided sprintf is applied
+ * @param string $key
+ * @param mixed $args,... optional arguments
  * @return string
  */
@@ -915 +916 @@
     if ($conf['debug_l10n'] and !isset($lang[$key]) and !empty($key))
     {
-      trigger_error('[l10n] language key "'.$key.'" is not defined', E_USER_WARNING);
+      trigger_error('[l10n] language key "'. $key .'" not defined', E_USER_WARNING);
     }
     $val = $key;
   }
+
+  if (func_num_args() > 1)
+  {
+    $val = vsprintf($val, array_slice(func_get_args(), 1));
+  }
+
   return $val;
 }
 
 /**
- * returns the prinft value for strings including %d
+ * returns the printf value for strings including %d
  * return is concorded with decimal value (singular, plural)
  *

trunk/include/template.class.php

@@ -96 +96 @@
     $this->smarty->registerPlugin('modifiercompiler', 'translate_dec', array('Template', 'modcompiler_translate_dec') );
     $this->smarty->registerPlugin('modifier', 'explode', array('Template', 'mod_explode') );
-    $this->smarty->registerPlugin( 'modifier', 'get_extent', array($this, 'get_extent') );
+    $this->smarty->registerPlugin('modifier', 'get_extent', array($this, 'get_extent') );
     $this->smarty->registerPlugin('block', 'html_head', array($this, 'block_html_head') );
     $this->smarty->registerPlugin('block', 'html_style', array($this, 'block_html_style') );
@@ -488 +488 @@
   }
 
-  static function get_php_str_val($str)
+  private static function get_php_str_val($str)
   {
     if (is_string($str) && strlen($str)>1)
@@ -508 +508 @@
   {
     global $conf, $lang;
-    if ( $conf['compiled_template_cache_language']
-      && ($key=self::get_php_str_val($params[0])) !== null)
-    {
-      if (isset($lang[$key]))
+
+    switch (count($params))
+    {
+    case 1:
+      if ($conf['compiled_template_cache_language']
+        && ($key=self::get_php_str_val($params[0])) !== null
+        && isset($lang[$key])
+      ) {
         return var_export($lang[$key], true);
-    }
-    return 'l10n('.$params[0].')';
+      }
+      return 'l10n('.$params[0].')';
+      
+    default:
+      if ($conf['compiled_template_cache_language'])
+      {
+        $ret = 'sprintf(';
+        $ret .= self::modcompiler_translate( array($params[0]) );
+        $ret .= ','. implode(',', array_slice($params, 1));
+        $ret .= ')';
+        return $ret;
+      }
+      return 'l10n('.$params[0].','.implode(',', array_slice($params, 1)).')';
+    }
   }
 
@@ -520 +536 @@
   {
     global $conf, $lang, $lang_info;
-    if ( $conf['compiled_template_cache_language'])
+    if ($conf['compiled_template_cache_language'])
     {
       $ret = 'sprintf(';