Denormalization of empty string date throws UnexpectedValueException

Think the change from #9 should be included.

Outside of javascript files not sure we use === ""

Tagging for novice as it should be an easy update.

Let me work on this.

Worked on #24 and rerolled the patch and created PR.

Change looks good.

Verified locally that 1 test fails without the fix. Data set "Empty string" throws Symfony\Component\Serializer\Exception\UnexpectedValueException : The specified date "" is not in an accepted format: "Y-m-d\TH:i:sP" (RFC 3339), "Y-m-d\TH:i:sO" (ISO 8601).

Somehow we ended back with the #9 approach, which as pointed out (albeit rudely) in #12 is wrong, because 0 is a valid unix timestamp.

Can we go back to #20 please.

Fix the failed Patch For 10.1 version.

I think we still need a test-case for 0 as the patch didn't fail without it.

That probably means we need to expand the test method to accept an 'expected' denormalized value, which would default to the original value.

@Prem Suthar please be sure to add an interdiff → when working with patches so reviewers can know what changed.

I’m triaging for DrupalCon mentored contribution and reviewing as this issue is tagged novice.

*******
Please leave this issue for DrupalCon mentored contribution as a possible first time issue.

*******
Next step is to update the test per the most recent comment.

Tag normalization happening:)