commons-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Rahul Akolkar (JIRA)" <j...@apache.org>
Subject [jira] Updated: (DIGESTER-74) [Digester][PATCH] Cannot add multiple CallMethodRules for the same pattern
Date Wed, 12 May 2010 22:36:43 GMT

     [ https://issues.apache.org/jira/browse/DIGESTER-74?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]

Rahul Akolkar updated DIGESTER-74:
----------------------------------

    Reporter: Rahul Akolkar  (was: Rahul)

Changing reporter in order to close old account.


> [Digester][PATCH] Cannot add multiple CallMethodRules for the same pattern
> --------------------------------------------------------------------------
>
>                 Key: DIGESTER-74
>                 URL: https://issues.apache.org/jira/browse/DIGESTER-74
>             Project: Commons Digester
>          Issue Type: Improvement
>    Affects Versions: 1.6
>         Environment: Operating System: other
> Platform: Other
>            Reporter: Rahul Akolkar
>            Priority: Minor
>         Attachments: CallMethodParamPatches.zip
>
>
> ISSUE:
> The issue is summarized here, URI might get fragmented, copy with care [ 
> http://marc.theaimsgroup.com/?l=jakarta-commons-user&m=111706440425278&w=2 ]
> PROPOSED SOLUTION:
> Instead of storing an object array as an element of the parameters stack in 
> Digester (which is utilized by CallMethodRule and CallParamRule rules), store 
> a list of object arrays.
> Add an index property to CallMethodRule corresponding to its document order 
> (within CallMethodRules for the same pattern), and use that to access the 
> appropriate element within the list at the top of the parameters stack.
> The same index provides users the ability to associate a CallParamRule with 
> the appropriate CallMethodRule.
> Patch is attached in next post. Javadoc needs to be brushed up, I can do that 
> depending on interest in the patch. Patch also contains updated call method 
> rule test cases (one change in the 'testTwoCalls' test case), all 15 now pass. 
> There is full backward compatibility. Comments/suggestions welcome.
> -Rahul

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


Mime
View raw message