locomotivemtl / charcoal-cms

src/Charcoal/Cms/Config/EventConfig.php

Doc Comments +1 added lines, -1 removed lines

@@ -59,7 +59,7 @@
     }
 
     /**
-     * @return boolean Entry cycle.
+     * @return string Entry cycle.
      */
     public function entryCycle()
     {

src/Charcoal/Cms/NewsCategory.php

Doc Comments +1 added lines, -1 removed lines

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

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/Section/BlocksSection.php

Unused Use Statements -2 removed lines

@@ -10,10 +10,8 @@
 use Charcoal\Cms\Mixin\Traits\BlocksSectionTrait;
 
 // dependencies from `charcoal-core`
-use Charcoal\Loader\CollectionLoader;
 
 // dependencies from `pimple`
-use Pimple\Container;
 
 // dependencies from Psr-7
 use \RuntimeException;

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

Doc Comments +3 added lines, -3 removed lines

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

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

Doc Comments +1 added lines, -1 removed lines

@@ -213,7 +213,7 @@
     // ==========================================================================
 
     /**
-     * @return object
+     * @return string
      */
     public function objType()
     {

Unused Use Statements -1 removed lines

@@ -4,7 +4,6 @@
 
 use Charcoal\Loader\CollectionLoader;
 use Charcoal\Object\ObjectRoute;
-use Exception;
 
 use \Charcoal\Translator\TranslatorAwareTrait;
 /**

Spacing +2 added lines, -2 removed lines

@@ -72,7 +72,7 @@ 
         $loader = $this->all();
         $operator = [];
         if (!$this->baseSection()) {
-            $operator = [ 'operator' => 'IS NULL' ];
+            $operator = ['operator' => 'IS NULL'];
         }
         $loader->addFilter('master', $this->baseSection(), $operator);
 
@@ -139,7 +139,7 @@ 
         }
         $q = 'SELECT * FROM `'.$proto->source()->table().'`
             WHERE active=1 AND ( '
-            . implode(' OR ', $filters) . ' )
+            . implode(' OR ', $filters).' )
             ORDER BY creation_date ASC';
 
         $objectRoutes = $loader->loadFromQuery($q);

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

Doc Comments +6 added lines, -6 removed lines

@@ -230,7 +230,7 @@ 
 
     /**
      * @param integer|null $id The event id.
-     * @return mixed
+     * @return EventInterface
      */
     public function entry($id = null)
     {
@@ -398,7 +398,7 @@ 
     }
 
     /**
-     * @return mixed The previous event
+     * @return EventInterface The previous event
      */
     public function prev()
     {
@@ -410,7 +410,7 @@ 
     }
 
     /**
-     * @return mixed The next event
+     * @return EventInterface The next event
      */
     public function next()
     {
@@ -504,7 +504,7 @@ 
 
     /**
      * Amount of event (total)
-     * @return integer How many event?
+     * @return boolean How many event?
      */
     public function numEvent()
     {
@@ -559,7 +559,7 @@ 
     }
 
     /**
-     * @return mixed
+     * @return string
      */
     public function objType()
     {
@@ -567,7 +567,7 @@ 
     }
 
     /**
-     * @return mixed
+     * @return string
      */
     public function featIdent()
     {

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

Doc Comments +5 added lines, -5 removed lines

@@ -135,7 +135,7 @@ 
 
     /**
      * @param integer|null $id The news id.
-     * @return mixed
+     * @return NewsInterface
      */
     public function entry($id = null)
     {
@@ -301,7 +301,7 @@ 
     }
 
     /**
-     * @return mixed The previous news
+     * @return NewsInterface The previous news
      */
     public function prev()
     {
@@ -313,7 +313,7 @@ 
     }
 
     /**
-     * @return mixed The next news
+     * @return NewsInterface The next news
      */
     public function next()
     {
@@ -385,7 +385,7 @@ 
 
     /**
      * Amount of news (total)
-     * @return integer How many news?
+     * @return boolean How many news?
      */
     public function numNews()
     {
@@ -448,7 +448,7 @@ 
     }
 
     /**
-     * @return mixed
+     * @return string
      */
     public function featIdent()
     {

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());

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 +8 added lines, -8 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) {
 
             return new DateHelper([
                 'date_formats' => $container['cms/config']->get('date_formats'),
@@ -107,7 +107,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(),
@@ -121,7 +121,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'],
@@ -150,7 +150,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'],
@@ -171,7 +171,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'],
@@ -196,7 +196,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'],
@@ -217,7 +217,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/Support/Traits/EventManagerAwareTrait.php

Doc Comments +4 added lines, -4 removed lines

@@ -27,7 +27,7 @@ 
     /**
      * Formatted event list
      * Return the entries for the current page
-     * @return \Generator|void
+     * @return \Generator
      */
     public function eventsList()
     {
@@ -41,7 +41,7 @@ 
     /**
      * Formatted event archive list
      * Returns the entries for the current page.
-     * @return \Generator|void
+     * @return \Generator
      */
     public function eventArchiveList()
     {
@@ -73,7 +73,7 @@ 
     }
 
     /**
-     * @return mixed
+     * @return \Generator
      */
     public function featEvents()
     {
@@ -122,7 +122,7 @@ 
 
     /**
      * Amount of events (total)
-     * @return integer How many events?
+     * @return boolean How many events?
      */
     public function numEvent()
     {

Spacing +10 added lines, -10 removed lines

@@ -232,8 +232,8 @@ 
     protected function eventFormatShort(EventInterface $event)
     {
         return [
-            'title'     => (string)$event->title(),
-            'url'       => (string)$event->url(),
+            'title'     => (string) $event->title(),
+            'url'       => (string) $event->url(),
             'startDate' => $this->getEventStartDateFormat($event),
             'endDate'   => $this->getEventEndDateFormat($event),
             'date'      => $this->getEventDateFormat($event),
@@ -254,7 +254,7 @@ 
             'endDate'   => $this->getEventEndDateFormat($event),
             'date'      => $this->getEventDateFormat($event),
             'time'      => $this->getEventTimeFormat($event),
-            'title'     => (string)$event->title(),
+            'title'     => (string) $event->title(),
             'url'       => $event->url()
         ];
     }
@@ -271,9 +271,9 @@ 
 
         return [
             'id'               => $event->id(),
-            'title'            => (string)$event->title(),
-            'summary'          => (string)$event->summary(),
-            'content'          => (string)$event->content(),
+            'title'            => (string) $event->title(),
+            'summary'          => (string) $event->summary(),
+            'content'          => (string) $event->content(),
             'image'            => $event->image(),
             'startDate'        => $this->getEventStartDateFormat($event),
             'endDate'          => $this->getEventEndDateFormat($event),
@@ -286,9 +286,9 @@ 
             'gallery'          => $gallery,
             'hasGallery'       => !!(count($gallery)),
             'url'              => $event->url(),
-            'metaTitle'        => (string)$event->metaTitle(),
-            'locationName'     => (string)$event->locationName(),
-            'dateNotes'        => (string)$event->dateNotes(),
+            'metaTitle'        => (string) $event->metaTitle(),
+            'locationName'     => (string) $event->locationName(),
+            'dateNotes'        => (string) $event->dateNotes(),
             'category'         => $event->category()
         ];
     }
@@ -301,7 +301,7 @@ 
     {
         return [
             'id'   => $category->id(),
-            'name' => (string)$category->name(),
+            'name' => (string) $category->name(),
         ];
     }