Rename validateParameters to validateAllParameters

Description

I got confused by validateParameter vs validateParameters
I think changing the name to validateAllParameters will solve the issue.

Environment

None

Activity

Show:

Hardy Ferentschik February 7, 2011 at 1:26 PM

Applied pull request. Not really sure what the issue would have been with methodvalidation as package name (maybe it is really a mouthful), but method works as well

Gunnar Morling February 6, 2011 at 2:12 PM

Emmanuel Bernard January 23, 2011 at 4:27 AM

My preference would be to use
method

Alternatives:
interception //probably better if we add a real interception layer one day
methodcheck
methodinterception
methodvalidation

Gunnar Morling January 22, 2011 at 3:19 PM

Ok, any other ideas for the package name then? I find "methodvalidation" pretty descriptive, but I'm open for alternatives. If it should be shorter, "metval" or similar might be an idea, though I'm no big fan of abbreviations here.

Emmanuel Bernard January 18, 2011 at 12:30 PM

about moving to a new package +1
methodvalidation might not be the best package name though.

About extending Validator, I'm not sure. I think thanks to unwrap, we might not need it. I'd stay with a standalone interface for now and see how people react.

Fixed

Details

Assignee

Reporter

Participants

Emmanuel Bernard
Gunnar Morling
Hardy Ferentschik

Components

Fix versions

Affects versions

Priority

Created January 13, 2011 at 4:21 PM
Updated August 5, 2011 at 5:55 PM
Resolved February 7, 2011 at 1:26 PM