Thanks to visit codestin.com
Credit goes to github.com

Skip to content

[Form] Remove DateTimeToStringTransformer $parseUsingPipe option #22911

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

Merged
merged 1 commit into from
May 25, 2017
Merged

[Form] Remove DateTimeToStringTransformer $parseUsingPipe option #22911

merged 1 commit into from
May 25, 2017

Conversation

ogizanagi
Copy link
Contributor

@ogizanagi ogizanagi commented May 25, 2017

Q A
Branch? 2.7
Bug fix? no
New feature? no
BC breaks? no
Deprecations? no
Tests pass? yes
Fixed tickets N/A
License MIT
Doc PR N/A

Looks like this part of the code should not have been used anymore since 2 years 😅 (6dc8979)

@nicolas-grekas
Copy link
Member

Good catch, thanks @ogizanagi.

@nicolas-grekas nicolas-grekas merged commit a841496 into symfony:2.7 May 25, 2017
nicolas-grekas added a commit that referenced this pull request May 25, 2017
…e option (ogizanagi)

This PR was merged into the 2.7 branch.

Discussion
----------

[Form] Remove DateTimeToStringTransformer $parseUsingPipe option

| Q             | A
| ------------- | ---
| Branch?       | 2.7 <!-- see comment below -->
| Bug fix?      | no
| New feature?  | no <!-- don't forget updating src/**/CHANGELOG.md files -->
| BC breaks?    | no
| Deprecations? | no <!-- don't forget updating UPGRADE-*.md files -->
| Tests pass?   | yes
| Fixed tickets | N/A <!-- #-prefixed issue number(s), if any -->
| License       | MIT
| Doc PR        | N/A

Looks like this part of the code should not have been used anymore since 2 years 😅 (6dc8979)

Commits
-------

a841496 [Form] Remove DateTimeToStringTransformer $parseUsingPipe option
@ogizanagi ogizanagi deleted the rm/form/date_parse_using_pipes branch May 26, 2017 08:34
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.

3 participants