Skip to content

Commit

Permalink
Fix: allow explicit 0 secfracs in datetime format
Browse files Browse the repository at this point in the history
  • Loading branch information
vaeryn-uk committed Feb 18, 2016
1 parent 37810e9 commit b6ce179
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 1 deletion.
2 changes: 1 addition & 1 deletion src/JsonSchema/Constraints/FormatConstraint.php
Original file line number Diff line number Diff line change
Expand Up @@ -138,7 +138,7 @@ protected function validateDateTime($datetime, $format)
// which will fail the above string comparison because the passed
// $datetime may be '2000-05-01T12:12:12.123Z' but format() will return
// '2000-05-01T12:12:12.123000Z'
if ((strpos('u', $format) !== -1) && (intval($dt->format('u')) > 0)) {
if ((strpos('u', $format) !== -1) && (preg_match('/\.\d+Z/', $datetime))) {
return true;
}

Expand Down
5 changes: 5 additions & 0 deletions tests/JsonSchema/Tests/Constraints/FormatTest.php
Original file line number Diff line number Diff line change
Expand Up @@ -81,6 +81,10 @@ public function getValidFormats()
array('2000-05-01T12:12:12+01:00', 'date-time'),
array('2000-05-01T12:12:12.123456Z', 'date-time'),
array('2000-05-01T12:12:12.123Z', 'date-time'),
array('2000-05-01T12:12:12.123000Z', 'date-time'),
array('2000-05-01T12:12:12.0Z', 'date-time'),
array('2000-05-01T12:12:12.000Z', 'date-time'),
array('2000-05-01T12:12:12.000000Z', 'date-time'),

array('0', 'utc-millisec'),

Expand Down Expand Up @@ -140,6 +144,7 @@ public function getInvalidFormats()
array('1999-1-11T00:00:00Z', 'date-time'),
array('1999-01-11T00:00:00+100', 'date-time'),
array('1999-01-11T00:00:00+1:00', 'date-time'),
array('1999.000Z-01-11T00:00:00+1:00', 'date-time'),

array('-1', 'utc-millisec'),
array(PHP_INT_MAX, 'utc-millisec'),
Expand Down

0 comments on commit b6ce179

Please sign in to comment.