Fix JsonLd default url (#261)

* Fix JsonLd default url The docs state that when providing `null` as the default JsonLd URL `URL::current()` is used. But actually `URL::full()` was used.
artesaos · Feb 6, 2022 · bd2b4da · bd2b4da
1 parent 2190dc4
commit bd2b4da
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 2 deletions.
diff --git a/src/SEOTools/JsonLd.php b/src/SEOTools/JsonLd.php
@@ -117,7 +117,13 @@ public function convertToArray(): array
         }
 
         if ($this->url !== false) {
-            $generated['url'] = $this->url ?? app('url')->full();
+            if ($this->url === null || $this->url === 'full') {
+                $generated['url'] = app('url')->full();
+            } elseif ($this->url === 'current') {
+                $generated['url'] = app('url')->current();
+            } else {
+                $generated['url'] = $this->url;
+            }
         }
 
         if (!empty($this->images)) {

diff --git a/src/resources/config/seotools.php b/src/resources/config/seotools.php
@@ -60,7 +60,7 @@
         'defaults' => [
             'title'       => 'Over 9000 Thousand!', // set false to total remove
             'description' => 'For those who helped create the Genki Dama', // set false to total remove
-            'url'         => false, // Set null for using Url::current(), set false to total remove
+            'url'         => false, // Set to null or 'full' to use Url::full(), set to 'current' to use Url::current(), set false to total remove
             'type'        => 'WebPage',
             'images'      => [],
         ],