Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Fix the wrong method name given the corresponding interface#18536

Merged
nicolas-grekas merged 1 commit into
symfony:masterfrom
joelwurtz:fix/fix-wrong-method-name
Apr 14, 2016
Merged

Fix the wrong method name given the corresponding interface#18536
nicolas-grekas merged 1 commit into
symfony:masterfrom
joelwurtz:fix/fix-wrong-method-name

Conversation

@joelwurtz
Copy link
Copy Markdown
Contributor

Q A
Branch? "master"
Bug fix? yes
New feature? no
BC breaks? no
Deprecations? no
Tests pass? yes
Fixed tickets #17545 Follow up
License MIT
Doc PR

This a follow up of #17545 as i introduce bad methods name in the traits sorry :/

@xabbuh
Copy link
Copy Markdown
Member

xabbuh commented Apr 14, 2016

👍

Status: Reviewed

@nicolas-grekas
Copy link
Copy Markdown
Member

Thank you @joelwurtz.

@nicolas-grekas nicolas-grekas merged commit 231819e into symfony:master Apr 14, 2016
nicolas-grekas added a commit that referenced this pull request Apr 14, 2016
…ace (joelwurtz)

This PR was merged into the 3.1-dev branch.

Discussion
----------

Fix the wrong method name given the corresponding interface

| Q             | A
| ------------- | ---
| Branch?       | "master"
| Bug fix?      | yes
| New feature?  | no
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | yes
| Fixed tickets | #17545 Follow up
| License       | MIT
| Doc PR        |

This a follow up of #17545 as i introduce bad methods name in the traits sorry :/

![](http://vomzi.com/wp-content/uploads/2016/02/sorry-gif-847.gif)

Commits
-------

231819e Fix the wrong method name given the corresponding interface
@joelwurtz joelwurtz deleted the fix/fix-wrong-method-name branch April 14, 2016 11:57
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.

4 participants