.get returning default

[jdunck]

I think it would be useful to have .get take an optional param which is returned iff the requested key is missing (or has an undefined value).

Agree?

[marcuswestin]

Hey jdunck,

Thanks for the request!

Would (store.get(key, defaultValue)) act differently from (store.get(key) || defaultValue)? If not, I think it's to the advantage of a user of store.js to add a parameter to the .get signature.

Cheers!

[jdunck]

Well, store.get(...) || default would use default if the get return is false-y. To be equivalent, it would need to be:
store.get(...) == undefined || default.
I'm just trying make use a bit more concise, but it's a small matter.

[marcuswestin]

Since local storage coerces values into string store.set('foo', false); store.get('foo') == 'false'.

Thanks for the suggestion, but I think keeping a clean and straight-forward small implementation that's easily modified will serve everyone well.

Closing the issue for now - feel free to reopen if you believe there are strong reasons to implement the feature.

Thanks!

[jdunck]

Well, my other commit serializes to JSON if available. I need that one, this one is just nice. :-)