[
https://issues.apache.org/jira/browse/CHAIN-63?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13183392#comment-13183392
]
Ales Dolecek commented on CHAIN-63:
-----------------------------------
Ahh - sorry. I looked checked out current trunk and there is still Context as
"extends Map<String, Object>".
Where is the development going on? And BTW CHAIN-58 does not mention to aim for
complete drop of Context - maybe it's "understood", but it better be stated
explicitly. After all 1.x chains also use "Context extens Map" without any
further advantage.
Ales
> Replace Context with Map<String, ? extends Object>
> --------------------------------------------------
>
> Key: CHAIN-63
> URL: https://issues.apache.org/jira/browse/CHAIN-63
> Project: Commons Chain
> Issue Type: Improvement
> Affects Versions: 2.0
> Reporter: Ales Dolecek
> Assignee: Simone Tripodi
> Priority: Minor
> Fix For: 2.0
>
>
> Hello,
> I always wandered why you *force* chain users to use the Context interface
> which adds nothing over Map. I would understand that if the Context would not
> inherit from map and only required
> Object put(String, Object)
> Object get(String)
> But since Context itself IS Map then you gain nothing while disallowing use
> of Map implementations. I know that sou suggest that Context shall implement
> "Attribute-Property Transparency" but it is only recomendation and is not
> forced by commons chains anyway.
> And if you define execute like
> Object execute(Map<String, ? extends Object> context)
> it can still accept Context instances and all its subclasses.
> The downside is that you would have to give up the retrieve method introduced
> to Context. And BTW the Context imposes hard restriction see my other bug.
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators:
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira
