locomotivemtl / charcoal-cms

src/Charcoal/Property/TemplateProperty.php

Unused Use Statements -3 removed lines

@@ -4,9 +4,6 @@
 
 use PDO;
 
-use RuntimeException;
-use InvalidArgumentException;
-
 // From Pimple
 use Pimple\Container;
 

Spacing +2 added lines, -2 removed lines

@@ -128,7 +128,7 @@ 
                 return '';
             }
         } elseif ($val instanceof Translation) {
-            $propertyValue = (string)$val;
+            $propertyValue = (string) $val;
         } else {
             $propertyValue = $val;
         }
@@ -176,7 +176,7 @@ 
         $val = $this->val();
         if ($this->hasChoice($val)) {
             $choice = $this->choice($val);
-            $keys   = [ 'controller', 'template', 'class' ];
+            $keys   = ['controller', 'template', 'class'];
             foreach ($keys as $key) {
                 if (isset($choice[$key])) {
                     return $choice[$key];

src/Charcoal/Admin/Widget/Cms/SectionTableTrait.php

Unused Use Statements -1 removed lines

@@ -2,7 +2,6 @@
 
 namespace Charcoal\Admin\Widget\Cms;
 
-use Exception;
 use RuntimeException;
 
 // From 'charcoal-core'

Spacing +3 added lines, -3 removed lines

@@ -96,7 +96,7 @@ 
 
                 switch ($sectionType) {
                     case AbstractSection::TYPE_EXTERNAL:
-                        $externalUrl = (string)$object->externalUrl();
+                        $externalUrl = (string) $object->externalUrl();
                         $linkExcerpt = '';
                         $tagTemplate = '<span class="glyphicon glyphicon-link" data-toggle="tooltip" '.
                             'data-placement="auto" title="%1$s"></span>';
@@ -123,7 +123,7 @@ 
         }
 
         if ($propertyIdent === 'title') {
-            if (is_callable([ $object, 'navMenu' ]) && $object->navMenu()) {
+            if (is_callable([$object, 'navMenu']) && $object->navMenu()) {
                 $propertyValue .= sprintf(
                     ' &nbsp; '.
                     '<span class="glyphicon glyphicon-list" data-toggle="tooltip" '.
@@ -135,7 +135,7 @@ 
                 );
             }
 
-            if (is_callable([ $object, 'locked' ]) && $object->locked()) {
+            if (is_callable([$object, 'locked']) && $object->locked()) {
                 $propertyValue .= sprintf(
                     ' &nbsp; '.
                     '<span class="glyphicon glyphicon-lock" data-toggle="tooltip" '.

src/Charcoal/Cms/AbstractSection.php

Unused Use Statements -1 removed lines

@@ -5,7 +5,6 @@
 use InvalidArgumentException;
 
 // From 'charcoal-core'
-use Charcoal\Model\Collection;
 use Charcoal\Loader\CollectionLoader;
 
 // From 'charcoal-object'

Spacing +2 added lines, -2 removed lines

@@ -128,7 +128,7 @@ 
     {
         parent::__construct($data);
 
-        if (is_callable([ $this, 'defaultData' ])) {
+        if (is_callable([$this, 'defaultData'])) {
             $this->setData($this->defaultData());
         }
     }
@@ -490,7 +490,7 @@ 
          * an empty string, or an object.
          */
         if (!is_array($value)) {
-            return [ $value ];
+            return [$value];
         }
 
         return $value;

src/Charcoal/Cms/ServiceProvider/CmsServiceProvider.php

Unused Use Statements -2 removed lines

@@ -12,13 +12,11 @@
 use Pimple\ServiceProviderInterface;
 
 // Local dependencies
-use Charcoal\Cms\Config;
 
 // Cms Loaders
 use Charcoal\Cms\Service\Loader\SectionLoader;
 
 // dependencies from `charcoal-core`
-use Charcoal\Model\AbstractModel;
 
 // dependencies from `charcoal-cms`
 use Charcoal\Cms\SectionInterface;

Spacing +9 added lines, -9 removed lines

@@ -51,7 +51,7 @@ 
          * @param Container $container
          * @return CmsConfig Website configurations (from cms.json).
          */
-        $container['cms/config'] = function (Container $container) {
+        $container['cms/config'] = function(Container $container) {
             $appConfig = $container['config'];
             $cms = $appConfig->get('cms');
 
@@ -83,7 +83,7 @@ 
          * @param Container $container Pimple DI Container.
          * @return DateHelper
          */
-        $container['date/helper'] = function (Container $container) {
+        $container['date/helper'] = function(Container $container) {
 
             trigger_error(sprintf(
                 '%s is deprecated, use %s instead',
@@ -102,7 +102,7 @@ 
          * @param Container $container Pimple DI Container.
          * @return DateHelper
          */
-        $container['cms/date/helper'] = function (Container $container) {
+        $container['cms/date/helper'] = function(Container $container) {
 
             return new DateHelper([
                 'date_formats' => $container['cms/config']->get('date_formats'),
@@ -126,7 +126,7 @@ 
          * @param Container $container Pimple DI Container.
          * @return GenericFactory
          */
-        $container['cms/section/factory'] = function (Container $container) {
+        $container['cms/section/factory'] = function(Container $container) {
             return new GenericFactory([
                 'base_class'       => SectionInterface::class,
                 'arguments'        => $container['model/factory']->arguments(),
@@ -140,7 +140,7 @@ 
          * @param Container $container Pimple DI Container.
          * @return SectionLoader
          */
-        $container['cms/section/loader'] = function (Container $container) {
+        $container['cms/section/loader'] = function(Container $container) {
             $sectionLoader = new SectionLoader([
                 'loader'     => $container['model/collection/loader'],
                 'factory'    => $container['model/factory'],
@@ -169,7 +169,7 @@ 
          * @param Container $container Pimple DI Container.
          * @return NewsLoader
          */
-        $container['cms/news/loader'] = function (Container $container) {
+        $container['cms/news/loader'] = function(Container $container) {
             $newsLoader = new NewsLoader([
                 'loader'     => $container['model/collection/loader'],
                 'factory'    => $container['model/factory'],
@@ -190,7 +190,7 @@ 
          * @param Container $container
          * @return NewsManager
          */
-        $container['cms/news/manager'] = function (Container $container) {
+        $container['cms/news/manager'] = function(Container $container) {
 
             $newsManager = new NewsManager([
                 'loader'      => $container['model/collection/loader'],
@@ -215,7 +215,7 @@ 
          * @param Container $container Pimple DI Container.
          * @return EventLoader
          */
-        $container['cms/event/loader'] = function (Container $container) {
+        $container['cms/event/loader'] = function(Container $container) {
             $eventLoader = new EventLoader([
                 'loader'     => $container['model/collection/loader'],
                 'factory'    => $container['model/factory'],
@@ -239,7 +239,7 @@ 
          * @param Container $container
          * @return EventManager
          */
-        $container['cms/event/manager'] = function (Container $container) {
+        $container['cms/event/manager'] = function(Container $container) {
 
             $eventManager = new EventManager([
                 'loader'       => $container['model/collection/loader'],

src/Charcoal/Cms/NewsCategory.php

Spacing +1 added lines, -1 removed lines

@@ -31,7 +31,7 @@
     {
         parent::__construct($data);
 
-        if (is_callable([ $this, 'defaultData' ])) {
+        if (is_callable([$this, 'defaultData'])) {
             $this->setData($this->defaultData());
         }
     }

src/Charcoal/Cms/Service/Loader/EventLoader.php

Spacing +3 added lines, -3 removed lines

@@ -54,7 +54,7 @@ 
     {
         $now = new DateTime();
         $loader = $this->all();
-        $loader->addFilter('publish_date', $now->format('Y-m-d H:i:s'), [ 'operator' => '<=' ])
+        $loader->addFilter('publish_date', $now->format('Y-m-d H:i:s'), ['operator' => '<='])
             ->addOrder('start_date', 'asc');
 
         return $loader;
@@ -95,7 +95,7 @@ 
     {
         $date = $this->parseAsDate($date);
         $loader = $this->published();
-        $loader->addFilter('end_date', $date->format('Y-m-d H:i:s'), [ 'operator' => '>=' ]);
+        $loader->addFilter('end_date', $date->format('Y-m-d H:i:s'), ['operator' => '>=']);
 
         return $loader;
     }
@@ -111,7 +111,7 @@ 
     {
         $date = $this->parseAsDate($date);
         $loader = $this->published();
-        $loader->addFilter('end_date', $date->format('Y-m-d H:i:s'), [ 'operator' => '<' ]);
+        $loader->addFilter('end_date', $date->format('Y-m-d H:i:s'), ['operator' => '<']);
 
         return $loader;
     }

src/Charcoal/Cms/Service/Loader/NewsLoader.php

Spacing +4 added lines, -4 removed lines

@@ -47,8 +47,8 @@ 
     {
         $now = new DateTime();
         $loader = $this->all();
-        $loader->addFilter('publish_date', $now->format('Y-m-d H:i:s'), [ 'operator' => '<=' ])
-            ->addFilter('expiry_date', $now->format('Y-m-d H:i:s'), [ 'operator' => '>=' ]);
+        $loader->addFilter('publish_date', $now->format('Y-m-d H:i:s'), ['operator' => '<='])
+            ->addFilter('expiry_date', $now->format('Y-m-d H:i:s'), ['operator' => '>=']);
 
         return $loader;
     }
@@ -60,8 +60,8 @@ 
     {
         $now = new DateTime();
         $loader = $this->all();
-        $loader->addFilter('publish_date', $now->format('Y-m-d H:i:s'), [ 'operator' => '<=' ])
-            ->addFilter('expiry_date', $now->format('Y-m-d H:i:s'), [ 'operator' => '<=' ]);
+        $loader->addFilter('publish_date', $now->format('Y-m-d H:i:s'), ['operator' => '<='])
+            ->addFilter('expiry_date', $now->format('Y-m-d H:i:s'), ['operator' => '<=']);
 
         return $loader;
     }

src/Charcoal/Cms/Service/Manager/EventManager.php

Spacing +4 added lines, -4 removed lines

@@ -216,7 +216,7 @@ 
         }
 
         if ($this->category()) {
-            $loader->addFilter('category', $this->category(), [ 'operator' => 'in' ]);
+            $loader->addFilter('category', $this->category(), ['operator' => 'in']);
         }
 
         if ($this->numPerPage()) {
@@ -321,8 +321,8 @@ 
 
         $ids = explode(',', $ids);
 
-        $loader->addFilter('id', $ids, [ 'operator' => 'in' ])
-            ->addOrder('id', 'values', [ 'values' => $ids ]);
+        $loader->addFilter('id', $ids, ['operator' => 'in'])
+            ->addOrder('id', 'values', ['values' => $ids]);
 
         if (count($options) > 0) {
             foreach ($options as $key => $option) {
@@ -369,7 +369,7 @@ 
         $loader = $this->loader()->archive();
 
         if ($this->category()) {
-            $loader->addFilter('category', $this->category(), [ 'operator' => 'in' ]);
+            $loader->addFilter('category', $this->category(), ['operator' => 'in']);
         }
         if ($this->numPerPage()) {
             $loader->setPage($this->page());

src/Charcoal/Cms/Service/Manager/NewsManager.php

Spacing +4 added lines, -4 removed lines

@@ -122,7 +122,7 @@ 
         $loader->addOrder('news_date', 'desc');
 
         if ($this->category()) {
-            $loader->addFilter('category', $this->category(), [ 'operator' => 'in' ]);
+            $loader->addFilter('category', $this->category(), ['operator' => 'in']);
         }
         if ($this->numPerPage()) {
             $loader->setPage($this->page());
@@ -223,8 +223,8 @@ 
 
         $ids = explode(',', $ids);
 
-        $loader->addFilter('id', $ids, [ 'operator' => 'in' ])
-            ->addOrder('id', 'values', [ 'values' => $ids ]);
+        $loader->addFilter('id', $ids, ['operator' => 'in'])
+            ->addOrder('id', 'values', ['values' => $ids]);
 
         if (count($options) > 0) {
             foreach ($options as $key => $option) {
@@ -272,7 +272,7 @@ 
         $loader->addOrder('news_date', 'desc');
 
         if ($this->category()) {
-            $loader->addFilter('category', $this->category(), [ 'operator' => 'in' ]);
+            $loader->addFilter('category', $this->category(), ['operator' => 'in']);
         }
         if ($this->numPerPage()) {
             $loader->setPage($this->page());