Nayjest / Collection

src/Extended/ObjectCollectionTrait.php

Doc Comments +8 added lines, -4 removed lines

@@ -14,14 +14,14 @@ 
 
     /**
      * @param callable $callback
-     * @param array|null $optionalArguments
+     * @param string[] $optionalArguments
      * @return static
      */
     abstract public function find(callable $callback, array $optionalArguments = null);
 
     /**
      * @param callable $callback
-     * @param array|null $optionalArguments
+     * @param string[] $optionalArguments
      * @return static
      */
     abstract public function filter(callable $callback, array $optionalArguments = null);
@@ -35,7 +35,7 @@ 
     /**
      * @param string $className
      *
-     * @return mixed
+     * @return ObjectCollectionTrait
      */
     public function findByType($className)
     {
@@ -68,7 +68,7 @@ 
      * @param string $propertyName
      * @param $value
      * @param bool $useGetters
-     * @return mixed
+     * @return ObjectCollectionTrait
      */
     public function findByProperty($propertyName, $value, $useGetters = false)
     {
@@ -94,6 +94,10 @@ 
         });
     }
 
+    /**
+     * @param string $propertyName
+     * @param boolean $useGetters
+     */
     private function getPropertyComparator($propertyName, $value, $useGetters)
     {
         if ($useGetters) {

Spacing +4 added lines, -4 removed lines

@@ -84,7 +84,7 @@ 
     public function sortByProperty($propertyName)
     {
 
-        return $this->sort(function ($itemA, $itemB) use ($propertyName) {
+        return $this->sort(function($itemA, $itemB) use ($propertyName) {
             $a = mp\getValue($itemA, $propertyName, 0);
             $b = mp\getValue($itemB, $propertyName, 0);
             if ($a < $b) {
@@ -97,12 +97,12 @@ 
     private function getPropertyComparator($propertyName, $value, $useGetters)
     {
         if ($useGetters) {
-            return function ($item) use ($propertyName, $value) {
+            return function($item) use ($propertyName, $value) {
                 // NAN used as default because NAN !== NAN
                 return mp\getValue($item, $propertyName, NAN) === $value;
             };
         } else {
-            return function ($item) use ($propertyName, $value) {
+            return function($item) use ($propertyName, $value) {
                 return isset($item->{$propertyName}) && $item->{$propertyName} === $value;
             };
         }
@@ -118,7 +118,7 @@ 
     public function indexByProperty($propertyName)
     {
         $results = [];
-        foreach($this as $item) {
+        foreach ($this as $item) {
             $key = mp\getValue($item, $propertyName);
             if ($key) {
                 $results[$key] = $item;