[ 
http://issues.apache.org/jira/browse/MYFACES-310?page=comments#action_12315324 
] 

Grant Smith commented on MYFACES-310:
-------------------------------------

I'm going to err on the side of safety and go with Steven's original 
modifications. When we have the new unit testing framework in place I will 
optimize as per Edmond's suggestion. Please take a look, test, and let me know 
if I can close this issue. Thanks.

> HtmlLinkRendererBase does not work with custom components that do not extend 
> HtmlCommandLink
> --------------------------------------------------------------------------------------------
>
>          Key: MYFACES-310
>          URL: http://issues.apache.org/jira/browse/MYFACES-310
>      Project: MyFaces
>         Type: Bug
>     Versions: 1.0.9 beta
>     Reporter: Steven Oglesby
>     Assignee: Grant Smith
>     Priority: Blocker
>  Attachments: log.txt
>
> HtmlLinkRendererBase does not work with custom components that do not extend 
> HtmlCommandLink since it casts to this class at line 323 in method 
> renderJavaScriptAnchorStart.
> i.e.
>  String target = ((HtmlCommandLink)component).getTarget();
> Other similar casting operations are done as follows:
>  String target = null;
>  if (component instanceof HtmlCommandLink) {
>    target = ((HtmlCommandLink)component).getTarget();}
>  else {
>    target = (String)component.getAttributes().get(HTML.TARGET_ATTR);}
> Which looks as if it would solve this problem.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
   http://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see:
   http://www.atlassian.com/software/jira

Reply via email to