[5.x] Handle collection instances in first modifier (#11608)

Author: marcorieser

src/Modifiers/CoreModifiers.php

@@ -738,6 +738,10 @@ public function first($value, $params)
             return Arr::first($value);
         }
 
+        if ($value instanceof Collection) {
+            return $value->first();
+        }
+
         return Stringy::first($value, Arr::get($params, 0));
     }
 

tests/Modifiers/FirstTest.php

@@ -45,6 +45,27 @@ public static function arrayProvider()
         ];
     }
 
+    #[Test]
+    #[DataProvider('collectionProvider')]
+    public function it_gets_the_first_value_of_a_collection($value, $expected)
+    {
+        $this->assertEquals($expected, $this->modify($value));
+    }
+
+    public static function collectionProvider()
+    {
+        return [
+            'list' => [
+                collect(['alfa', 'bravo', 'charlie']),
+                'alfa',
+            ],
+            'associative' => [
+                collect(['alfa' => 'bravo', 'charlie' => 'delta']),
+                'bravo',
+            ],
+        ];
+    }
+
     private function modify($value, $arg = null)
     {
         return Modify::value($value)->first($arg)->fetch();