commons-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From bugzi...@apache.org
Subject DO NOT REPLY [Bug 34766] - CallMethodRule doesn't call with null parameters, String-ifies them first.
Date Fri, 06 May 2005 00:58:02 GMT
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
<http://issues.apache.org/bugzilla/show_bug.cgi?id=34766>.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=34766





------- Additional Comments From skitching@apache.org  2005-05-06 02:58 -------
I think you're right that the javadoc does not match the code (my mistake; I
wrote that piece of javadoc!).

However the current behaviour (nulls get passed through ConvertUtils) is
deliberate, and can't be changed.

I will correct the javadoc.

To address your issue, one option is to register custom ConvertUtils converters
that leave nulls unchanged. This way, digester calls ConvertUtils.convert, but a
null just comes straight back. See the BeanUtils library, method
ConvertUtils.register for more details. Note that setting a converter on
ConvertUtils changes global behaviour for the whole JVM (ConvertUtils is a
Singleton) so if you are using any other libs that use ConvertUtils you may need
to think carefully about the implications of doing this.

The other option would be to add a flag to the CallMethodRule class to indicate
whether it should use the current behaviour or leave nulls alone. If you wish to
provide a patch for this (and unit tests) I would be happy to review and commit
it. Or I might get around to implementing this sometime (it seems a worthy
request) but I can't give you any particular date this might be done by.

-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.

---------------------------------------------------------------------
To unsubscribe, e-mail: commons-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: commons-dev-help@jakarta.apache.org


Mime
View raw message