samsoncms / seo

application/Application.php

Unused Use Statements -2 removed lines

@@ -8,8 +8,6 @@
 
 namespace samsoncms\seo\application;
 
-use samsonphp\event\Event;
-
 if (class_exists('\samsoncms\Application', false)) {
 
     class Application extends \samsoncms\Application

src/Core.php

Unused Use Statements -5 removed lines

@@ -2,19 +2,14 @@
 
 namespace samsoncms\seo;
 
-use samson\activerecord\dbRecord;
 use samson\activerecord\structure;
 use samson\activerecord\structurematerial;
-use samson\cms\CMSMaterial;
 use samsoncms\seo\Migrate;
 use samsoncms\seo\render\Element;
 use samsoncms\seo\schema\control\seo\Dynamic;
-use samsoncms\seo\schema\material\Facebook;
-use samsoncms\seo\schema\Main;
 use samsoncms\seo\schema\Schema;
 use samson\core\CompressableService;
 use samsoncms\seo\sitemap\SiteMap;
-use samsoncms\seo\sitemap\Xml;
 use samsoncms\seo\tab\InfoTab;
 use samsonphp\event\Event;
 

Spacing +6 added lines, -6 removed lines

@@ -72,7 +72,7 @@ 
             if ($schema->getStructureId() == $structureId) {
 
                 // Render elements if exists
-                $isElements = isset($schema->elements)&&(!empty($schema->elements));
+                $isElements = isset($schema->elements) && (!empty($schema->elements));
 
                 // If exists elements on the tab then show their
                 if ($isElements) {
@@ -86,7 +86,7 @@ 
                     // Insert element as first child of table
                     $content = preg_replace(
                         '/\<div class=\"material_table_tab\"\>/',
-                        '<div class="material_table_tab">' . $contentNestedElement,
+                        '<div class="material_table_tab">'.$contentNestedElement,
                         $content,
                         1
                     );
@@ -153,7 +153,7 @@ 
      * @param $query
      * @param $entity
      */
-    public function renderMaterialTab(\samsoncms\app\material\form\Form &$form, $renderer, $query, $entity)
+    public function renderMaterialTab(\samsoncms\app\material\form\Form&$form, $renderer, $query, $entity)
     {
 
         $migrate = new \samsoncms\seo\Migrate($this->query);
@@ -249,7 +249,7 @@ 
                 // If content not empty render field(meta tag)
                 if (isset($content{0})) {
                     // Save html view
-                    $html .= $this->view($schema->view)->name($alias)->content($content)->output() . "\n";
+                    $html .= $this->view($schema->view)->name($alias)->content($content)->output()."\n";
                 }
             }
         }
@@ -300,9 +300,9 @@ 
     public function templateRenderer(&$html, $parameters, $module)
     {
         // TODO: Change this to normal dependency
-        if($module->id() != 'compressor') {
+        if ($module->id() != 'compressor') {
             $content = $this->show();
-            $html = str_ireplace('</head>', $content . '</head>', $html);
+            $html = str_ireplace('</head>', $content.'</head>', $html);
         }
     }
 }

src/Display.php

Doc Comments +5 added lines, -5 removed lines

@@ -20,7 +20,7 @@ 
 
     /**
      * Init query
-     * @param $query
+     * @param \samson\activerecord\dbQuery $query
      */
     public function __construct($query)
     {
@@ -29,7 +29,7 @@ 
 
     /**
      * Get data from passed material
-     * @param $fieldName
+     * @param string $fieldName
      * @param $material
      * @return null
      * @throws \Exception
@@ -108,7 +108,7 @@ 
      * @param $schema
      * @param $fieldName
      * @param $material
-     * @return null
+     * @return string|null
      */
     public function findField($schema, $fieldName, $material)
     {
@@ -138,7 +138,7 @@ 
 
     /**
      * Get material by url
-     * @param $url
+     * @param string $url
      * @return mixed
      */
     public function getMaterialByUrl($url)
@@ -187,7 +187,7 @@ 
 
     /**
      * Get all views which not assign to any material
-     * @param $renderer
+     * @param Core $renderer
      * @return String
      */
     public function getCommonViews($renderer)

Unused Use Statements -1 removed lines

@@ -8,7 +8,6 @@
 
 namespace samsoncms\seo;
 
-use samsoncms\seo\schema\control\seo\Dynamic;
 use samsoncms\seo\schema\material\Publisher;
 use samsoncms\seo\schema\Schema;
 

Spacing +4 added lines, -4 removed lines

@@ -65,7 +65,7 @@ 
     {
 
         // Set field name with prefix of schema
-        $fieldNameFull = $fieldName . '_' . $schema->id;
+        $fieldNameFull = $fieldName.'_'.$schema->id;
         $fieldValue = $this->getDataField($fieldNameFull, $material);
 
         // If in the current schema need value wasn't found then find in sibling schemas
@@ -80,7 +80,7 @@ 
                 }
 
                 // Set field name with prefix of schema
-                $fieldNameFull = $fieldName . '_' . $schemaFind->id;
+                $fieldNameFull = $fieldName.'_'.$schemaFind->id;
 
                 // If it is deep search i.e search in parent structures then change current material to parent
                 if ($deep == true) {
@@ -204,11 +204,11 @@ 
             foreach ($schema->relations as $fieldName => $alias) {
 
                 // Get value
-                $content = trim($material[$fieldName . '_' . $schema->id]);
+                $content = trim($material[$fieldName.'_'.$schema->id]);
 
                 if (isset($content{0})) {
                     // Render
-                    $html .= $renderer->view($schema->view)->name($alias)->content($content)->output() . "\n";
+                    $html .= $renderer->view($schema->view)->name($alias)->content($content)->output()."\n";
                 }
             }
         }

src/sitemap/SiteMap.php

Doc Comments +5 added lines, -2 removed lines

@@ -113,7 +113,7 @@ 
 
     /**
      * @param $params
-     * @param $xml
+     * @param Xml $xml
      * @return int
      * @throws \Exception
      */
@@ -148,7 +148,7 @@ 
 
     /**
      * @param $params
-     * @param $xml
+     * @param Xml $xml
      * @return int
      * @throws \Exception
      */
@@ -276,6 +276,9 @@ 
         }
     }
 
+    /**
+     * @param ControlSchema $controlSchema
+     */
     public function getRelatedStructuresInMaterial($controlSchema, $mainMaterial)
     {
 

Spacing +5 added lines, -5 removed lines

@@ -101,7 +101,7 @@ 
             $mainSiteMapContent = $xml->generateIndexSiteMap($mainParams, $this->filePrefix);
 
             // Save
-            $xml->saveXmlToFile($this->mainMapName . '.xml', $mainSiteMapContent);
+            $xml->saveXmlToFile($this->mainMapName.'.xml', $mainSiteMapContent);
         }
 
         $time_elapsed_secs = microtime(true) - $start;
@@ -141,7 +141,7 @@ 
         $fileName = 'static.xml';
 
         // Create site map file for current category
-        $xml->saveXmlToFile($this->filePrefix . $fileName, $result);
+        $xml->saveXmlToFile($this->filePrefix.$fileName, $result);
 
         return $count;
     }
@@ -178,16 +178,16 @@ 
                 $fileName = preg_replace('/^\//', '', $link);
 
                 // Exchange all slash to dash
-                $fileName = preg_replace('/\//', '-', $fileName) . '.xml';
+                $fileName = preg_replace('/\//', '-', $fileName).'.xml';
 
                 // Create site map file for current category
-                $xml->saveXmlToFile($this->filePrefix . $fileName, $result);
+                $xml->saveXmlToFile($this->filePrefix.$fileName, $result);
 
                 $links[] = $param;
             }
 
             // If need recursively execute structures
-            if ((isset($param['__SEO_IsRecursive']))&&($param['__SEO_IsRecursive'] == true)||$recursive) {
+            if ((isset($param['__SEO_IsRecursive'])) && ($param['__SEO_IsRecursive'] == true) || $recursive) {
 
                 $structures = null;
                 $this->query->className('structure')

src/tab/InfoTab.php

Doc Comments +3 added lines

@@ -18,6 +18,9 @@
     /** @var string Path to main view */
     public $indexView = 'info/index';
 
+    /**
+     * @param \samsoncms\seo\Core $renderer
+     */
     public function __construct($renderer, $data = null)
     {
         // Save renderer

src/tab/Tab.php

Doc Comments +11 added lines, -4 removed lines

@@ -84,10 +84,10 @@ 
 
         /**
          * Render default tab with fields
-         * @param $renderer
-         * @param $query
-         * @param $entity
-         * @param $schema
+         * @param RenderInterface $renderer
+         * @param QueryInterface $query
+         * @param Record $entity
+         * @param MaterialSchema $schema
          */
         public function renderDefaultStructure($renderer, $query, $entity, $schema)
         {
@@ -105,6 +105,10 @@ 
 
         /**
          * Render control schemas
+         * @param RenderInterface $renderer
+         * @param QueryInterface $query
+         * @param Record $entity
+         * @param ControlSchema $schema
          */
         public function renderControlStructure($renderer, $query, $entity, $schema)
         {
@@ -170,6 +174,9 @@ 
             return $this->renderer->view($this->contentView)->content($content)->output();
         }
 
+        /**
+         * @param string $name
+         */
         public function setName($name)
         {
             $this->name = $name;

Unused Use Statements -2 removed lines

@@ -8,12 +8,10 @@
 
 namespace samsoncms\seo\tab;
 
-use samson\activerecord\dbRelation;
 use samsoncms\seo\render\Element;
 use samsoncms\seo\schema\control\ControlSchema;
 use samsoncms\seo\schema\material\MaterialSchema;
 use samsoncms\seo\schema\Schema;
-use samson\core\SamsonLocale;
 use samsonframework\core\RenderInterface;
 use samsonframework\orm\QueryInterface;
 use samsonframework\orm\Record;

Spacing +4 added lines, -4 removed lines

@@ -139,7 +139,7 @@ 
                 $html = $subTab->content();
 
                 // Render elements if exists
-                $isElements = isset($subTab->schema->elements)&&(!empty($subTab->schema->elements));
+                $isElements = isset($subTab->schema->elements) && (!empty($subTab->schema->elements));
                 if ($isElements) {
 
                     // Create element instance
@@ -149,7 +149,7 @@ 
                     $contentNestedElement = $elements->renderNestedElements($subTab->schema->elements);
 
                     // Insert element as first child of table
-                    $html = preg_replace('/>/', '>' . $contentNestedElement, $html, 1);
+                    $html = preg_replace('/>/', '>'.$contentNestedElement, $html, 1);
 
                     // Get all not nested element
                     $contentNotNestedElement = $elements->renderNotNestedElements($subTab->schema->elements);
@@ -162,7 +162,7 @@ 
                 if ($isElements) {
 
                     // Concatenate element to main view
-                    $content = $contentNotNestedElement . $content;
+                    $content = $contentNotNestedElement.$content;
                 }
             }
 
@@ -177,5 +177,5 @@ 
     }
 
 } else {
-    class Tab{}
+    class Tab {}
 }

src/render/Element.php

Spacing +2 added lines, -2 removed lines

@@ -63,7 +63,7 @@ 
         foreach ($elements as $element) {
 
             // Check if this element is nested
-            $isNested = (isset($element['Nested'])&&$element['Nested'] == true);
+            $isNested = (isset($element['Nested']) && $element['Nested'] == true);
 
             if (isset($element['Field']) && ($field == false)) {
                 continue;
@@ -77,7 +77,7 @@ 
 
 
             // Check if this element is need to be hide
-            $isHide = (isset($element['Hide'])&&$element['Hide'] == true);
+            $isHide = (isset($element['Hide']) && $element['Hide'] == true);
 
             // If not right go further
             if ($isHide) {

Doc Comments +1 added lines

@@ -116,6 +116,7 @@
     /**
      * Render only nested elements
      * @param $elements
+     * @param boolean $field
      * @return String
      */
     public function renderNestedElements($elements, $field = null)

Unused Use Statements -2 removed lines

@@ -8,8 +8,6 @@
 
 namespace samsoncms\seo\render;
 
-use samsoncms\seo\render\Button;
-
 /**
  * Class Element generate elements
  * @package samsoncms\seo\render

src/render/Info.php

Spacing +1 added lines, -1 removed lines

@@ -12,7 +12,7 @@
  * Class Info block which show info about something
  * @package samsoncms\seo\render
  */
-class Info{
+class Info {
 
     public function __construct($renderer, $data)
     {

src/sitemap/Xml.php

Spacing +4 added lines, -4 removed lines

@@ -54,13 +54,13 @@ 
         if (!empty($urls)) {
 
             // Get full path
-            $path = $this->currentHost . $category . DIRECTORY_SEPARATOR;
+            $path = $this->currentHost.$category.DIRECTORY_SEPARATOR;
 
             // Add separator
             $tagAsSeparator = "</loc></url><url><loc>{$path}";
 
             // Implode all array elements and concatenate separator with right xml tags and concatenate first and last tags
-            $xml = '<url><loc>' . $path . implode($tagAsSeparator, $urls) . '</loc></url>';
+            $xml = '<url><loc>'.$path.implode($tagAsSeparator, $urls).'</loc></url>';
         }
 
         return $xml;
@@ -90,11 +90,11 @@ 
         $innerBlock = $this->getXmlContentByUrls($urls, $path);
 
         // Concatenate header and footer of xml text
-        $result = "<?xml version='1.0' standalone='yes'?>" .
+        $result = "<?xml version='1.0' standalone='yes'?>".
             '<urlset xmlns="http://www.sitemaps.org/schemas/sitemap/0.9"
             xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
             xsi:schemaLocation="http://www.sitemaps.org/schemas/sitemap/0.9
-            http://www.sitemaps.org/schemas/sitemap/0.9/sitemap.xsd">' . $innerBlock . '</urlset>';
+            http://www.sitemaps.org/schemas/sitemap/0.9/sitemap.xsd">' . $innerBlock.'</urlset>';
 
         return $result;
     }