Update tests for normalized timezones in VCP dates

No longer normalizes timezone for non-VCP dates.
Additional test and comments on each test.

Author: gRegorLove

Mf2/Parser.php

@@ -244,13 +244,13 @@ function convertTimeFormat($time) {
  * @return string isolated, normalized TZ offset for implied TZ for other dt- properties
  */
 function normalizeTimezoneOffset(&$dtValue) {
-	preg_match('/Z|[+-]\d{1,2}:?(\d{2})?$/i', $dtValue, $matches);	
+	preg_match('/Z|[+-]\d{1,2}:?(\d{2})?$/i', $dtValue, $matches);
 
 	if (empty($matches)) {
 		return null;
 	}
 
-  $timezoneOffset = null;
+	$timezoneOffset = null;
 
 	if ( $matches[0] != 'Z' ) {
 		$timezoneString = str_replace(':', '', $matches[0]);

tests/Mf2/ParseDTTest.php

@@ -260,9 +260,11 @@ public function testDTStartDateOnly() {
 	}
 
 	/**
-	 * 
+	 * TZ offsets normalized only for VCP.
+	 * This behavior is implied from "However the colons ":" separating the hours and minutes of any timezone offset are optional and discouraged in order to make it less likely that a timezone offset will be confused for a time."
+	 * @see http://microformats.org/wiki/index.php?title=value-class-pattern&oldid=66473##However+the+colons
 	 */
-	public function testNormalizeTZOffset() {
+	public function testNormalizeTZOffsetVCP() {
 		$input = '<div class="h-event">
             <span class="dt-start"> <time class="value" datetime="2017-05-27">May 27</time>, from
             <time class="value">20:57-07:00</time> </span>
@@ -273,6 +275,19 @@ public function testNormalizeTZOffset() {
 		$this->assertEquals('2017-05-27 20:57-0700', $output['items'][0]['properties']['start'][0]);
 	}
 
+
+	/**
+	 * TZ offsets *not* normalized for non-VCP dates
+	 */
+	public function testNoNormalizeTZOffset() {
+		$input = '<div class="h-entry"> <time class="dt-start" datetime="2018-03-13 15:30-07:00">March 13, 2018 3:30PM</time> </div>';
+		$parser = new Parser($input);
+		$output = $parser->parse();
+
+		$this->assertEquals('2018-03-13 15:30-07:00', $output['items'][0]['properties']['start'][0]);
+	}
+
+
 	/**
 	 * @see https://github.com/indieweb/php-mf2/issues/115
 	 */
@@ -297,7 +312,7 @@ public function testPreserrveTIfAuthored() {
 		$parser = new Parser($input);
 		$output = $parser->parse();
 
-		$this->assertEquals('2009-06-26T19:01-0800', $output['items'][0]['properties']['start'][0]);
+		$this->assertEquals('2009-06-26T19:01-08:00', $output['items'][0]['properties']['start'][0]);
 	}
 
 	/**