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)

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

Doc Comments +19 added lines, -1 removed lines

@@ -250,6 +250,10 @@ 
     }
 
     /* Transforms the incoming calls to native calls */
+
+    /**
+     * @param Arrgh $object
+     */
     private static function invoke($method, $args, $object = null)
     {
         self::getSortDirection();
@@ -318,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";
@@ -327,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";
@@ -633,6 +647,10 @@ 
     }
 
     /* arr_get: Find next node by index */
+
+    /**
+     * @param integer $plug_index
+     */
     private static function _arr_get_traverse_next_node_index($data, $plug_index)
     {
         // Adjust negative index
@@ -708,7 +726,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, Closure $callable)