revert PR 21214 (#21255)

Author: themsaid

src/Illuminate/Support/Collection.php

@@ -1348,7 +1348,7 @@ public function sortBy($callback, $options = SORT_REGULAR, $descending = false)
         // function which we were given. Then, we will sort the returned values and
         // and grab the corresponding values for the sorted keys from this array.
         foreach ($this->items as $key => $value) {
-            $results[$key] = [$callback($value, $key), $key];
+            $results[$key] = $callback($value, $key);
         }
 
         $descending ? arsort($results, $options)

tests/Support/SupportCollectionTest.php

@@ -798,6 +798,11 @@ public function testSortByString()
         $data = $data->sortBy('name');
 
         $this->assertEquals([['name' => 'dayle'], ['name' => 'taylor']], array_values($data->all()));
+
+        $data = new Collection([['name' => 'taylor'], ['name' => 'dayle']]);
+        $data = $data->sortBy('name', SORT_STRING);
+
+        $this->assertEquals([['name' => 'dayle'], ['name' => 'taylor']], array_values($data->all()));
     }
 
     public function testSortByAlwaysReturnsAssoc()
@@ -817,15 +822,6 @@ public function testSortByAlwaysReturnsAssoc()
         $this->assertEquals([1 => 'dayle', 0 => 'taylor'], $data->all());
     }
 
-    public function testSortByMaintainsOriginalOrderOfItemsWithIdenticalValues()
-    {
-        $data = new Collection([['name' => 'taylor'], ['name' => 'dayle'], ['name' => 'dayle']]);
-        $data = $data->sortBy('name');
-
-        $this->assertEquals([['name' => 'dayle'], ['name' => 'dayle'], ['name' => 'taylor']], array_values($data->all()));
-        $this->assertEquals([1, 2, 0], $data->keys()->all());
-    }
-
     public function testReverse()
     {
         $data = new Collection(['zaeed', 'alan']);