Skip to content
This repository has been archived by the owner on Dec 1, 2021. It is now read-only.

Add support for property converters to findBy. #166

Open
TSchuermans opened this issue Nov 10, 2017 · 0 comments
Open

Add support for property converters to findBy. #166

TSchuermans opened this issue Nov 10, 2017 · 0 comments

Comments

@TSchuermans
Copy link
Contributor

If a property has a DateTime converter annotation, it should be possible to use a DateTime object in findBy.
AFAIK you can only use the unix timestamp or the long unix timestamp in findBy.

It would be useful to be able to do:
$repository->findBy(['property' => new \DateTime()])

This involves parsing the metadata of the class, so it might not be that optimal.
The other approach could be to convert \DateTime values to their unix timestamp, but then you risk a mismatch if the long timestamp format is used.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant