handle non-string variables in php view engine + improve view engine …

…tests

src/ViewEngine/PHP.php

@@ -63,6 +63,10 @@ public function renderView(View $view)
 
         // escape HTML special characters
         foreach ($parameters as &$value) {
+            if (is_array($value) || is_object($value)) {
+                continue;
+            }
+
             $value = htmlspecialchars($value, ENT_QUOTES, 'UTF-8', false);
         }
 

tests/ViewEngine/MustacheTest.php

@@ -57,7 +57,10 @@ public function testRenderView()
 
         self::$engine->setGlobalParameters([
             'to' => 'should_be_overwritten',
-            'greeting' => 'Hello', ]);
+            'greeting' => 'Hello',
+            'object' => new stdClass(),
+            'array' => [],
+        ]);
 
         $this->assertEquals("Hello, world!\n<script>console.log("hello");</script>", self::$engine->renderView($view));
     }

tests/ViewEngine/PHPTest.php

@@ -47,7 +47,12 @@ public function testGlobalParameters()
 
     public function testRenderView()
     {
-        $view = new View('test', ['to' => 'world', 'escape' => '<script>console.log("hello");</script>']);
+        $view = new View('test', [
+            'to' => 'world',
+            'escape' => '<script>console.log("hello");</script>',
+            'object' => new stdClass(),
+            'array' => [],
+        ]);
 
         self::$engine->setGlobalParameters([
             'to' => 'should_be_overwritten',

tests/ViewEngine/SmartyTest.php

@@ -65,7 +65,12 @@ public function testGetSmarty()
 
     public function testRenderView()
     {
-        $view = new View('test', ['to' => 'world', 'escape' => '<script>console.log("hello");</script>']);
+        $view = new View('test', [
+            'to' => 'world',
+            'escape' => '<script>console.log("hello");</script>',
+            'object' => new stdClass(),
+            'array' => [],
+        ]);
 
         self::$engine->setGlobalParameters([
             'to' => 'should_be_overwritten',

tests/ViewEngine/TwigTest.php

@@ -53,7 +53,12 @@ public function testTwig()
 
     public function testRenderView()
     {
-        $view = new View('test', ['to' => 'world', 'escape' => '<script>console.log("hello");</script>']);
+        $view = new View('test', [
+            'to' => 'world',
+            'escape' => '<script>console.log("hello");</script>',
+            'object' => new stdClass(),
+            'array' => [],
+        ]);
 
         self::$engine->setGlobalParameters([
             'to' => 'should_be_overwritten',