Fix issues reported by PHPCS

locomotivemtl · Nov 1, 2023 · 3250b22 · 3250b22
1 parent f167ff7
commit 3250b22
Show file tree

Hide file tree

Showing 3 changed files with 16 additions and 12 deletions.
diff --git a/src/Charcoal/Sitemap/Action/SitemapAction.php b/src/Charcoal/Sitemap/Action/SitemapAction.php
@@ -99,7 +99,7 @@ protected function addAlternateToXml(SimpleXMLElement $urlElement, array $altern
     {
         $alternateUrl = ltrim($alternate['url'], '/');
         if (parse_url($alternateUrl, PHP_URL_HOST) === null) {
-            $alternateUrl = $this->baseUrl.$alternateUrl;
+            $alternateUrl = $this->baseUrl . $alternateUrl;
         }
 
         if ($this->isExternalHost($alternateUrl)) {
@@ -137,7 +137,7 @@ protected function addLinkToXml(SimpleXMLElement $urlsetElement, array $link)
     {
         $linkUrl = ltrim($link['url'], '/');
         if (parse_url($linkUrl, PHP_URL_HOST) === null) {
-            $linkUrl = $this->baseUrl.$linkUrl;
+            $linkUrl = $this->baseUrl . $linkUrl;
         }
 
         if (!$this->isExternalHost($linkUrl)) {
@@ -173,13 +173,13 @@ protected function addLinkToXml(SimpleXMLElement $urlsetElement, array $link)
      */
     protected function createXmlEnvelope()
     {
-        $xml = '<?xml version="1.0" encoding="UTF-8"?>'
-              .'<urlset'
-              .' xmlns="'.$this->xmlNamespaces['xmlns'].'"'
-              .' xmlns:xhtml="'.$this->xmlNamespaces['xhtml'].'"'
-              .' xmlns:xsi="'.$this->xmlNamespaces['xsi'].'"'
-              .' xsi:schemaLocation="'.$this->xsiNamespaces['schemaLocation'].'"'
-              .'/>';
+        $xml  = '<?xml version="1.0" encoding="UTF-8"?>';
+        $xml .= '<urlset';
+        $xml .= ' xmlns="' . $this->xmlNamespaces['xmlns'] . '"';
+        $xml .= ' xmlns:xhtml="' . $this->xmlNamespaces['xhtml'] . '"';
+        $xml .= ' xmlns:xsi="' . $this->xmlNamespaces['xsi'] . '"';
+        $xml .= ' xsi:schemaLocation="' . $this->xsiNamespaces['schemaLocation'] . '"';
+        $xml .= '/>';
 
         return new SimpleXmlElement($xml);
     }

diff --git a/src/Charcoal/Sitemap/Service/SitemapPresenter.php b/src/Charcoal/Sitemap/Service/SitemapPresenter.php
@@ -91,7 +91,6 @@ public function transform($obj, $transformer = null)
             );
         }
 
-
         $that = $this;
 
         return $this->getCacheFacade()->get(
@@ -189,7 +188,9 @@ private function objectGet($obj, $propertyName)
             return $obj->{$propertyName};
         }
 
-        if (is_string($propertyName) && (is_array($obj) || $obj instanceof ArrayAccess) && (isset($obj[$propertyName]))) {
+        if (is_string($propertyName) &&
+            (is_array($obj) || $obj instanceof ArrayAccess) &&
+            isset($obj[$propertyName])) {
             return $obj[$propertyName];
         }
 

diff --git a/src/Charcoal/Sitemap/SitemapModule.php b/src/Charcoal/Sitemap/SitemapModule.php
@@ -49,7 +49,10 @@ private function setupPublicRoutes()
             RequestInterface $request,
             ResponseInterface $response,
             array $args = []
-        ) use ($config, $container) {
+        ) use (
+            $config,
+            $container
+        ) {
             $routeControllerClass = $this['route/controller/action/class'];
 
             $routeController = $container['route/factory']->create($routeControllerClass, [