Simplify Xml::encodeJsCall
authorjeroendedauw <jeroendedauw@gmail.com>
Sat, 22 Dec 2012 00:01:27 +0000 (01:01 +0100)
committerjeroendedauw <jeroendedauw@gmail.com>
Sat, 22 Dec 2012 00:05:22 +0000 (01:05 +0100)
Change-Id: Idf2838a1b32170ca6336eb4a67f16cd3335218f9

includes/Xml.php

index 67f3658..d5e9189 100644 (file)
@@ -682,22 +682,11 @@ class Xml {
         * @return string
         */
        public static function encodeJsCall( $name, $args ) {
-               $s = "$name(";
-               $first = true;
-
-               foreach ( $args as $arg ) {
-                       if ( $first ) {
-                               $first = false;
-                       } else {
-                               $s .= ', ';
-                       }
-
-                       $s .= Xml::encodeJsVar( $arg );
+               foreach ( $args as &$arg ) {
+                       $arg = Xml::encodeJsVar( $arg );
                }
 
-               $s .= ");\n";
-
-               return $s;
+               return "$name(" . implode( ', ', $args ) . ");\n";
        }
 
        /**