mblarsen / arrgh

src/Arrgh.php

Upper-Lower-Casing +1 added lines, -1 removed lines

@@ -240,7 +240,7 @@
         }
 
         if ($matching_function === null) {
-            throw new InvalidArgumentException("Method {$method} doesn't exist");
+            throw new InvalidArgumentException("method {$method} doesn't exist");
         }
 
         // asort in PHP5 reverses equals ("arsort" doen't mess up for some reason)

Doc Comments +19 added lines, -1 removed lines

@@ -210,6 +210,10 @@ 
     }
 
     /* Wraps a callable with the purpose of fixing bad PHP sort implementations */
+
+    /**
+     * @param Closure $callable
+     */
     private static function wrapCallable($callable)
     {
         $direction = self::getSortDirection();
@@ -250,6 +254,10 @@ 
     }
 
     /* Transforms the incoming calls to native calls */
+
+    /**
+     * @param Arrgh $object
+     */
     private static function invoke($method, $args, $object = null)
     {
         self::getSortDirection();
@@ -314,6 +322,11 @@ 
     }
 
     /* Handles special case: asort - In PHP5 reverses equals ("arsort" doen't mess up for some reason) */
+
+    /**
+     * @param string|null $matching_handler
+     * @param string $matching_function
+     */
     private static function handleCaseAsort(&$matching_handler, &$matching_function, &$post_handler, &$args)
     {
         $matching_function = "uasort";
@@ -323,6 +336,11 @@ 
     /* Handles special case: array_column - Native array_column filters away null values.
      * That means you cannot use array_column for multisort since array size no longer matches.
      * This version of array_column returns null if the column is missing. */
+
+    /**
+     * @param string|null $matching_handler
+     * @param string $matching_function
+     */
     private static function handleCaseArrayColumn(&$matching_handler, &$matching_function, &$post_handler, &$args)
     {
         $matching_handler  = "_rotateRight";
@@ -679,7 +697,7 @@ 
      * Partion the input based on the result of the callback function.
      *
      * @param array    $array    A collection
-     * @param Closeure $callable A callable returning true or false depending on which way to partion the element—left or right.
+     * @param Closure $callable A callable returning true or false depending on which way to partion the element—left or right.
      * @return array An array with two arrays—left and right: [left, right]
      */
     private static function arr_partition($array, $callable)

Spacing +3 added lines, -3 removed lines

@@ -327,7 +327,7 @@ 
     {
         $matching_handler  = "_rotateRight";
         $matching_function = "array_map";
-        $column_id    =  null;
+        $column_id    = null;
         $column_array = $args[0];
         $column_key   = $args[1];
         if (count($args) === 3) {
@@ -471,7 +471,7 @@ 
         }
 
         $is_collection = self::arr_is_collection($array);
-        $array = $is_collection ? $array : [ $array ];
+        $array = $is_collection ? $array : [$array];
 
         $result = array_map(function($item) use ($except) {
             foreach ($except as $key) {
@@ -493,7 +493,7 @@ 
         }
 
         $is_collection = self::arr_is_collection($array);
-        $array = $is_collection ? $array : [ $array ];
+        $array = $is_collection ? $array : [$array];
 
         $result = array_map(function($item) use ($only) {
             foreach ($item as $key => $value) {

Braces +9 added lines, -3 removed lines

@@ -215,7 +215,9 @@ 
         $direction = self::getSortDirection();
         return function($a, $b) use ($direction, $callable) {
             $result = $callable($a, $b);
-            if ($result === 0) return $direction;
+            if ($result === 0) {
+                return $direction;
+            }
             return $result;
         };
     }
@@ -663,8 +665,12 @@ 
      */
     private static function arr_depth($array)
     {
-        if (empty($array) && is_array($array)) return 0;
-        if (!self::arr_is_collection($array)) return null;
+        if (empty($array) && is_array($array)) {
+            return 0;
+        }
+        if (!self::arr_is_collection($array)) {
+            return null;
+        }
 
         $depth = 0;
         $child = array_shift($array);