Merge pull request #57 from jonhudson/master

File validation fails when file contains sub-arrays

Author: weierophinney

src/ServerRequestFactory.php

@@ -469,6 +469,7 @@ private static function createUploadedFileFromSpec(array $value)
      */
     private static function normalizeNestedFileSpec(array $files = [])
     {
+        $normalizedFiles = [];
         foreach (array_keys($files['tmp_name']) as $key) {
             $spec = [
                 'tmp_name' => $files['tmp_name'][$key],
@@ -477,8 +478,8 @@ private static function normalizeNestedFileSpec(array $files = [])
                 'name'     => $files['name'][$key],
                 'type'     => $files['type'][$key],
             ];
-            $files[$key] = self::createUploadedFileFromSpec($spec);
+            $normalizedFiles[$key] = self::createUploadedFileFromSpec($spec);
         }
-        return $files;
+        return $normalizedFiles;
     }
 }

test/ServerRequestFactoryTest.php

@@ -415,4 +415,19 @@ public function testNormalizeServerReturnsArrayUnalteredIfApacheHeadersDoNotCont
 
         $this->assertEquals($expected, $server);
     }
+
+    public function testNormalizeFilesReturnsOnlyActualFilesWhenOriginalFilesContainsNestedArrays()
+    {
+        $files = [ 'fooFiles' => [
+            'tmp_name' => ['file' => 'php://temp'],
+            'size'     => ['file' => 0],
+            'error'    => ['file' => 0],
+            'name'     => ['file' => 'foo.bar'],
+            'type'     => ['file' => 'text/plain'],
+        ]];
+
+        $normalizedFiles = ServerRequestFactory::normalizeFiles($files);
+
+        $this->assertCount(1, $normalizedFiles['fooFiles']);
+    }
 }