Opened 12 years ago

Closed 12 years ago

#5810 closed defect (fixed)

combine dojo.cookie and dojo.cookie.useObject

Reported by: Adam Peller Owned by: Adam Peller
Priority: high Milestone: 1.1
Component: General Version: 1.0
Keywords: Cc: jgarfield, dante
Blocked By: Blocking:

Description (last modified by Adam Peller)

why not just treat objects in dojo.cookie this way by default? It seems like if an object is passed into the regular dojo.cookie method, it gets serialized with toString() which is usually something worthless like [object Object] and would seem like a bug.

We could do this now for 1.1 and avoid a round of deprecation for useObject.

Change History (4)

comment:1 Changed 12 years ago by Adam Peller

Cc: jgarfield dante added
Description: modified (diff)

refs #4969

comment:2 Changed 12 years ago by Adam Peller

Owner: changed from Ben Lowery to Adam Peller

comment:3 Changed 12 years ago by Adam Peller

...because, Justin reminds me, you can't know it's an object when you de-serialize without an API hint.

We could combine this into the existing API as an additional (4th?) param? Would mixing it in with 'props' be confusing?

comment:4 Changed 12 years ago by Adam Peller

Resolution: fixed
Status: newclosed

(In [12445]) remove useObject, since it's really a one-liner for the developer. Fixes #5810. reduce some code. use expires to remove test cookie. Refs #4969 !strict

Note: See TracTickets for help on using tickets.