Bug #85083

Validation error with DateTimeImmutable

Added by Mathias Brodala about 1 year ago. Updated 11 months ago.

Status:
Closed
Priority:
Must have
Category:
Extbase
Target version:
Start date:
2018-05-25
Due date:
% Done:

100%

TYPO3 Version:
9
PHP Version:
Tags:
Complexity:
Is Regression:
Sprint Focus:

Description

Having an object property typed as \DateTimeImmutable yields a validation error when using that object as Extbase controller argument:

The given subject was not a valid DateTime. Got: 'object'

This is caused by the DateTimeValidator which only checks for DateTime but not DateTimeImmutable. This should be changed to DateTimeInterface to cover both.

Associated revisions

Revision 92d0783f (diff)
Added by Mathias Brodala about 1 year ago

[BUGFIX] Properly validate DateTimeImmutable objects

Change-Id: I29159e835d362e3f0e06c2145e84c1119df152cf
Resolves: #85083
Releases: master, 8.7
Reviewed-on: https://review.typo3.org/57054
Tested-by: TYPO3com <>
Reviewed-by: Wouter Wolters <>
Tested-by: Wouter Wolters <>
Reviewed-by: Mathias Brodala <>
Tested-by: Mathias Brodala <>

Revision 39c72543 (diff)
Added by Mathias Brodala about 1 year ago

[BUGFIX] Properly validate DateTimeImmutable objects

Change-Id: I29159e835d362e3f0e06c2145e84c1119df152cf
Resolves: #85083
Releases: master, 8.7
Reviewed-on: https://review.typo3.org/57058
Tested-by: TYPO3com <>
Reviewed-by: Mathias Brodala <>
Tested-by: Mathias Brodala <>

History

#1 Updated by Gerrit Code Review about 1 year ago

  • Status changed from New to Under Review

Patch set 1 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/57054

#2 Updated by Gerrit Code Review about 1 year ago

Patch set 2 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/57054

#3 Updated by Gerrit Code Review about 1 year ago

Patch set 1 for branch TYPO3_8-7 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/57058

#4 Updated by Mathias Brodala about 1 year ago

  • Status changed from Under Review to Resolved
  • % Done changed from 0 to 100

#5 Updated by Benni Mack 11 months ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF