Merge pull request #47 from jacekll/master

Fix error parsing JSON with apostrophe in data
Torann · Apr 25, 2019 · 4d43922 · 4d43922
2 parents 368a66e + 3623a79
commit 4d43922
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/src/Context.php b/src/Context.php
@@ -59,7 +59,7 @@ public function generate()
     {
         $properties = $this->getProperties();
 
-        return $properties ? "<script type=\"application/ld+json\">" . json_encode($properties,JSON_UNESCAPED_UNICODE) . "</script>" : '';
+        return $properties ? "<script type=\"application/ld+json\">" . json_encode($properties,JSON_HEX_APOS | JSON_UNESCAPED_UNICODE) . "</script>" : '';
     }
 
     /**

diff --git a/tests/ContextTypes/ProductTest.php b/tests/ContextTypes/ProductTest.php
@@ -13,7 +13,7 @@ class ProductTest extends TestCase
         'category' => 'Droppables / Anvils',
         'model' => 'Acme Exec. Anvil',
         'image' => 'http://www.example.com/anvil_executive.jpg',
-        'description' => 'Sleeker than ACME\'s Classic Anvil, the Executive Anvil is perfect for the business traveler looking for something to drop from a height.',
+        'description' => 'Sleeker than ACME\u0027s Classic Anvil, the Executive Anvil is perfect for the business traveler looking for something to drop from a height.',
         'sku' => '925872',
         'brand' => 'Acme Inc.',
         'aggregateRating' => [