8000 [3.0] [Form] Extract FloatToLocalizedStringTransformer · Issue #8631 · symfony/symfony · GitHub
[go: up one dir, main page]
Skip to content

[3.0] [Form] Extract FloatToLocalizedStringTransformer #8631

New issue
B8D6

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

Sign up for GitHub

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

Jump to bottom
Closed
webmozart opened this issue Aug 1, 2013 · 0 comments
Closed

[3.0] [Form] Extract FloatToLocalizedStringTransformer #8631

webmozart opened this issue Aug 1, 2013 · 0 comments
Labels
Enhancement Form

Comments

@webmozart
Copy link
Contributor

Currently, NumberToLocalizedStringTransformer has a parameter precision, which does not make sense for all numbers, such as integers (and consequently in the subclass IntegerToLocalizedStringTransformer).

In Symfony 3.0, the precision parameter should be split away into a dedicated subclass.

See also #8473.
@webmozart webmozart changed the title [3.0][Form] Extract FloatToLocalizedStringTransformer [3.0] [Form] Extract FloatToLocalizedStringTransformer Oct 16, 2014
@xabbuh xabbuh mentioned this issue Sep 23, 2018
Merged
@fabpot fabpot closed this as completed Sep 24, 2018
fabpot added a commit that referenced this issue Sep 24, 2018
@fabpot
feature #28569 [Form] deprecate precision in IntegerToLocalizedString…
31f8cb9 
…Transformer (xabbuh)

This PR was merged into the 4.2-dev branch.

Discussion
----------

[Form] deprecate precision in IntegerToLocalizedStringTransformer

| Q             | A
| ------------- | ---
| Branch?       | master
| Bug fix?      | no
| New feature?  | no
| BC breaks?    | no
| Deprecations? | yes
| Tests pass?   | yes
| Fixed tickets | #8631
| License       | MIT
| Doc PR        |

Commits
-------

65362e0 deprecate precision in IntegerToLocalizedStringTransformer
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Enhancement Form
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@fabpot @webmozart
0