Fix HTML special chars in JSON renderer should be escaped

Signed-off-by: Jack Cherng <[email protected]>

Author: jfcherng

src/Renderer/Html/AbstractHtml.php

@@ -186,11 +186,6 @@ protected function getDefaultBlock(string $tag, int $i1, int $j1): array
      */
     protected function formatLines(array $lines): array
     {
-        // for example, the "Json" template does not need these
-        if (static::IS_TEXT_TEMPLATE) {
-            return $lines;
-        }
-
         // glue all lines into a single string to get rid of multiple function calls later
         // unnecessary, but should improve performance if there are many lines
         $string = \implode(RendererConstant::IMPLODE_DELIMITER, $lines);

src/Renderer/Html/Json.php

@@ -4,6 +4,8 @@
 
 namespace Jfcherng\Diff\Renderer\Html;
 
+use Jfcherng\Diff\Renderer\RendererConstant;
+
 /**
  * Json diff generator.
  */
@@ -36,4 +38,19 @@ public function render(): string
             \JSON_UNESCAPED_UNICODE | \JSON_UNESCAPED_SLASHES
         );
     }
+
+    /**
+     * {@inheritdoc}
+     */
+    protected function formatLines(array $lines): array
+    {
+        // glue all lines into a single string to get rid of multiple function calls later
+        // unnecessary, but should improve performance if there are many lines
+        $string = \implode(RendererConstant::IMPLODE_DELIMITER, $lines);
+
+        $string = $this->htmlSafe($string);
+
+        // split the string back to lines
+        return \explode(RendererConstant::IMPLODE_DELIMITER, $string);
+    }
 }