[jifty-devel] Wrong logic in state-preserving links?

[Nelson Elhage]

Indeed, you are correct. We actually just ran into this in our internal 
app, as well, shortly after you sent the bug report.

I just pushed a fix to Jifty SVN, and it'll hit CPAN whenever we push the 
next version.

Thanks for the report,

- Nelson Elhage

On Sun, 30 Jul 2006, Wolfgang Kinkeldei wrote:

> Hello Jifty fans!
>
> when using things like
>   Jifty->web->set_variable('varname', some_value);
>
> in order to transport the variable's value to the next request, I found out, 
> that this works perfectly within forms, but does not correctly transport the 
> variables in text-links that are generated like this:
>   <% Jifty->web->link(..., preserve_state=>1) %>
>
> After some browsing through the sources (of version 0.60722 from CPAN) I 
> found out that the request's variables are built into the link, not the 
> Jifty->web's variables set with the construct above.
> If I understood things right, line 175 of Jifty::Web::Clickable.pm inside the 
> "new" function should use
>  Jifty->web->state_variables
>
> instead of the request's vars. Am I right?
>
>
> Regards,
>
> Wolfgang
>
> -- 
>
> ' /\_/\ ' .print[split??,"".(($/=q|Cms)+-03467:;<=|)=~tr!C-z -B! -z!)x
> '( o.o )' .$/]->[hex]foreach split qr<>,qq+1ecd039ad65b025b8063475b+||
> ' > ^ < ' .q<!-- Wolfgang Kinkeldei - mailto:wolfgang at kinkeldei.de -->
>
>
>
> _______________________________________________
> jifty-devel mailing list
> jifty-devel at lists.jifty.org
> http://lists.jifty.org/cgi-bin/mailman/listinfo/jifty-devel
>