Merge pull request #15 from bluemorbo/fix-entities-with-scalar-arrays

Fix handling of scalar arrays

src/Entity.php

@@ -66,7 +66,11 @@ private function findSubEntities($props)
 
             if (is_array($value)) {
                 $newProps[$name] = array_map(function ($item) {
-                    return new Entity($item);
+                    if (is_object($item) || is_array($item)) {
+                        return new Entity($item);
+                    }
+
+                    return $item;
                 }, $value);
                 continue;
             }

tests/ClientTest.php

@@ -505,7 +505,7 @@ public function can_send_query_params_as_array_for_delete_requests()
         $this->assertEquals('id:in=1,2,3', urldecode($uri->getQuery()));
     }
 
-    protected function writeFunctionProvider()
+    public function writeFunctionProvider()
     {
         return [
             ['create'],

tests/EntityTest.php

@@ -64,6 +64,17 @@ public function can_serialize_nested_arrays()
         $this->assertEquals('Jane', $this->entity->contacts[1]->name);
     }
 
+    /**
+     * @test
+     */
+    public function does_not_serialize_nested_scalar_values()
+    {
+        $this->loadSampleEntity();
+
+        $this->assertSame("Mr Jack Doe", $this->entity->aliases[0]);
+        $this->assertSame("Mr J Doe", $this->entity->aliases[1]);
+    }
+
     protected function loadSampleEntity()
     {
         $this->entity = new Entity([
@@ -82,6 +93,10 @@ protected function loadSampleEntity()
                     'name' => 'Jane',
                     'email' => '[email protected]',
                 ]
+            ],
+            'aliases' => [
+                'Mr Jack Doe',
+                'Mr J Doe'
             ]
         ]);
     }