Skip to content

[Translation] TransChoice invalide plural translation #11425

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 1 commit into from

Conversation

Simperfit
Copy link
Contributor

@Simperfit Simperfit commented Apr 15, 2019

This is linked to the new feature at symfony/symfony#31110

@Simperfit
Copy link
Contributor Author

Needs update after code review ping @Simperfit

@Simperfit
Copy link
Contributor Author

Status: Needs Work

@@ -230,11 +230,18 @@ method or the ``transchoice`` tag/filter in your :ref:`template <translation-tag
For much more information, see :ref:`component-translation-pluralization`
in the Translation component documentation.


Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This blank line should be removed

.. deprecated:: 4.2

In Symfony 4.2 the ``Translator::transChoice()`` method was deprecated in
favor of using ``Translator::trans()`` with ``%count%`` as the parameter
driving plurals.


Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This blank line should be removed

@maxhelias
Copy link
Contributor

maxhelias commented Aug 8, 2019

This PR is outdated in view of the latest update 😉

@HeahDude
Copy link
Contributor

HeahDude commented Feb 7, 2020

Closing here as the related PR has been closed. Thanks!

@HeahDude HeahDude closed this Feb 7, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Status: Needs Work Translation Waiting Code Merge Docs for features pending to be merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants