Update YAML to process timestamp string timezone

noahheck · noahheck · commit 92e5b6d96e71 · 2016-11-17T08:00:37.000-07:00
Parse date strings containing timezone data correctly
Default date strings not containing timezone data to UTC
diff --git a/src/Symfony/Component/Yaml/Inline.php b/src/Symfony/Component/Yaml/Inline.php
@@ -604,11 +604,18 @@ private static function evaluateScalar($scalar, $flags, $references = array())
                         return self::evaluateBinaryScalar(substr($scalar, 9));
                     case preg_match('/^(-|\+)?[0-9,]+(\.[0-9]+)?$/', $scalar):
                         return (float) str_replace(',', '', $scalar);
-                    case preg_match(self::getTimestampRegex(), $scalar):
+                    case preg_match(self::getTimestampRegex(), $scalar, $datetimeMatches):
                         if (Yaml::PARSE_DATETIME & $flags) {
-                            $date = new \DateTime($scalar);
-                            $date->setTimeZone(new \DateTimeZone('UTC'));
 
+                            // Timestamps that have timezone data should be parsed in the specified timezone first, then
+                            // converted to UTC; if no tz data is supplied, parse as UTC date initially
+                            if (isset($datetimeMatches['tz'])) {
+                                $date = new \DateTime($scalar);
+                                $date->setTimeZone(new \DateTimeZone('UTC'));
+                                return $date;
+                            }
+
+                            $date = new \DateTime($scalar, new \DateTimeZone('UTC'));
                             return $date;
                         }
 
diff --git a/src/Symfony/Component/Yaml/Tests/InlineTest.php b/src/Symfony/Component/Yaml/Tests/InlineTest.php
@@ -535,7 +535,11 @@ public function testParseNestedTimestampListAsDateTimeObject($yaml, $year, $mont
         $expected = new \DateTime($yaml);
         $expected->setTimeZone(new \DateTimeZone('UTC'));
         $expected->setDate($year, $month, $day);
-        @$expected->setTime($hour, $minute, $second, 1000000 * ($second - (int) $second));
+        if (PHP_VERSION_ID >= 70100) {
+            $expected->setTime($hour, $minute, $second, 1000000 * ($second - (int) $second));
+        } else {
+            $expected->setTime($hour, $minute, $second);
+        }
 
         $expectedNested = array('nested' => array($expected));
         $yamlNested = "{nested: [$yaml]}";
