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

Ambiguous methods bridge #646

Merged
merged 4 commits into from Nov 28, 2018

Conversation

Projects
None yet
3 participants
@froque
Copy link
Contributor

froque commented Nov 27, 2018

fixes issue #645

froque added some commits Nov 27, 2018

REST annotations:
renamed misspeledd loadBoundleClasses
cleaned javadoc
show all methods that are ambiguous
REST annotations
added slf4j-simple to unit tests
added debug log to mapped methods
REST annotations
when loading methods, skip bridge methods
added unit test and related classes

fixes issue 645 Ambiguous methods mapped for the current request when compiler creates bridge methods

@solomax solomax merged commit f6ab49f into wicketstuff:wicket-8.x Nov 28, 2018

solomax added a commit that referenced this pull request Dec 1, 2018

Ambiguous methods bridge (#646)
* REST annotations:

renamed misspeledd loadBoundleClasses
cleaned javadoc
show all methods that are ambiguous

* REST annotations

added slf4j-simple to unit tests
added debug log to mapped methods

* REST annotations

when loading methods, skip bridge methods
added unit test and related classes

fixes issue 645 Ambiguous methods mapped for the current request when compiler creates bridge methods

* Changes to string operations as requested in code review
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.