[VarDumper] Tweak display options implementation

nicolas-grekas · nicolas-grekas · commit bfb9087cd2bd · 2016-06-02T10:53:06.000+02:00
diff --git a/src/Symfony/Component/VarDumper/Dumper/HtmlDumper.php b/src/Symfony/Component/VarDumper/Dumper/HtmlDumper.php
@@ -46,13 +46,12 @@ class HtmlDumper extends CliDumper
         'str .max-string-length b' => 'color:#A0A0A0',
     );
 
-    protected $displayOptions = array(
+    private $displayOptions = array(
         'initDepth' => 1,
         'maxDepth' => 1,
         'maxStringLength' => 160,
     );
-
-    protected $displayOptionsIsUpdated = false;
+    private $extraDisplayOptions = array();
 
     /**
      * {@inheritdoc}
@@ -87,15 +86,12 @@ public function setStyles(array $styles)
     /**
      * Configures display options.
      *
-     * @param array $displayOptions A map of displayOptions names to customize the behavior.
+     * @param array $displayOptions A map of display options to customize the behavior.
      */
     public function setDisplayOptions(array $displayOptions)
     {
-        if ($displayOptions)
-        {
-            $this->displayOptionsIsUpdated = true;
-            $this->displayOptions = $displayOptions + $this->displayOptions;
-        }
+        $this->headerIsDumped = false;
+        $this->displayOptions = $displayOptions + $this->displayOptions;
     }
 
     /**
@@ -123,9 +119,9 @@ public function setDumpBoundaries($prefix, $suffix)
     /**
      * {@inheritdoc}
      */
-    public function dump(Data $data, $output = null, array $displayOptions = [])
+    public function dump(Data $data, $output = null, array $extraDisplayOptions = array())
     {
-        $this->setDisplayOptions($displayOptions);
+        $this->extraDisplayOptions = $extraDisplayOptions;
         parent::dump($data, $output);
         $this->dumpId = 'sf-dump-'.mt_rand();
     }
@@ -141,7 +137,8 @@ protected function getDumpHeader()
             return $this->dumpHeader;
         }
 
-        $line = <<<'EOHTML'
+        $line = json_encode($this->displayOptions, JSON_FORCE_OBJECT);
+        $line = <<<EOHTML
 <script>
 Sfdump = window.Sfdump || (function (doc) {
 
@@ -218,13 +215,9 @@ function getLevelNodeForParent(parentNode, currentNode, level) {
     return getLevelNodeForParent(parentNode, currentNode, level);
 }
 
-return function (root, options) {
+return function (root, extraOptions) {
     root = doc.getElementById(root);
-EOHTML;
 
-    $line .= 'options = options || '.json_encode($this->displayOptions).';';
-
-    $line .= <<<'EOHTML'
     function a(e, f) {
         addEventListener(root, e, function (e) {
             if ('A' == e.target.tagName) {
@@ -285,6 +278,7 @@ function isCtrlKey(e) {
     });
 
     var indentRx = new RegExp('^('+(root.getAttribute('data-indent-pad') || '  ').replace(rxEsc, '\\$1')+')+', 'm'),
+        options = {$line},
         elt = root.getElementsByTagName('A'),
         len = elt.length,
         i = 0,
@@ -293,6 +287,10 @@ function isCtrlKey(e) {
 
     while (i < len) t.push(elt[i++]);
 
+    for (i in extraOptions) {
+        options[i] = $extraOptions[i];
+    }
+
     elt = root.getElementsByTagName('SAMP');
     len = elt.length;
     i = 0;
@@ -378,7 +376,7 @@ function toggleMaxStringLength(elt) {
 
             if (elt.className == 'max-string-length expanded') {
                 elt.className = 'max-string-length collapsed';
-            }else{
+            } else {
                 elt.className = 'max-string-length expanded';
             }
         }
@@ -536,7 +534,7 @@ protected function dumpLine($depth, $endOfValue = false)
             $this->line .= sprintf(
                 $this->dumpSuffix,
                 $this->dumpId,
-                $this->displayOptionsIsUpdated ? ','.json_encode($this->displayOptions) : ''
+                $this->extraDisplayOptions ? ','.json_encode($this->extraDisplayOptions) : ''
             );
         }
         $this->lastDepth = $depth;
