Help setting a cookie

[Peter]

Like I said, use toUTCString, as toGMTString is deprecated. toUTCString does exactly the same thing, they are the same function. toGMTString is provided for compatibility, it's recommended you use the other method instead.

[Boy_Wonder]

Okay, I'll go change that now Peter - thanks

[Michael]

Jul 30, 2010 3:42:06 GMT -8 Peter said:

Like I said, use toUTCString, as toGMTString is deprecated. toUTCString does exactly the same thing, they are the same function. toGMTString is provided for compatibility, it's recommended you use the other method instead.

The actual problem wasn't the "toGMTString", it was the fact that it was on the end of the .setTime()

And I didn't know it was deprecated, thanks Peter.

[Charles Stover]

var deleteCookie = function(name)
{
document.cookie = name + "=x; expires=Thu, 01 Jan 1970 00:00:00 GMT; path=/";
},
readCookie = function(name)
{
var cookies = document.cookie.split(";");
name = name + "=";
for (var rc = 0; rc < cookies.length; rc++)
{
var c = cookies[rc].replace(/^\s+/, "");
if (c.indexOf(name) == 0)
return c.substring(name.length, c.length);
}
return null;
},
setCookie = function(name, value)
{
var temp = new Date();
temp.setTime(temp.getTime() + 31536000000);
document.cookie = name + "=" + value + "; expires=" + temp.toGMTString() + "; path=/";
};

That should work. I did some minor edits inline and haven't tested it since, but it should work. If there's a problem, it would be at the replace() in readCookie.

setCookie("test", "gummy bears");
alert(readCookie("test")); // gummy bears
deleteCookie("test");

[Charles Stover]

Jul 30, 2010 4:20:05 GMT -8 Michael said:

Jul 30, 2010 3:42:06 GMT -8 Peter said:

Like I said, use toUTCString, as toGMTString is deprecated. toUTCString does exactly the same thing, they are the same function. toGMTString is provided for compatibility, it's recommended you use the other method instead.

The actual problem wasn't the "toGMTString", it was the fact that it was on the end of the .setTime()

And I didn't know it was deprecated, thanks Peter.

I didn't either. Googled and it is.
So OP should update that part of meh code if they use it.

[Boy_Wonder]

As the code I have is working fine I'll keep it as it is right now, however I'd be interested in how that code works differently to the one I'm currently using for the cookies and if anything is 'better'

[Charles Stover]

The same things are happening in terms of setting and reading cookies. The one I posted just makes it easier to do, instead of having to include all the parts of document.cookie (e.g. expire and path) when setting a cookie and searching through all parts of all cookies when reading a cookie, you can just do setCookie() and readCookie().
And if you ever want to use cookies for another project in the future, it's a lot easier to just copy-paste the functions than to re-code the whole document.cookie thing.

[Boy_Wonder]

Ooh okay, thanks Charles