-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

  This issue deals with input parameters being trimmed as specified in
IValidator. A comment by Howard on 3 May 05 advises against trimming
getParameter() instead performing the trimming inside ValidField or
AbstractTextField. The issue with those are that they are abstract
classes which do not deal with the IValidator's toObject method (they
don't even implement IValidator).

  The only way I see to 'fix' this issue is to go into each validator
(DateValidator, EmailValidator, IntValidator, NumberValidator, etc...)
and trim inside their toObject methods individually - or - go into
BaseValidator, implement a toObject that performs the trim, then calls
an abstract method of the same basic signature for subclasses to
implement (instead of having all subclasses implement toObject, they
would be changed to implement the new abstract method).

  The only other possibility is to change the documentation of the
IValidator interface to state "input will NOT be trimmed" - then all
current validators will work as the javadoc states. I, personally, think
input should be trimmed by default and am willing to make the changes
necessary - but that toObject method is public and changing its
implementation to a superclass could have wider ramifications than I can
envision.

Thoughts?

Note: This issue is written against 3.0, but the same condition exists
in trunk.

Brian
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.2.5 (MingW32)

iD8DBQFED2o+aCoPKRow/gARAgpTAKC6bcOEAI+KsUkxuAyz7cFKc8cCmgCaApR0
ww3/mixsP3QwlhngA/sdmSI=
=wceR
-----END PGP SIGNATURE-----

---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to