Make cookie path configurable

sridhars711 / dynatree

Automatically exported from code.google.com/p/dynatree

0 stars 0 forks source link

Make cookie path configurable #78

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago

We're using dynatree to render the navigation tree of a DMS component. dynatree 
in this case is 
loaded for all the different folders, but always shows the entire navigation 
tree. Since each folder 
has a different URL, this currently means that for each of them, a cookie with 
persistent information 
is written, which produces an awful lot of cookies and isn't really helpful, 
since we want the tree 
status to be consistent in the entire DMS.

So we've written a patch against 0.4beta2 that i want to propose for inclusion. 
It allows you to 
specify the cookie path during initialization. If it is not set, the old 
behavior kicks in and the cookie 
path corresponds to the page url.

Original issue reported on code.google.com by kont...@contentcontrol-berlin.de on 18 Feb 2009 at 12:45

Attachments:

patch_dynatree.txt

GoogleCodeExporter commented 9 years ago

Original comment by moo...@wwwendt.de on 18 Feb 2009 at 6:47

Changed state: Accepted
Added labels: Type-Enhancement
Removed labels: Type-Defect

GoogleCodeExporter commented 9 years ago

Implemented with r179.

Thanks for the patch! I modified it a to allow setting all available$cookie 
options.
Could you check if it works for you?

Original comment by moo...@wwwendt.de on 19 Feb 2009 at 7:41

Changed state: undefined

GoogleCodeExporter commented 9 years ago

Original comment by moo...@wwwendt.de on 20 Feb 2009 at 6:44

Changed state: Fixed
Added labels: Milestone-Release0.4

GoogleCodeExporter commented 9 years ago

considered verified

Original comment by moo...@wwwendt.de on 17 Jul 2012 at 4:12

Changed state: Verified

GoogleCodeExporter commented 9 years ago

Was this ever merged into trunk?

Original comment by ma...@stasismedia.com on 26 Aug 2012 at 6:26

GoogleCodeExporter commented 9 years ago

it was fixed in the trunk. any doubts?

Original comment by moo...@wwwendt.de on 26 Aug 2012 at 7:02

GoogleCodeExporter commented 9 years ago

I see, it was implemented slightly differently. I was searching through current 
trunk and documentation for 'cookiePath', but it's actually under cookie.path

Perhaps the new cookie options should be added to the docs at 
http://wwwendt.de/tech/dynatree/doc/dynatree-doc.html

Either way, my issue is now resolved. Thanks

Original comment by ma...@stasismedia.com on 26 Aug 2012 at 7:08