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 13022] New: - Pass namespace URI and name args to the Rule methods
Date Thu, 26 Sep 2002 09:18:39 GMT
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG 
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
<http://nagoya.apache.org/bugzilla/show_bug.cgi?id=13022>.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND 
INSERTED IN THE BUG DATABASE.

http://nagoya.apache.org/bugzilla/show_bug.cgi?id=13022

Pass namespace URI and name args to the Rule methods

           Summary: Pass namespace URI and name args to the Rule methods
           Product: Commons
           Version: 1.3 Final
          Platform: All
        OS/Version: All
            Status: NEW
          Severity: Normal
          Priority: Other
         Component: Digester
        AssignedTo: commons-dev@jakarta.apache.org
        ReportedBy: cmlenz@apache.org


As an enhancement to the current Digester API, I propose to add new variants to
the Rule methods begin(), body() and end(). Those new variants would accept the
namespace URI and local/qualified name of the current element.

As Rule is not an interface, but an abstract class, this change can be done
without hurting backwards compatibility. It will loosen the coupling between
Rule implementations and Digester, as the rule would get sufficient information
about the current element, without needing to call
Digester.getCurrentElementName(). In addition, it provides access to the current
namespace URI, which is not possible with the current version of Digester.

I'll be attaching a patch that implements these changes. This involves changes
to Rule.java, Digester.java and BeanPropertySetterRule.java. The latter change
is not totally necessary, but rather demonstrates the benefits of the proposed
API. All other Rule implementations could be changed later to remove the plenty
deprecation warnings.

[I've posted this patch to the commons-dev list before, I'm just adding it here
so it doesn't get lost]

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


Mime
View raw message