Skip to content
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

Fixed updating entires with plural form #2

Merged
merged 2 commits into from Nov 19, 2015
Merged

Fixed updating entires with plural form #2

merged 2 commits into from Nov 19, 2015

Conversation

dhm80
Copy link
Contributor

@dhm80 dhm80 commented Nov 18, 2015

Plural form could not be saved when using updateEntry method (plural form was deleted).

Here is a fix for that.

  • updated Parser::updateEntry()
  • added Parser::updateEntries()
  • refactored removing fuzzy from Parser::updateEntry() into Parser::removeFuzzyFlagForMsgId()
  • added tests

- updated Parser::updateEntry()
- added Parser::updateEntries()
- refactored removing fuzzy from Parser::updateEntry() into Parser::removeFuzzyFlagForMsgId()
- added tests
unset($flags[array_search('fuzzy', $flags, true)]);
$this->entriesAsArrays[$original]['flags'] = $flags;
/**
* Allows modifaction of all translations of an entry
Copy link
Owner

Choose a reason for hiding this comment

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

There is a typo here

@MAXakaWIZARD
Copy link
Owner

@dhm80 Thanks for this PR. Have you tested if file saved with this code can be opened in PoEdit and correctly displayed there?

@dhm80
Copy link
Contributor Author

dhm80 commented Nov 19, 2015

@MAXakaWIZARD No problem, glad to help.

I changed the exception messages, corrected the typo and used || instead of or in the methods. That should be it ;)

And: Yes, the generated .po file works like charm in poEdit. I just testet that. Did you have any problems?

MAXakaWIZARD added a commit that referenced this pull request Nov 19, 2015
Fixed updating entires with plural form
@MAXakaWIZARD MAXakaWIZARD merged commit 4a7a9cf into MAXakaWIZARD:master Nov 19, 2015
@MAXakaWIZARD
Copy link
Owner

@dhm80 Thanks!

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.

None yet

2 participants