Changeset 3872:39861f53bcc0 for inc/admin/lib.dc.page.php

Timestamp:

09/12/18 14:45:17 (7 years ago)

Author:

franck <carnet.franck.paul@…>

Branch:

default

Message:

Use ... token for variable argument list rather than func_get_args() whenever is possible

File:

• inc/admin/lib.dc.page.php (modified) (4 diffs)

inc/admin/lib.dc.page.php

@@ -619 +619 @@
     }
 
-    public static function helpBlock()
+    public static function helpBlock(...$params)
     {
         global $core;
@@ -627 +627 @@
         }
 
-        $args = func_get_args();
-        $args = new ArrayObject($args);
+        $args = new ArrayObject($params);
 
         # --BEHAVIOR-- adminPageHelpBlock
@@ -914 +913 @@
     }
 
-    public static function jsConfirmClose()
-    {
-        $args = func_get_args();
+    public static function jsConfirmClose(...$args)
+    {
         if (count($args) > 0) {
             foreach ($args as $k => $v) {
                 $args[$k] = "'" . html::escapeJS($v) . "'";
             }
-            $args = implode(',', $args);
+            $params = implode(',', $args);
         } else {
-            $args = '';
+            $params = '';
         }
 
@@ -929 +927 @@
         self::jsLoad('js/confirm-close.js') .
         '<script type="text/javascript">' . "\n" .
-        "confirmClosePage = new confirmClose(" . $args . "); " .
+        "confirmClosePage = new confirmClose(" . $params . "); " .
         "confirmClose.prototype.prompt = '" . html::escapeJS(__('You have unsaved changes.')) . "'; " .
             "</script>\n";