Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Makes the Converter respect stange date formats #572

Closed
wants to merge 1 commit into from

Conversation

tim-massey
Copy link

The dates I store in elastic are formatted as follows:

/**
  * @ES\Property(type="date", options={"format": "Y-M-D'T'HH:mm:ss"})
  */
  public $start;

Before this change they are Converted as false.

After this change, they are correctly converted into \DateTime objects.

@saimaz
Copy link
Member

saimaz commented Mar 1, 2016

Can you cover the bug you found with the test case?

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

Successfully merging this pull request may close these issues.

None yet

2 participants