locomotivemtl / charcoal-cms

src/Charcoal/Cms/AbstractNews.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());
         }
     }

Doc Comments +2 added lines, -2 removed lines

@@ -182,7 +182,7 @@ 
     }
 
     /**
-     * @param  string|DateTimeInterface|null $newsDate The news date.
+     * @param  string $newsDate The news date.
      * @throws InvalidArgumentException If the timestamp is invalid.
      * @return self
      */
@@ -307,7 +307,7 @@ 
 
     /**
      * @see MetatagTrait::defaultMetaDescription()
-     * @return Translation|string|null
+     * @return null|Translation
      */
     public function defaultMetaDescription()
     {

src/Charcoal/Cms/EventCategory.php

Doc Comments +1 added lines, -1 removed lines

@@ -55,7 +55,7 @@
     }
 
     /**
-     * @return mixed
+     * @return string[]
      */
     public function name()
     {

Spacing +3 added lines, -3 removed lines

@@ -31,7 +31,7 @@ 
     {
         parent::__construct($data);
 
-        if (is_callable([ $this, 'defaultData' ])) {
+        if (is_callable([$this, 'defaultData'])) {
             $this->setData($this->defaultData());
         }
     }
@@ -82,9 +82,9 @@ 
         parent::validate($v);
 
         foreach ($this->translator()->locales() as $locale => $value) {
-            if (!(string)$this['name'][$locale]) {
+            if (!(string) $this['name'][$locale]) {
                 $this->validator()->error(
-                    (string)$this->translator()->translation([
+                    (string) $this->translator()->translation([
                         'fr' => 'Le NOM doit être rempli dans toutes les langues.',
                         'en' => 'The NAME must be filled in all languages.',
                     ])

src/Charcoal/Cms/Section/ContentSection.php

Unused Use Statements -1 removed lines

@@ -4,7 +4,6 @@
 
 // From 'charcoal-cms'
 use Charcoal\Cms\AbstractSection;
-use Charcoal\Cms\Mixin\ContentSectionInterface;
 
 /**
  * Content section

src/Charcoal/Cms/Section/EmptySection.php

Unused Use Statements -1 removed lines

@@ -4,7 +4,6 @@
 
 // From 'charcoal-cms'
 use Charcoal\Cms\AbstractSection;
-use Charcoal\Cms\Mixin\EmptySectionInterface;
 
 /**
  * Empty section

src/Charcoal/Cms/TemplateableTrait.php

Doc Comments +1 added lines, -1 removed lines

@@ -86,7 +86,7 @@
     /**
      * Retrieve the renderable object's template identifier.
      *
-     * @return mixed
+     * @return string
      */
     public function templateIdent()
     {

Spacing +4 added lines, -4 removed lines

@@ -107,7 +107,7 @@ 
         if ($property instanceof SelectablePropertyInterface) {
             if ($property->hasChoice($templateIdent)) {
                 $choice = $property->choice($templateIdent);
-                $keys   = [ 'controller', 'template', 'class' ];
+                $keys   = ['controller', 'template', 'class'];
                 foreach ($keys as $key) {
                     if (isset($choice[$key])) {
                         return $choice[$key];
@@ -183,7 +183,7 @@ 
     public function hasTemplateOptions()
     {
         if ($this->templateOptionsStructure()) {
-            return (bool)count(
+            return (bool) count(
                 iterator_to_array(
                     $this->templateOptionsStructure()->properties()
                 )
@@ -245,7 +245,7 @@ 
                 $struct = $property->structureVal($obj[$propertyIdent]);
 
                 /** Provide support for dynamically wrapping translation sets.  */
-                if (in_array(get_class($struct), [ Model::class, StructureModel::class ])) {
+                if (in_array(get_class($struct), [Model::class, StructureModel::class])) {
                     $struct = $this->translateTemplateOptionsModel($struct);
                 }
 
@@ -290,7 +290,7 @@ 
             if ($property instanceof SelectablePropertyInterface) {
                 if ($property->hasChoice($val)) {
                     $choice = $property->choice($val);
-                    $keys   = [ 'controller', 'template', 'class' ];
+                    $keys   = ['controller', 'template', 'class'];
                     foreach ($keys as $key) {
                         if (isset($choice[$key])) {
                             $interface = $choice[$key];

src/Charcoal/Admin/Widget/GroupAttachmentWidget.php

Spacing +1 added lines, -1 removed lines

@@ -200,7 +200,7 @@
             }
 
             $attachmentWidgets = $structureMetadata->get('attachments.widgets');
-            foreach ((array)$attachmentWidgets as $ident => $metadata) {
+            foreach ((array) $attachmentWidgets as $ident => $metadata) {
                 $this->addGroup($ident, $metadata);
             }
 

Doc Comments +1 added lines, -1 removed lines

@@ -238,7 +238,7 @@
     /**
      * Retrieve the form object's template controller identifier.
      *
-     * @return mixed
+     * @return string
      */
     public function controllerIdent()
     {

src/Charcoal/Cms/Section/BlocksSection.php

Unused Use Statements -4 removed lines

@@ -2,13 +2,9 @@
 
 namespace Charcoal\Cms\Section;
 
-use RuntimeException;
-
 // From Pimple
-use Pimple\Container;
 
 // From 'charcoal-core'
-use Charcoal\Loader\CollectionLoader;
 
 // From 'charcoal-cms'
 use Charcoal\Cms\AbstractSection;

src/Charcoal/Cms/ServiceProvider/CmsServiceProvider.php

Unused Use Statements -2 removed lines

@@ -7,14 +7,12 @@
 use Pimple\ServiceProviderInterface;
 
 // From 'charcoal-core'
-use Charcoal\Model\AbstractModel;
 
 // From 'charcoal-factory'
 use Charcoal\Factory\GenericFactory as Factory;
 
 // From 'charcoal-cms'
 use Charcoal\Cms\SectionInterface;
-use Charcoal\Cms\Config;
 use Charcoal\Cms\Config\CmsConfig;
 use Charcoal\Cms\Service\Loader\EventLoader;
 use Charcoal\Cms\Service\Loader\NewsLoader;

Spacing +9 added lines, -9 removed lines

@@ -60,7 +60,7 @@ 
          * @param Container $container Pimple DI 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');
 
@@ -95,7 +95,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'),
                 'time_formats' => $container['cms/config']->get('time_formats'),
@@ -107,7 +107,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',
                 '$container[\'date/helper\']',
@@ -128,7 +128,7 @@ 
          * @param Container $container Pimple DI Container.
          * @return Factory
          */
-        $container['cms/section/factory'] = function (Container $container) {
+        $container['cms/section/factory'] = function(Container $container) {
             return new Factory([
                 'base_class'       => SectionInterface::class,
                 'arguments'        => $container['model/factory']->arguments(),
@@ -142,7 +142,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'],
@@ -171,7 +171,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'],
@@ -192,7 +192,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'],
@@ -217,7 +217,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'],
@@ -241,7 +241,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/Service/Loader/EventLoader.php

Doc Comments +3 added lines, -3 removed lines

@@ -87,7 +87,7 @@ 
     }
 
     /**
-     * @param mixed $date The news date to filter
+     * @param DateTime $date The news date to filter
      *                    [startDate, endDate]
      *                    DateTimeInterface
      *                    string.
@@ -103,7 +103,7 @@ 
     }
 
     /**
-     * @param mixed $date The news date to filter
+     * @param DateTime $date The news date to filter
      *                    [startDate, endDate]
      *                    DateTimeInterface
      *                    string.
@@ -119,7 +119,7 @@ 
     }
 
     /**
-     * @return mixed
+     * @return string
      */
     public function lifespan()
     {

Spacing +3 added lines, -3 removed lines

@@ -56,7 +56,7 @@ 
     {
         $now = new DateTime();
         $loader = $this->all();
-        $loader->addFilter('publishDate', $now->format('Y-m-d H:i:s'), [ 'operator' => '<=' ])
+        $loader->addFilter('publishDate', $now->format('Y-m-d H:i:s'), ['operator' => '<='])
             ->addOrder('start_date', 'asc');
 
         return $loader;
@@ -97,7 +97,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;
     }
@@ -113,7 +113,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;
     }