Changeset 3929:c871a4f0a22f for inc/admin/lib.dc.page.php

Timestamp:

11/13/18 15:42:09 (7 years ago)

Author:

franck <carnet.franck.paul@…>

Branch:

default

Message:

Switching from inline JS variables to JSON script. Confirm close is on the way

File:

• inc/admin/lib.dc.page.php (modified) (1 diff)

inc/admin/lib.dc.page.php

@@ -865 +865 @@
     public static function jsConfirmClose(...$args)
     {
-        if (count($args) > 0) {
-            foreach ($args as $k => $v) {
-                $args[$k] = "'" . html::escapeJS($v) . "'";
-            }
-            $params = implode(',', $args);
-        } else {
-            $params = '';
-        }
-
+        $js = [
+            'prompt' => __('You have unsaved changes.'),
+            'forms' => $args
+        ];
         return
-        self::jsLoad('js/confirm-close.js') .
-        '<script type="text/javascript">' . "\n" .
-        "confirmClosePage = new confirmClose(" . $params . "); " .
-        "confirmClose.prototype.prompt = '" . html::escapeJS(__('You have unsaved changes.')) . "'; " .
-            "</script>\n";
+        self::jsJson('confirm_close', $js) .
+        self::jsLoad('js/confirm-close.js');
     }