Skip to content

Conversation

deguif
Copy link
Contributor

@deguif deguif commented Apr 6, 2019

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

When working on #30909 I encountered this bug.
Currently the tests were passing, because the po fixture file contained only one translation.

@deguif deguif changed the base branch from master to 3.4 April 6, 2019 17:08
@fabpot
Copy link
Member

fabpot commented Apr 6, 2019

Thank you @deguif.

@fabpot fabpot merged commit b54abfc into symfony:3.4 Apr 6, 2019
fabpot added a commit that referenced this pull request Apr 6, 2019
This PR was merged into the 3.4 branch.

Discussion
----------

[Translator] Fix wrong dump for PO files

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

When working on #30909 I encountered this bug.
Currently the tests were passing, because the po fixture file contained only one translation.

Commits
-------

b54abfc Fix wrong dump for PO files
@deguif deguif deleted the fix-po-loader branch April 6, 2019 18:47
@nicolas-grekas nicolas-grekas added this to the 3.4 milestone Apr 7, 2019
This was referenced Apr 16, 2019
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.

5 participants