Skip to content

SPR-9692: Change to match converter to interface before Enum #127

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

wkleu
Copy link

@wkleu wkleu commented Aug 29, 2012

GenericConversionService changed to not match converter to Enum.class
before trying to match converter to an interface.

Prior to this commit a converter would be matched to the superclass of
an enum (which will always be Enum.class) ignoring converters matching
the interface.

Enum.class will now be ignored when retrieving superclasses and only
after trying to match converters to interfaces will it try to match
converters to Enum (just before trying to match to Object).

This is the behaviour encountered prior to 3.1.0.

Issue: SPR-9692

I have signed and agree to the terms of the SpringSource Individual Contributor License Agreement.

GenericConversionService changed to not match converter to Enum.class
before trying to match converter to an interface.

Prior to this commit a converter would be matched to the superclass of
an enum (which will always be Enum.class) ignoring converters matching
the interface.

Enum.class will now be ignored when retrieving superclasses and only
after trying to match converters to interfaces will it try to match
converters to Enum (just before trying to match to Object).

This is the behaviour encountered prior to 3.1.0.

Issue: SPR-9692
@ghost ghost assigned cbeams Oct 25, 2012
@cbeams
Copy link
Contributor

cbeams commented Oct 25, 2012

Closing as this pull request appears to be superseded by #132. @philwebb, please confirm.

@cbeams cbeams closed this Oct 25, 2012
@philwebb
Copy link
Member

@cbeams : Correct. Pull 132 fixes this issue (albeit in a slightly different way)

rwinch pushed a commit to rwinch/spring-framework that referenced this pull request Jul 13, 2016
…-state

AbstractRequestBodyPublisher.onDataAvailable improvement
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.

3 participants