Added tests to the Native mapper

Author: Scott Pringle

lib/PHPExif/Mapper/Native.php

@@ -13,6 +13,7 @@
 
 use PHPExif\Exif;
 use DateTime;
+use Exception;
 
 /**
  * PHP Exif Native Mapper
@@ -142,7 +143,11 @@ public function mapRawData(array $data)
             // manipulate the value if necessary
             switch ($field) {
                 case self::DATETIMEORIGINAL:
-                    $value = DateTime::createFromFormat('Y:m:d H:i:s', $value);
+                    try {
+                        $value = new DateTime($value);
+                    } catch (Exception $exception) {
+                        continue 2;
+                    }
                     break;
                 case self::EXPOSURETIME:
                     // normalize ExposureTime

tests/PHPExif/Mapper/NativeMapperTest.php

@@ -187,4 +187,39 @@ public function testMapRawDataCorrectlyFormatsGPSData()
         $expected = '40.333452380952,-20.167314814815';
         $this->assertEquals($expected, reset($result));
     }
+
+    public function testMapRawDataCorrectlyFormatsDifferentDateTimeString()
+    {
+        $rawData = array(
+            \PHPExif\Mapper\Native::DATETIMEORIGINAL => '2014-12-15 00:12:00'
+        );
+
+        $mapped = $this->mapper->mapRawData(
+            $rawData
+        );
+
+        $result = reset($mapped);
+        $this->assertInstanceOf('\DateTime', $result);
+        $this->assertEquals(
+            reset($rawData),
+            $result->format("Y-m-d H:i:s")
+        );
+    }
+
+    public function testMapRawDataCorrectlyIgnoresInvalidCreateDate()
+    {
+        $rawData = array(
+            \PHPExif\Mapper\Native::DATETIMEORIGINAL => 'Invalid Date String'
+        );
+
+        $result = $this->mapper->mapRawData(
+            $rawData
+        );
+
+        $this->assertCount(0, $result);
+        $this->assertNotEquals(
+            reset($rawData),
+            $result
+        );
+    }
 }