locomotivemtl / charcoal-cms

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

Doc Comments +5 added lines, -5 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()
     {
@@ -567,7 +567,7 @@ 
     }
 
     /**
-     * @return mixed
+     * @return string
      */
     public function featIdent()
     {

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

Doc Comments +6 added lines, -6 removed lines

@@ -149,7 +149,7 @@ 
 
     /**
      * @param integer|null $id The news id.
-     * @return mixed
+     * @return NewsInterface
      */
     public function entry($id = null)
     {
@@ -316,7 +316,7 @@ 
     }
 
     /**
-     * @return mixed The previous news
+     * @return NewsInterface The previous news
      */
     public function prev()
     {
@@ -328,7 +328,7 @@ 
     }
 
     /**
-     * @return mixed The next news
+     * @return NewsInterface The next news
      */
     public function next()
     {
@@ -400,7 +400,7 @@ 
 
     /**
      * Amount of news (total)
-     * @return integer How many news?
+     * @return boolean How many news?
      */
     public function numNews()
     {
@@ -442,7 +442,7 @@ 
     }
 
     /**
-     * @return mixed
+     * @return string
      */
     public function objType()
     {
@@ -450,7 +450,7 @@ 
     }
 
     /**
-     * @return mixed
+     * @return string
      */
     public function featIdent()
     {

src/Charcoal/Cms/Config/NewsConfig.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/Support/Helpers/DateHelper.php

Doc Comments +1 added lines, -1 removed lines

@@ -263,7 +263,7 @@
     }
 
     /**
-     * @param mixed $format DateTime to be formatted.
+     * @param string $format DateTime to be formatted.
      * @return mixed
      */
     private function crossPlatformFormat($format)

Spacing +6 added lines, -6 removed lines

@@ -199,18 +199,18 @@ 
                            ? $this->translator()->translation($case['formats']['to'])
                            : null;
 
-        $formats['from'] = $this->crossPlatformFormat((string)$formats['from']);
-        $formats['to']   = $this->crossPlatformFormat((string)$formats['to']);
+        $formats['from'] = $this->crossPlatformFormat((string) $formats['from']);
+        $formats['to']   = $this->crossPlatformFormat((string) $formats['to']);
 
         if (!$this->to || !$formats['to']) {
             return sprintf(
-                (string)$content,
+                (string) $content,
                 strftime($formats['from'], $this->from->getTimestamp())
             );
         }
 
         return sprintf(
-            (string)$content,
+            (string) $content,
             strftime($formats['from'], $this->from->getTimestamp()),
             strftime($formats['to'], $this->to->getTimestamp())
         );
@@ -235,13 +235,13 @@ 
 
         if (!$this->to || !$formats['to']) {
             return sprintf(
-                (string)$content,
+                (string) $content,
                 strftime($formats['from'], $this->from->getTimestamp())
             );
         }
 
         return sprintf(
-            (string)$content,
+            (string) $content,
             strftime($formats['from'], $this->from->getTimestamp()),
             strftime($formats['to'], $this->to->getTimestamp())
         );

src/Charcoal/Cms/AbstractEvent.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 +3 added lines, -3 removed lines

@@ -213,7 +213,7 @@ 
     }
 
     /**
-     * @param  string|DateTimeInterface|null $startDate Event starting date.
+     * @param  string $startDate Event starting date.
      * @throws InvalidArgumentException If the timestamp is invalid.
      * @return self
      */
@@ -246,7 +246,7 @@ 
     }
 
     /**
-     * @param  string|DateTimeInterface|null $endDate Event end date.
+     * @param  null|DateTimeInterface $endDate Event end date.
      * @throws InvalidArgumentException If the timestamp is invalid.
      * @return self
      */
@@ -298,7 +298,7 @@ 
     }
 
     /**
-     * @return Translation|string|null
+     * @return null|Translation
      */
     public function defaultMetaDescription()
     {

src/Charcoal/Property/TemplateProperty.php

Unused Use Statements -2 removed lines

@@ -3,8 +3,6 @@
 namespace Charcoal\Property;
 
 use PDO;
-
-use RuntimeException;
 use InvalidArgumentException;
 
 // From Pimple

Spacing +3 added lines, -3 removed lines

@@ -99,7 +99,7 @@ 
                 }
 
                 $struct    = $this->availableTemplates[$choiceIdent];
-                $immutable = [ 'value' => $struct['value'] ];
+                $immutable = ['value' => $struct['value']];
 
                 return array_replace($struct, $choice, $immutable);
             } elseif (isset($choice['template']) || isset($choice['controller']) || isset($choice['class'])) {
@@ -139,7 +139,7 @@ 
                 return '';
             }
         } elseif ($val instanceof Translation) {
-            $propertyValue = (string)$val;
+            $propertyValue = (string) $val;
         } else {
             $propertyValue = $val;
         }
@@ -187,7 +187,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/Cms/Route/GenericRoute.php

Spacing +1 added lines, -1 removed lines

@@ -499,7 +499,7 @@
             if (isset($available[$code])) {
                 $locale = $available[$code];
                 if (isset($locale['locales'])) {
-                    $choices = (array)$locale['locales'];
+                    $choices = (array) $locale['locales'];
                     array_push($locales, ...$choices);
                 } elseif (isset($locale['locale'])) {
                     array_push($locales, $locale['locale']);

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

Spacing +1 added lines, -1 removed lines

@@ -73,7 +73,7 @@
         $loader = $this->all();
         $operator = [];
         if (!$this->baseSection()) {
-            $operator = [ 'operator' => 'IS NULL' ];
+            $operator = ['operator' => 'IS NULL'];
         }
         $loader->addFilter('master', $this->baseSection(), $operator);
 

Unused Use Statements -3 removed lines

@@ -2,8 +2,6 @@ 
 
 namespace Charcoal\Cms\Service\Loader;
 
-use Exception;
-
 // From 'charcoal-core'
 use Charcoal\Loader\CollectionLoader;
 
@@ -11,7 +9,6 @@ 
 use Charcoal\Object\ObjectRoute;
 
 // From 'charcoal-translator'
-use Charcoal\Translator\TranslatorAwareTrait;
 
 /**
  * Section Loader

Doc Comments +1 added lines, -1 removed lines

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