[ 
https://issues.apache.org/jira/browse/COCOON-1984?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12472609
 ] 

Carsten Ziegeler commented on COCOON-1984:
------------------------------------------

The delegating is done as this class implements both, Parameters and Map, and 
should provide the same values as the wrapped parameters object. We could avoid 
delegating with the cost of copying the information from the original 
parameters object to this one. Because of performance we chose to use 
delegation instead.

> o.a.c.environment.TemplateObjectModelHelper.ParametersMap does not override 
> toString()
> --------------------------------------------------------------------------------------
>
>                 Key: COCOON-1984
>                 URL: https://issues.apache.org/jira/browse/COCOON-1984
>             Project: Cocoon
>          Issue Type: Bug
>          Components: * Cocoon Core
>    Affects Versions: 2.2-dev (Current SVN)
>            Reporter: Mark Lundquist
>         Assigned To: Carsten Ziegeler
>            Priority: Minor
>
> ...and as a result, the object displays as empty, e.g. when debugging in 
> Eclipse.  I added
>          public String toString() {
>                       return wrappedParameters.toString();
>         }
> and that fixed it.  Although, I am puzzled as to why this class both extends 
> the Avalon Parameters class and is implemented using a delegate 
> ('wrappedParameters') of that type.  Why is not extending Parameters enough?  
> The delegate seems like an unnecessary complication.

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

Reply via email to