locomotivemtl / charcoal-object

src/Charcoal/Object/RoutableTrait.php

Doc Comments +3 added lines, -3 removed lines

@@ -87,7 +87,7 @@ 
      * Set the object's URL slug pattern.
      *
      * @param  mixed $pattern The slug pattern.
-     * @return RoutableInterface Chainable
+     * @return RoutableTrait Chainable
      */
     public function setSlugPattern($pattern)
     {
@@ -193,7 +193,7 @@ 
      * Set the object's URL slug.
      *
      * @param  mixed $slug The slug.
-     * @return RoutableInterface Chainable
+     * @return RoutableTrait Chainable
      */
     public function setSlug($slug)
     {
@@ -677,7 +677,7 @@ 
      *
      * @param  string $className The class name of the object route model.
      * @throws InvalidArgumentException If the class name is not a string.
-     * @return AbstractPropertyDisplay Chainable
+     * @return RoutableTrait Chainable
      */
     protected function setObjectRouteClass($className)
     {

Spacing +3 added lines, -3 removed lines

@@ -283,7 +283,7 @@ 
         if ($this instanceof ViewableInterface && $this->view() !== null) {
             $route = $this->view()->render($pattern, $this->viewController());
         } else {
-            $route = preg_replace_callback('~\{\{\s*(.*?)\s*\}\}~i', [ $this, 'parseRouteToken' ], $pattern);
+            $route = preg_replace_callback('~\{\{\s*(.*?)\s*\}\}~i', [$this, 'parseRouteToken'], $pattern);
         }
 
         return $this->slugify($route);
@@ -306,7 +306,7 @@ 
         }
 
         $token = trim($token);
-        $method = [ $this, $token ];
+        $method = [$this, $token];
 
         if (is_callable($method)) {
             $value = call_user_func($method);
@@ -423,7 +423,7 @@ 
                     'route_options'       => $newData['route_options'],
                     'route_options_ident' => $newData['route_options_ident'],
                 ]);
-                $oldRoute->update([ 'route_template', 'route_options' ]);
+                $oldRoute->update(['route_template', 'route_options']);
 
                 continue;
             }

src/Charcoal/Object/ObjectRoute.php

Doc Comments +2 added lines, -2 removed lines

@@ -287,7 +287,7 @@ 
     /**
      * Set the route's last creation date.
      *
-     * @param  string|DateTimeInterface|null $time The date/time value.
+     * @param  string $time The date/time value.
      * @throws InvalidArgumentException If the date/time value is invalid.
      * @return self
      */
@@ -323,7 +323,7 @@ 
     /**
      * Set the route's last modification date.
      *
-     * @param  string|DateTimeInterface|null $time The date/time value.
+     * @param  string $time The date/time value.
      * @throws InvalidArgumentException If the date/time value is invalid.
      * @return self
      */

Unused Use Statements -4 removed lines

@@ -4,7 +4,6 @@ 
 use DateTime;
 use DateTimeInterface;
 use InvalidArgumentException;
-use RuntimeException;
 use Exception;
 
 // From Pimple
@@ -12,13 +11,10 @@ 
 
 // From 'charcoal-core'
 use Charcoal\Model\AbstractModel;
-use Charcoal\Loader\CollectionLoader;
-
 use Charcoal\Model\ModelFactoryTrait;
 use Charcoal\Loader\CollectionLoaderAwareTrait;
 
 // From 'charcoal-factory'
-use Charcoal\Factory\FactoryInterface;
 
 // From 'charcoal-object'
 use Charcoal\Object\ObjectRouteInterface;

src/Charcoal/Object/HierarchicalCollection.php

Doc Comments +2 added lines, -2 removed lines

@@ -281,7 +281,7 @@ 
     /**
      * @param  integer $page The current page. Start at 0.
      * @throws InvalidArgumentException If the parameter is not numeric or < 0.
-     * @return Pagination (Chainable)
+     * @return HierarchicalCollection (Chainable)
      */
     public function setPage($page)
     {
@@ -314,7 +314,7 @@ 
     /**
      * @param  integer $num The number of results to retrieve, per page.
      * @throws InvalidArgumentException If the parameter is not numeric or < 0.
-     * @return Pagination (Chainable)
+     * @return HierarchicalCollection (Chainable)
      */
     public function setNumPerPage($num)
     {

Spacing +3 added lines, -3 removed lines

@@ -73,7 +73,7 @@ 
             // Repair bad hierarchy.
             if ($object->hasMaster() && $object->getMaster() === $object->id()) {
                 $object->setMaster(0);
-                $object->update([ 'master' ]);
+                $object->update(['master']);
             }
 
             if ($object->hasMaster()) {
@@ -112,7 +112,7 @@ 
                 }
             }
         } else {
-            $start = (( $pageNum - 1 ) * $perPage);
+            $start = (($pageNum - 1) * $perPage);
             $end   = ($start + $perPage);
 
             foreach ($this->objects as $object) {
@@ -225,7 +225,7 @@ 
                 }
             }
         } else {
-            $start = (( $pageNum - 1 ) * $perPage);
+            $start = (($pageNum - 1) * $perPage);
             $end   = ($start + $perPage);
 
             foreach ($childObjects[$parentObj->id()] as $object) {

src/Charcoal/Object/ObjectSchedule.php

Doc Comments +1 added lines, -1 removed lines

@@ -259,7 +259,7 @@
     /**
      * Set the date/time the item was processed at.
      *
-     * @param  null|string|DateTimeInterface $ts A date/time string or object.
+     * @param  string $ts A date/time string or object.
      * @throws InvalidArgumentException If the date/time is invalid.
      * @return ObjectScheduleInterface Chainable
      */

Spacing +1 added lines, -1 removed lines

@@ -371,7 +371,7 @@
         if ($update) {
             $this->setProcessed(true);
             $this->setProcessedDate('now');
-            $this->update([ 'processed', 'processed_date' ]);
+            $this->update(['processed', 'processed_date']);
 
             if ($successCallback !== null) {
                 $successCallback($this);

src/Charcoal/Object/UserData.php

Doc Comments +1 added lines, -1 removed lines

@@ -195,7 +195,7 @@
     /**
      * Set when the object was created.
      *
-     * @param  DateTimeInterface|string|null $timestamp The timestamp at object's creation.
+     * @param  string $timestamp The timestamp at object's creation.
      *     NULL is accepted and instances of DateTimeInterface are recommended;
      *     any other value will be converted (if possible) into one.
      * @throws InvalidArgumentException If the timestamp is invalid.

src/Charcoal/Object/PublishableTrait.php

Spacing +1 added lines, -1 removed lines

@@ -197,7 +197,7 @@
         $now = new DateTime();
         $publish = $this->getPublishDate();
         $expiry  = $this->getExpiryDate();
-        $status  = $this->getPublishStatus()        ;
+        $status  = $this->getPublishStatus();
 
         if ($status !== static::STATUS_PUBLISHED) {
             return $status;

src/Charcoal/Object/ObjectRevision.php

Doc Comments +1 added lines, -1 removed lines

@@ -158,7 +158,7 @@
     }
 
     /**
-     * @param  mixed $revTs The revision's timestamp.
+     * @param  string $revTs The revision's timestamp.
      * @throws InvalidArgumentException If the timestamp is invalid.
      * @return ObjectRevision Chainable
      */

Unused Use Statements -1 removed lines

@@ -10,7 +10,6 @@
 use Pimple\Container;
 
 // From 'charcoal-factory'
-use Charcoal\Factory\FactoryInterface;
 
 // From 'charcoal-core'
 use Charcoal\Model\AbstractModel;

src/Charcoal/Object/RevisionableTrait.php

Doc Comments +2 added lines, -2 removed lines

@@ -33,7 +33,7 @@ 
 
     /**
      * @param  boolean $enabled The (revision) enabled flag.
-     * @return RevisionableInterface Chainable
+     * @return RevisionableTrait Chainable
      */
     public function setRevisionEnabled($enabled)
     {
@@ -66,7 +66,7 @@ 
      *
      * @param  string $className The class name of the object revision model.
      * @throws InvalidArgumentException If the class name is not a string.
-     * @return AbstractPropertyDisplay Chainable
+     * @return RevisionableTrait Chainable
      */
     protected function setObjectRevisionClass($className)
     {

src/Charcoal/Object/HierarchicalTrait.php

Doc Comments +5 added lines, -5 removed lines

@@ -60,7 +60,7 @@ 
      *
      * The object's hierarchy can be rebuilt with {@see self::hierarchy()}.
      *
-     * @return HierarchicalInterface Chainable
+     * @return HierarchicalTrait Chainable
      */
     public function resetHierarchy()
     {
@@ -73,7 +73,7 @@ 
      * Set this object's immediate parent.
      *
      * @param mixed $master The object's parent (or master).
-     * @return HierarchicalInterface Chainable
+     * @return HierarchicalTrait Chainable
      */
     public function setMaster($master)
     {
@@ -292,7 +292,7 @@ 
 
     /**
      * @param array $children The children to set.
-     * @return HierarchicalInterface Chainable
+     * @return HierarchicalTrait Chainable
      */
     public function setChildren(array $children)
     {
@@ -306,7 +306,7 @@ 
 
     /**
      * @param mixed $child The child object (or ident) to add.
-     * @return HierarchicalInterface Chainable
+     * @return HierarchicalTrait Chainable
      * @throws UnexpectedValueException The current object cannot be its own child.
      */
     public function addChild($child)
@@ -511,7 +511,7 @@ 
      * Add an object to the cache store.
      *
      * @param ModelInterface $obj The object to store.
-     * @return HierarchicalInterface Chainable
+     * @return HierarchicalTrait Chainable
      */
     private function addObjectToCache(ModelInterface $obj)
     {