I've incorporated Jacob's changes into a recent change bundle:
https://javaserverfaces.dev.java.net/issues/show_bug.cgi?id=71
Please review.
Thanks, Roger (and thank you Jacob!)
Jacob Hookom wrote:
> Seeking Review for Issue #71 -
>
> https://javaserverfaces.dev.java.net/issues/show_bug.cgi?id=71
>
> The fix will now allow users to specify their own onclick event in the
> commandLink tag. No modifications were made to the original code to
> generate the commandLink JavaScript, the fix simply prefixes and
> suffixes data in the StringBuffer used. This 'wrapping' will only
> occur if the user specified an onclick value on the tag, otherwise
> just the injected JavaScript is rendered.
>
> Best Regards, Jacob
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe_at_javaserverfaces.dev.java.net
> For additional commands, e-mail: dev-help_at_javaserverfaces.dev.java.net
>