Merge pull request #113 from StefanSchuechl/patch-1

jesseleite · web-flow · commit df00be1b31c6 · 2020-08-25T09:52:52.000-04:00
Fix `parse_url()` path error on home page in Rules\ThreeSegmentUrls
diff --git a/src/Reporting/Rules/ThreeSegmentUrls.php b/src/Reporting/Rules/ThreeSegmentUrls.php
@@ -27,7 +27,7 @@ public function siteWarningComment()
 
     public function processPage()
     {
-        $url = parse_url($this->page->url())['path'];
+        $url = parse_url($this->page->url())['path'] ?? '/';
         $this->slashes = substr_count($url, '/');
     }
 

Original file line number	Diff line number	Diff line change
`@@ -27,7 +27,7 @@ public function siteWarningComment()`
`27`	`27`
`28`	`28`	`public function processPage()`
`29`	`29`	`{`
`30`		`- $url = parse_url($this->page->url())['path'];`
	`30`	`+ $url = parse_url($this->page->url())['path'] ?? '/';`
`31`	`31`	`$this->slashes = substr_count($url, '/');`
`32`	`32`	`}`
`33`	`33`