Item10827: TWISTY sets empty cookie value for remember="off" instead of clearing it, causes excessive overhead

Priority:	CurrentState:	AppliesTo:	Component:	WaitingFor:
Normal	Closed	Extension	TwistyPlugin

TWISTY uses cookies to store "remembered" state. That makes sense. But it sets up cookies even when remember="off".

Aside from the fact that this is suboptimal and potentially inefficient, we've run into a bug with the Safari browser. If there are a LOT of TWISTY calls, the cookie overloads and Safari gets confused.

The confusion is a Safari probloem.

But in the case we investigated, all 100 (!!) TWISTY calls set remember="off" There doesn't need to be a cookie if the cookie isn't of value.

-- VickiBrown - 01 Jun 2011.

remember="off" sets an empty value for the pref instead of clearing the setting.

I have made a fix that will be effective in Foswiki 1.1.4. You can also grab TwistyPlugin 1.6.13 and the core javascript attached here.

-- ArthurClemens - 02 Jun 2011