Fix warning "Warning: A non-numeric value encountered" in Model_Url (#933)

tmotyl · web-flow · commit 2056b4bc8d7f · 2020-04-23T18:48:53.000-04:00
Cast value to int and change isset to !empty to privent
url model to add empty string to an 1.
diff --git a/app/code/core/Mage/Catalog/Model/Url.php b/app/code/core/Mage/Catalog/Model/Url.php
@@ -676,7 +676,7 @@ public function getUnusedPathByUrlKey($storeId, $requestPath, $idPath, $urlKey)
                 $match['increment'] = $lastRequestPath;
             }
             return $match['prefix']
-                . (isset($match['increment']) ? ($match['increment'] + 1) : '1')
+                . (!empty($match['increment']) ? ((int)$match['increment'] + 1) : '1')
                 . $match['suffix'];
         }
         else {

Original file line number	Diff line number	Diff line change
`@@ -676,7 +676,7 @@ public function getUnusedPathByUrlKey($storeId, $requestPath, $idPath, $urlKey)`
`676`	`676`	`$match['increment'] = $lastRequestPath;`
`677`	`677`	`}`
`678`	`678`	`return $match['prefix']`
`679`		`- . (isset($match['increment']) ? ($match['increment'] + 1) : '1')`
	`679`	`+ . (!empty($match['increment']) ? ((int)$match['increment'] + 1) : '1')`
`680`	`680`	`. $match['suffix'];`
`681`	`681`	`}`
`682`	`682`	`else {`