Forum Replies Created

Viewing 2 replies - 1 through 2 (of 2 total)
  • Thanks for the follow-up Andrew. Digging into the code, I find in looking at the code in autosave.js, it appears that the autosaveLocal() function which is used to initialize the autosave.local property willa return an undefined/null value if the checkStorage() method does not return true (for example, if local storage is disabled or otherwise not available).

    The after-the-dark component of jetpack assumes that this object is always available and the method therein is always present.

    From my read, then, either ATD needs to check for that being present (per my proposed patch), or the core autosave module needs to set a flag which indicates that the methods should not proceed but rather act as a noop and return. (full disclosure: I’m not a JS guru, I’m a server dev by trade, so there may be a third option I’m not aware of).

    I discovered on one of my client sites this evening that the patch above does not completely solve the issue. If the autosave.local object is null, but autosave.server is not, you will still get an exception.

Viewing 2 replies - 1 through 2 (of 2 total)