in reply to Re: Free Nodelet Hack: Add parent & root links to nodes in 'Nodes You Wrote' (or anywhere)
in thread Have parent 'Node ID' as link in 'Nodes You Wrote' node

I haven't tried modifying your code above, but I point out that parameter names with leading underscores are already handled for you by the Free Nodelet templating. So instead of 

if ( params.indexOf( ";_redir=root" ) > 0 )

[download]
you should be able to write 
if ( `_redir` == "root" )

[download]
We're building the house of the future together.
  • Comment on Re^2: Free Nodelet Hack: Add parent & root links to nodes in 'Nodes You Wrote' (or anywhere)
  • Select or Download Code

Replies are listed 'Best First'.
Re^3: Free Nodelet Hack: Add parent & root links to nodes in 'Nodes You Wrote' (escape)
by tye (Sage) on Dec 22, 2006 at 17:32 UTC 
    if ( `_redir` == "root" )

    [download]

    No, that'd turn into one of the following, either: 

    if ( root == "root" )

    [download]

    or 

    if ( `_redir` == "root" )

    [download]

    neither of which looks like valid javascript to me. So you'd want something more like: 

    if ( "`_redir`" == "root" )

    [download]

    which would induce a syntax error if somehow you got a quote (or perhaps a trailing backslash) in your parameter value.

    There needs to be an escape specifier for use in javascript strings. Perhaps: 

    if ( "`_redir\`" == "root" )

    [download]

    where that trailing \ tells the Free Nodelet templating to apply quotemeta to the parameter value (not implemented yet). I haven't checked exactly how escaping works in javascript strings, so quotemeta might not be appropriate. Perhaps `_redir"` would surround it in quotes and escape as appropriate.

    - tye        

[reply]
[d/l]
[select]

      root is a URI component, so it could be escaped.

      if ( decodeURIComponent(`_redir"`) == "root" )

      I've discussed the need for a variables returned as JavaScript literals before. I even provided a workaround.

      if ( decodeURIComponent(decodeURIComponent('`_redir%`')) == "root" )
      if ( decodeURIComponent( '`_redir%`' ) == "root" )

[reply]
[d/l]
[select]

        I'm not sure why you are thinking that it needs to be decoded twice. `_redir` expands to the parameter value, not to how that value might have been escaped in order to get passed through HTTP.

        I was aware of your work-around, which was part of why I was aware that a better solution was warranted. But, with the better solution, decodeURIComponent() would not be required.

        - tye        

[reply]

In Section Perl Monks Discussion