Skip to content

[WIP][Form] added support for different invalid messages for date, time. #4444

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

Closed
wants to merge 2 commits into from
Closed

[WIP][Form] added support for different invalid messages for date, time. #4444

wants to merge 2 commits into from

Conversation

gh0stonio
Copy link

Bug fix: yes
Feature addition: yes
Backwards compatibility break: no
Symfony2 tests pass: yes
Fixes the following tickets: #4359
Todo: DateTimeType, when used with the widget "text", currently
duplicates the invalid error when an invalid value is entered.
License of the code: MIT
Documentation PR: none

Bug fix: yes
Feature addition: yes
Backwards compatibility break: no
Symfony2 tests pass: yes
Fixes the following tickets: #4359
Todo: DateTimeType, when used with the widget "text", currently
duplicates the invalid error when an invalid value is entered.
License of the code: MIT
Documentation PR: none
@travisbot
Copy link

This pull request fails (merged 195361f into cc42a18).

@travisbot
Copy link

This pull request passes (merged 82f5ed2 into cc42a18).

@MattKetmo
Copy link
Contributor

What's the status of this PR? Is it still maintained?

@stof
Copy link
Member

stof commented Oct 13, 2012

@gh0stonio this change should be done in master, not in 2.0, as it is a new feature.

@fabpot
Copy link
Member

fabpot commented Dec 11, 2012

@bschussek Is this PR still needed in master? Can it be merged there?

@webmozart
Copy link
Contributor

I prefer that generic date_options and time_options are implemented, where the invalid_message option can be set. See #3825.

@webmozart webmozart closed this Dec 13, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants