DO NOT REPLY [Bug 33064] - [taglib] Struts tags must check attribute values using isEmpty(), not against nulls

[bu...@apache.org]

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=33064>.
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=33064





------- Additional Comments From laurie@holoweb.net  2005-09-28 04:39 -------
Created an attachment (id=16541)
 --> (http://issues.apache.org/bugzilla/attachment.cgi?id=16541&action=view)
First-pass at one approach to addressing this issue

This is a patch which updates all tag attribute setters to 'normalize' empty
values to null, as one approach to resolving this issue.

NOTE: this is *not* intended to be a final, ready-to-apply patch but rather as
a vehicle for further discussion. It includes some annotations (comments marked
'[lh]') pointing out some issues to be resolved.

The alternative approach, preserving attribute values as set and changing
checks for null to checks for null-or-empty will follow as a second, alternate
patch for comparison.

-- 
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: dev-unsubscribe@struts.apache.org
For additional commands, e-mail: dev-help@struts.apache.org