Conversation
Edhrendal
force-pushed
the
feat-datetime_collection
branch
2 times, most recently
from
f4af396 to
c3ce9f0
Compare
Edhrendal
force-pushed
the
feat-datetime_collection
branch
from
fb4f00b to
293f0d9
Compare
|
Refonte de la PR en prenant en compte les classes générées de la ✅ Tests unitaires effectués seulement sur les |
Edhrendal
force-pushed
the
feat-datetime_collection
branch
2 times, most recently
from
f9b3387 to
cb62abb
Compare
There was a problem hiding this comment.
pourquoi on n'utilise pas \DateTImeInterface dans ce trait, mais on l'utilise dans l'autre ?
There was a problem hiding this comment.
j'ai pas précisé ailleurs dans la lib quand on n'a que le typage demandé, c'est quand on autorise null que j'ai précisé
| trait ContainsDateTimeStrictValueTrait | |
| trait ContainsDateTimeValueTrait |
| { | ||
| foreach ($this->toArray() as $value) { | ||
| if ( | ||
| $value === null |
There was a problem hiding this comment.
je me demande à quel point on peut ne faire qu'un seul trait, qui gère tous les cas.
ici on pourrait faire un $value instanceof \DateTime, et ne plus se soucier du nullable ?
There was a problem hiding this comment.
C'est pour qu'une collection "stricte" n'ait pas besoin de faire une vérification complémentaire.
Edhrendal
force-pushed
the
feat-datetime_collection
branch
from
cb62abb to
6711335
Compare
* ✅ **Tested!** upon `\DateTime` * ✅ **Tested!** upon `\DateTimeImmutable` ✨ Features ========== * Define `containsDateTimeValue()` function for all `DateTimeInterfaceCollection` - Verify upon "date time + timezone" rather than object hash - Same for nullable `\DateTimeInterface`
Edhrendal
force-pushed
the
feat-datetime_collection
branch
from
6711335 to
766625b
Compare
2 participants
✅ Tested! upon
\DateTime✨ Features
containsDateTimeValue()function for allDateTimeInterfaceCollection\DateTimeInterface