10000 [Translator] Fix Empty translations with Qt files by vlefort · Pull Request #9952 · symfony/symfony · GitHub
[go: up one dir, main page]

Skip to content

[Translator] Fix Empty translations with Qt files #9952

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

vlefort
Copy link
Contributor
@vlefort vlefort commented Jan 6, 2014
Q A
Bug fix? yes
New feature? no
BC breaks? no
Deprecations? no
Tests pass? n/a
Fixed tickets no
License MIT
Doc PR

When you use ts files, QtLinguist, generate empty string for all translations, even if there are not translated yet.

It could be a good idea to generate an entry in the MessageCatalogue only if translation value is not empty.

@fabpot
Copy link
Member
fabpot commented Jan 6, 2014

@fabpot
Copy link
Member
fabpot commented Jan 7, 2014

Thank you @vlefort.

fabpot added a commit that referenced this pull request Jan 7, 2014
This PR was submitted for the 2.3-dev branch but it was merged into the 2.3 branch instead (closes #9952).

Discussion
----------

[Translator] Fix Empty translations with Qt files

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

When you use ts files, QtLinguist, generate empty string for all translations, even if there are not translated yet.

It could be a good idea to generate an entry in the MessageCatalogue only if translation value is not empty.

Commits
-------

1c220be [Translator] Fix Empty translations with Qt files
@fabpot fabpot closed this Jan 7, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants
0