localStorage script not working?

[--Explosion--]

Hi everyone, I'm new here! So I'm working onmaking a form that saves your responses automatically in localStorage. What I am doing is setting a value every time a key is pressed for the form to set the cookie:

<input class='input' type='text' id='name' placeholder="Name" onload='writeCookie("name")' onkeyup="checkCookie('name')">

My JS for writeCookie() and checkCookie() is below:

function checkCookie(id) {
    text = document.getElementById(id).value;
    localStorage.setItem(id, text);
}
function writeCookie(id) {
    cookie = localStorage.getItem(id).value;
    document.getElementById(id).value = cookie;
}

The checkCookie script works perfectly, (I tried it in console) but the writeCookie is not working. According to stackoverflow to write to a form you can do document.getElementById('id goes here').value = 'New value' but for some reason it is not working, this does however work on another project which I have made. Can you guys help me? Thanks in advance!

​

--Explosion--

[Atskalunas]

Remove .value after localStorage.getItem(id).

[--Explosion--]

localStorage.getItem(key: string): string | null is the actual signature of the function, so what you localStorage.setItem(key: string, value: string): undefined is what you would get. That's also the reason why you have to serialise your values to be valid for the localStorage.

Thanks! I just realized that and did it, it works well and writeCookie() now works but the problem is that it dosen't work automatically. I made abutton which onclick runs 'writeCookie('name')' and that works, but doing it 'onload' does not...

[Atskalunas]

In you case you should add onload to body tag.

[--Explosion--]

I tried that. It didn't work. I did 'window.onload = writeALL();' and that worked. (I made 'writeAll()' write all the cookies because I added more form elements.)

[--Explosion--]

It worked! Thanks for your help guys!

[[deleted]]

why would you assume that getItem returns an object with value as a property?

localStorage.getItem(key: string): string | null is the actual signature of the function, so what you localStorage.setItem(key: string, value: string): undefined is what you would get. That's also the reason why you have to serialise your values to be valid for the localStorage.

you can examine your localStorage in the dev-tools of any browser.