Fix key handling bug in KeyWalker

CHANGELOG.md

@@ -1,6 +1,11 @@
 Change Log
 ==========
 
+2020-07-16 - 3.3.2
+------------------
+
+ * Fix explode() requires a string and will fail if given an int (in KeyWalker)
+
 2020-04-23 - 3.3.1
 ------------------
 

composer.json

@@ -20,7 +20,7 @@
     },
     "require-dev": {
         "phpunit/phpunit": "~8.2",
-        "symfony/var-dumper": "~4.3",
+        "symfony/var-dumper": "~5",
         "victorjonsson/markdowndocs": "^1.3"
     },
     "autoload": {

src/Utils/KeyWalker.php

@@ -44,7 +44,7 @@ public static function get($collection, $key, $default = null)
             return $collection[$key];
         }
 
-        $key = is_array($key) ? $key : explode('.', $key);
+        $key = is_array($key) ? $key : explode('.', (string)$key);
 
         while (!is_null($segment = array_shift($key))) {
             if ($segment === '*') {
@@ -92,7 +92,7 @@ public static function has($collection, $key): bool
             return true;
         }
 
-        $key = is_array($key) ? $key : explode('.', $key);
+        $key = is_array($key) ? $key : explode('.', (string)$key);
 
         while (!is_null($segment = array_shift($key))) {
             if ($segment === '*') {
@@ -163,7 +163,7 @@ public static function extract($collection, $value, $key = null, $default = null
     protected static function extractKeyValueParameters($value, $key): array
     {
         $value = is_string($value) ? explode('.', $value) : $value;
-        $key   = is_null($key) || is_array($key) ? $key : explode('.', $key);
+        $key   = is_null($key) || is_array($key) ? $key : explode('.', (string)$key);
 
         return [$value, $key];
     }

tests/Behaviours/Query/GetTest.php

@@ -30,6 +30,22 @@ public function testGet()
         $this->assertNull($col->get('abe'));
     }
 
+    /**
+     * @group accessors
+     */
+    public function testGetByInt()
+    {
+        $col = new Collection([
+            'test1',
+            'test2',
+            'test3',
+            'test4',
+        ]);
+
+        $this->assertEquals('test3', $col->get(2));
+        $this->assertNull($col->get(1234));
+    }
+
     /**
      * @group accessors
      */

tests/Behaviours/Query/HasTest.php

@@ -32,6 +32,22 @@ public function testHas()
         $this->assertFalse($col->has('abe'));
     }
 
+    /**
+     * @group collection
+     */
+    public function testHasByInt()
+    {
+        $col = new Collection([
+            'test-1',
+            'test-2',
+            'test-abc',
+            'test-abe',
+        ]);
+
+        $this->assertTrue($col->has(3));
+        $this->assertFalse($col->has(678));
+    }
+
     /**
      * @group collection
      */