Error converting datefield value about mongodb-odm

[avalanche123]

Interesting issue
I would suggest to fix it with casting:

public function convertToDatabaseValue($value)
{
    if ($value instanceof \DateTime) {
        $value = $value->getTimestamp();
    }
    if (is_string($value)) {
        $value = strtotime($value);
    }
    return new \MongoDate((int) $value);
}

Jon, what do you think?

[jseverson]

In your fix it seems like you change behavior though, now we ALWAYS get a mongodate where it seems like the old behavior we actually wanted to return null whenever $value was null (or $value was an invalid date)

[avalanche123]

Yeah, but I assume that's what you get on Unix systems anyway, right?

[jseverson]

dunno, i'm on windows. :) i havent pushed this to our staging server and tested there yet. I've issued a pull request with my fix in it. feel free to adjust it to your code which also fixes the problem for me, but does actually change behavior. Before saving the form field the date shows as 1890 (because the date is empty in the database), however, after submitting the form the date has changed to 1970, so your change actually will change the date field value from null to 1970.

[avalanche123]

Yes, but unix timestamp starts somewhere in 1969. And I think if the value returned was null, I don't understand how your form still shows the correct value...

[jwage]

This is an old issue I think. It is fixed now. If it is a date before the epoch we have to store the date as a string formatted like this:

        $date = new \DateTime($value);
        return $date->format('c');