You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tapestry.apache.org by "Howard M. Lewis Ship (JIRA)" <de...@tapestry.apache.org> on 2008/06/27 18:33:45 UTC

[jira] Commented: (TAPESTRY-2483) addScript does not like the @ symbol

    [ https://issues.apache.org/jira/browse/TAPESTRY-2483?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12608827#action_12608827 ] 

Howard M. Lewis Ship commented on TAPESTRY-2483:
------------------------------------------------

The URL was encoded, so the @ became %40.

When fed to RenderSupport.addScript(), the %40 looks like a replacement point (like %s or %d).

I think the solution is to have an addScript() method on RenderSupport that does not invoke java.lang.String.format().

> addScript does not like the @ symbol
> ------------------------------------
>
>                 Key: TAPESTRY-2483
>                 URL: https://issues.apache.org/jira/browse/TAPESTRY-2483
>             Project: Tapestry
>          Issue Type: Bug
>          Components: Core Components
>    Affects Versions: 5.0.13
>            Reporter: Martijn Brinkers
>
> Tapestry throws a Render queue error when the addScript contains the @ symbol.
> The @ symbol is converted to
> %40 and the following script is added with addScript:
> new pageLinkPopup("pagelinkpopup", "user/edit/m.brinkers%40pobox.com",
> "edit user", "width=600,height=300");
> But, this results in:
> Render queue error in BeginRender[Users:pagelinkpopup]: Conversion = 'p'
> Stack trace
>               * java.util.Formatter
>                 $FormatSpecifier.conversion(Formatter.java:2605)
>               * java.util.Formatter
>                 $FormatSpecifier.<init>(Formatter.java:2633)
>               * java.util.Formatter.parse(Formatter.java:2479)
>               * java.util.Formatter.format(Formatter.java:2413)
>               * java.util.Formatter.format(Formatter.java:2366)
>               * java.lang.String.format(String.java:2770)
>               * org.apache.tapestry5.internal.services.RenderSupportImpl.addScript(RenderSupportImpl.java:129)

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


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tapestry.apache.org
For additional commands, e-mail: dev-help@tapestry.apache.org