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

Indentation +5 added lines, -5 removed lines

@@ -1,10 +1,10 @@
 <?php
 /**
- * Created by PhpStorm.
- * User: molodyko
- * Date: 29.09.2015
- * Time: 11:32
- */
+     * Created by PhpStorm.
+     * User: molodyko
+     * Date: 29.09.2015
+     * Time: 11:32
+     */
 
 namespace samsoncms\seo\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)
     {
 

Indentation +5 added lines, -5 removed lines

@@ -1,10 +1,10 @@
 <?php
 /**
- * Created by PhpStorm.
- * User: molodyko
- * Date: 24.09.2015
- * Time: 18:31
- */
+     * Created by PhpStorm.
+     * User: molodyko
+     * Date: 24.09.2015
+     * Time: 18:31
+     */
 
 namespace samsoncms\seo\sitemap;
 

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

Indentation +5 added lines, -5 removed lines

@@ -1,10 +1,10 @@
 <?php
 /**
- * Created by PhpStorm.
- * User: molodyko
- * Date: 29.09.2015
- * Time: 13:21
- */
+     * Created by PhpStorm.
+     * User: molodyko
+     * Date: 29.09.2015
+     * Time: 13:21
+     */
 
 namespace samsoncms\seo\tab;
 

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;

Indentation +5 added lines, -5 removed lines

@@ -1,10 +1,10 @@
 <?php
 /**
- * Created by PhpStorm.
- * User: molodyko
- * Date: 20.09.2015
- * Time: 14:39
- */
+     * Created by PhpStorm.
+     * User: molodyko
+     * Date: 20.09.2015
+     * Time: 14:39
+     */
 
 namespace samsoncms\seo\tab;
 

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/Migrate.php

Indentation +5 added lines, -5 removed lines

@@ -1,10 +1,10 @@
 <?php
 /**
- * Created by PhpStorm.
- * User: molodyko
- * Date: 18.09.2015
- * Time: 16:13
- */
+     * Created by PhpStorm.
+     * User: molodyko
+     * Date: 18.09.2015
+     * Time: 16:13
+     */
 
 namespace samsoncms\seo;
 

Spacing +4 added lines, -4 removed lines

@@ -169,13 +169,13 @@ 
         // Iterate and create all fields
         foreach ($fields as $field) {
 
-            $fieldInstance = $this->isFieldExists($field['Name'] . '_' . $prefix);
+            $fieldInstance = $this->isFieldExists($field['Name'].'_'.$prefix);
 
             // If field not exists then create it
             if (!$fieldInstance) {
                 // Create and add field to structure
                 $fieldInstance = $this->createField(
-                    $field['Name'] . '_' . $prefix,
+                    $field['Name'].'_'.$prefix,
                     $field['Description'],
                     $field['Type'],
                     // If it is the select type then add value
@@ -361,8 +361,8 @@ 
             // Set prefix of material
             $prefix = 'Material of ';
             $material = $this->createNestedMaterial(
-                $prefix . $structure->Name,
-                $prefix . $structure->Url
+                $prefix.$structure->Name,
+                $prefix.$structure->Url
             );
 
             $this->assignNestedMaterial($material, $structure);

Doc Comments +12 added lines, -9 removed lines

@@ -28,6 +28,9 @@ 
 
     const MAIN_PREFIX_NAME = 'main';
 
+    /**
+     * @param \samson\activerecord\dbQuery $query
+     */
     public function __construct($query)
     {
         $this->query = $query;
@@ -99,7 +102,7 @@ 
 
     /**
      * Assign structure to passed material
-     * @param $material
+     * @param \samson\activerecord\Material $material
      * @param $structure
      * @return mixed|structurematerial
      */
@@ -203,7 +206,7 @@ 
 
     /**
      * Create field
-     * @param $name
+     * @param string $name
      * @param $description
      * @param $type
      * @param $value
@@ -272,8 +275,8 @@ 
 
     /**
      * Get structure if exists
-     * @param $name
-     * @param $url
+     * @param string $name
+     * @param string $url
      * @return mixed
      */
     public function isStructureExists($name, $url)
@@ -286,7 +289,7 @@ 
 
     /**
      * Get field if exists
-     * @param $name
+     * @param string $name
      * @return mixed
      */
     public function isFieldExists($name)
@@ -312,7 +315,7 @@ 
 
     /**
      * Assign nested material to the structure
-     * @param $material
+     * @param \samson\activerecord\Material $material
      * @param $structure
      */
     public function assignNestedMaterial($material, $structure)
@@ -331,8 +334,8 @@ 
 
     /**
      * Create nested material of structure
-     * @param $name
-     * @param $url
+     * @param string $name
+     * @param string $url
      * @return \samson\activerecord\material
      */
     public function createNestedMaterial($name, $url)
@@ -349,7 +352,7 @@ 
     /**
      * Create nested material on the structure if it don't exists and assign it to the passed structure
      * @param $structure
-     * @return null|\samson\activerecord\material
+     * @return \samson\activerecord\Material
      */
     public function buildNestedMaterial($structure)
     {

src/render/Button.php

Indentation +5 added lines, -5 removed lines

@@ -1,10 +1,10 @@
 <?php
 /**
- * Created by PhpStorm.
- * User: molodyko
- * Date: 24.09.2015
- * Time: 16:17
- */
+     * Created by PhpStorm.
+     * User: molodyko
+     * Date: 24.09.2015
+     * Time: 16:17
+     */
 
 namespace samsoncms\seo\render;
 

src/render/Element.php

Indentation +5 added lines, -5 removed lines

@@ -1,10 +1,10 @@
 <?php
 /**
- * Created by PhpStorm.
- * User: molodyko
- * Date: 24.09.2015
- * Time: 16:21
- */
+     * Created by PhpStorm.
+     * User: molodyko
+     * Date: 24.09.2015
+     * Time: 16:21
+     */
 
 namespace samsoncms\seo\render;
 

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) {