loadMethods is crashing in v0.0.21 about ground-db

[raix]

I've had this issue myself - I think grounddb is missing a version lock on minimax - not sure - anyways I'm working grounddb at the moment - so there should be a lot of progress the next couple of days.
//try localstorage.clear()

[waeltken]

Thanks for the quick reply. Thats great news. I've been using GroundDB for an earlier demo before and it worked amazingly well ootb.
Tried localStorage.clear() already...

[raix]

I'm doing a rewrite of grounddb at the moment - I hope it will be more stable/easier to test/debug/contribute

[jakobdamjensen]

@raix @waeltken it seems like it's a bug in the load/save object methods which stores a string that can't be parsed again. Adding JSON.stringify and JSON.parse to save and load fixes this. But I'm not sure if this is an ideal solution

// save object into localstorage
_gDB._saveObject = function(name, object) {
  if (_gDB.storage && _gDB._isReloading === false) {
    var cachedDoc = object && EJSON.minify(object);
    try {
      _gDB.storage.setItem(_gDB._getGroundDBPrefix(name), JSON.stringify(cachedDoc));
    } catch (e) {
      GroundDB.onQuotaExceeded();
    }
  }
};

// get object from localstorage, retur null if not found
_gDB._loadObject = function(name) {
  // If storage is supported
  if (_gDB.storage) {
    // Then load cached document
    var cachedDoc = JSON.parse(_gDB.storage.getItem(_gDB._getGroundDBPrefix(name)));
    return (cachedDoc)? EJSON.maxify(cachedDoc): null;
  }
  return null;
};

[waeltken]

@jakobdamjensen: Is this sufficient to get GroundDB working again?

@raix told me he is working on a new restructured release of GroundDB.
I'll try to wait until he releases that and then work from there.

[jakobdamjensen]

@waeltken yes it is.. At least in my demo app..

[waeltken]

I can confirm that this brings v.0.0.21 back to life! I can send you a pull request based on @jakobdamjensen suggestions and v0.0.21 if you like.

[raix]

@waeltken I guess it makes sense to move the refactoring to a separate branch - I'll see if its possible

[raix]

pr yes :)

[raix]

@waeltken master should be at 0.0.21 now

[raix]

I've pushed ground:db should work - closing this