Added SensioLabs config. Fixed tiny issues suggested by SensioLabs.

Athari · Oct 30, 2015 · d3c5019 · d3c5019
1 parent 7721aa1
commit d3c5019
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 4 deletions.
diff --git a/.sensiolabs.yml b/.sensiolabs.yml
@@ -0,0 +1,3 @@
+rules:
+    missing_e_o_l:
+        enabled: false
diff --git a/YaLinqo/Errors.php b/YaLinqo/Errors.php
@@ -2,8 +2,6 @@
 
 namespace YaLinqo;
 
-use YaLinqo;
-
 class Errors
 {
     /** Error message: "Sequence contains no elements." */

diff --git a/YaLinqo/OrderedEnumerable.php b/YaLinqo/OrderedEnumerable.php
@@ -154,7 +154,7 @@ private function trySortBySingleField ($array, $canMultisort)
     private function sortByMultipleFields ($array, $canMultisort)
     {
         $orders = [ ];
-        for ($order = $this; $order != null; $order = $order->parent) {
+        for ($order = $this; $order !== null; $order = $order->parent) {
             $orders[] = $order;
             if ($order->sortFlags === null)
                 $canMultisort = false;
@@ -240,7 +240,8 @@ private function sortIteratorWithUsort (&$enum, $orders)
 
         usort($enum, function ($a, $b) use ($orders) {
             /** @var $order OrderedEnumerable */
-            for ($i = 0; $i < count($orders); $i++) {
+            $count = count($orders);
+            for ($i = 0; $i < $count; $i++) {
                 $order = $orders[$i];
                 $comparer = $order->comparer;
                 $diff = $comparer($a[$i + 2], $b[$i + 2]);