Leverage validated value unwrapper for validateProperty() and validateValue()

Description

Registered value unwrappers are ignored by Validator#validateProperty() and validateValue(). setValidatedValueHandlerToValueContextIfPresent must be invoked for these entry points into ValidatorImpl as well.

Environment

None

Activity

Show:
George Gastaldi
January 24, 2014, 8:47 PM

It didn't work for me. My test still fails with the latest snapshot.
Suggestion: the method handleValidatedValue() could be called unwrapValidatedValue()

Gunnar Morling
January 24, 2014, 10:03 PM

It didn't work for me. My test still fails with the latest snapshot.

Ah, sorry, the snapshot had not been deployed (I assumed the continuous job would trigger that but actually the deployment is running only once per week; we should change that). Anyways, I've triggered the deploy job manually and the JARs are there now.

Suggestion: the method handleValidatedValue() could be called unwrapValidatedValue()

We went for the more generic name having a potential evolvement into generic means of value unhandling in mind (see for the discussion of the details).

George Gastaldi
January 24, 2014, 10:25 PM

Yes, now it works! Thanks for fixing this. Good job!

Gunnar Morling
January 24, 2014, 10:54 PM

Yeah, great to hear everything is working now

Fixed

Assignee

Unassigned

Reporter

Gunnar Morling

Labels

None

Feedback Requested

None

Feedback Requested By

None

backPortable

None

Suitable for new contributors

None

Pull Request

None

backportDecision

None

backportReEvaluate

None

Components

Fix versions

Affects versions

Priority

Major
Configure