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

Change getPluralOffset to return numbers and not boolean values #2798

Closed
wants to merge 1 commit into from

Conversation

gfr
Copy link
Contributor

@gfr gfr commented Mar 10, 2016

Q A
Bug fix? yes
New feature? no
BC breaks? no
Deprecations? no
Tests pass? -
Fixed tickets -
1.4 PR -
Refs tickets -
License LGPLv3+
Doc PR -
  • check 1.4

With this change the function getPluralOffset will return numbers and not boolean values. This change is required because the function getPluralMessage uses this value as offset to access array keys.

With this change the function getPluralOffset will return numbers and not boolean values. This change is required because the function getPluralMessage uses this value as offset to access array keys.
@craigh
Copy link
Member

craigh commented Mar 10, 2016

Thank you for the contribution!

please fix StyleCI issues. After, if you know how, please squash the commits (soft reset, force push) so there is only one commit. If not, no problem.

Also please create a PR for 1.4

@craigh craigh added this to the 1.3.11 milestone Mar 10, 2016
@craigh
Copy link
Member

craigh commented Mar 10, 2016

also - please add a changelog entry.

@cmfcmf
Copy link
Contributor

cmfcmf commented Aug 15, 2016

ping @gfr :)

@craigh
Copy link
Member

craigh commented Aug 15, 2016

if you know how, please squash the commits (soft reset, force push) so there is only one commit.

this is no longer necessary. Github magic will do it now.

@Guite
Copy link
Member

Guite commented Aug 20, 2016

Closed in favour of #3010

@Guite Guite closed this Aug 20, 2016
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.

4 participants