Proposed change to Persist for dijits

[tk]

Instead of persist: true/false, I propose we modify persist to be a numeric value so we can use {expire: persist} in dojo.cookie() to specify longer cookie expiration dates. This could be done several way....

my suggestion would basicalyl be to treat anything !== false or possible >-1 since 0 is a valid option (session cookie) as true...

Comments/suggestions?

[Adam Peller]

it would be nice if we were consistent with dojo.cookie. dojo.cookie seems to use days for props.expire...

[Adam Peller]

(In [12033]) Persist splitter size. How's this look? Refs #5585

[Adam Peller]

(In [12034]) Just use Boolean persist for now. Realized that Number or Date isn't possible in the parser, so the previous example was really coming through as expires: null, not expires: 1. expires: yyyy-mm-dd is funky but perhaps not really useful? Refs #5585

[tk]

I would say not support dates.... how many websites say "your cookie will expire on X day (unless they are telling you what X days from now is)

this would mean false, 0+ are the only values we care about, and you can say true==0 for boolean compatability if oyu really want to.

in this case, false means no persist 0 == sessions (default current only behavior)

0 == X days

Off to bed, just wanted to put this in a safe spot for others to ref later.

[Adam Peller]

Replying to tk:

this would mean false, 0+ are the only values we care about, and you can say true==0 for boolean compatability if oyu really want to.

This would still imply mixed types, which the parser doesn't handle well, plus it complicates the API. Bill has suggested we just go with boolean and have some global setting that lets the user customize the persist behavior.

[bill]

Right, and also perhaps we should change the default behavior so that persist="true" on a widget persists forever, not just for the session. That's a simple change that could be added to 1.1 to partly address this bug.

[Adam Peller]

moving back to 1.1 so we don't forget that part

[Adam Peller]

please change the default to 365 days for 1.1, but don't change widget APIs.

[Karl Tiedt]

(In [15017]) fixes #5585 adding {expires: 365} as 3rd arg to all the persist cookie write statements... !strict