dexie-mongoify icon indicating copy to clipboard operation
dexie-mongoify copied to clipboard

dexie-mongoify not working with latest dexie.js

Open nevf opened this issue 7 years ago • 20 comments

I've just downloaded the latest dexie.js and the latest dexie-mongoify.min.js and am getting "TypeError: Cannot read property 'push' of undefined"

It appears as though quite a bit has changed in dexie.js lately

nevf avatar Oct 13 '16 22:10 nevf

I'll look into it, thanks

YurySolovyov avatar Oct 14 '16 06:10 YurySolovyov

In the meantime, can you try to track down last version that works?

YurySolovyov avatar Oct 14 '16 08:10 YurySolovyov

Thanks for looking into it. I don't know where to get older versions of Dexie. I did have a look in the branches.

My problem is I've just found out that my app isn't working with IndexedDb in Firefox and I'm hoping that moving to the latest version of Dexie will resolve this, but I can't because of the dexie-mongoify issue.

nevf avatar Oct 14 '16 11:10 nevf

@nevf I just fixed some issues with Dexie 1.5, you can grab new dist in master. Check if it works better for you. I the meantime, I'll check Dexie 2

YurySolovyov avatar Oct 15 '16 11:10 YurySolovyov

Filled the issue: https://github.com/dfahlander/Dexie.js/issues/343

YurySolovyov avatar Oct 15 '16 11:10 YurySolovyov

Possible fix:

    var WriteableTable;
    if (db.WriteableTable) {
        WriteableTable = db.WriteableTable;
    } else {
        WriteableTable = db.Table;
    }

    WriteableTable.prototype.update = ...

YurySolovyov avatar Oct 15 '16 11:10 YurySolovyov

@YurySolovyov Thanks for the Dexie 1.5 update. Unfortunately I'm still getting the same error in Firefox.

open() error: Object { _e: Error, name: "UnknownError", message: ""The operation failed for reasons unrelated to the database itself and not covered by any other error code."", inner: DOMError, stack: "" }

I'm unable to access the DB in Firefox Debugger - Storage as IndexedDB is not available otherwise I'd delete it and see if that resolved my issue. It could just be that the DB is screwed. I'll try a few more things.

nevf avatar Oct 16 '16 20:10 nevf

@nevf also try out the branch from #14

YurySolovyov avatar Oct 16 '16 20:10 YurySolovyov

@YurySolovyov Do you mean - try the dexie-2-support branch?

nevf avatar Oct 16 '16 20:10 nevf

Yup

YurySolovyov avatar Oct 16 '16 20:10 YurySolovyov

@YurySolovyov Same problem. This is seriously weird. Ok I've just tried the first Sample at https://github.com/dfahlander/Dexie.js/wiki/Dexie in jsFiddle and am getting the same error. So it would appear to be a Firefox or Dexie issue. I'll dig a bit deeper.

Will you release an master with Dexie 2 once your Dexie issue is resolved?

nevf avatar Oct 16 '16 20:10 nevf

@YurySolovyov Ok I've found this Dexie issue https://github.com/dfahlander/Dexie.js/issues/298 - I'll post there.

nevf avatar Oct 16 '16 20:10 nevf

@YurySolovyov I've reinstalled Firefox and the issue has gone away. Flakey, buggy Firefox! More info @ https://github.com/dfahlander/Dexie.js/issues/298

nevf avatar Oct 16 '16 23:10 nevf

@YurySolovyov I see that indicates the Docs have been updated in the Dexie 2.0.0-beta.2 Release notes.

Are you planning on updating the dexie-mongoify master branch for Dexie V2?

Also could you please include an unminified dexie.mongoify.js version in the dist folder.

Thanks.

nevf avatar Nov 27 '16 21:11 nevf

@nevf if you can confirm that version from dexie-2-support works, I'll merge the PR soon and release the next version.

YurySolovyov avatar Nov 28 '16 06:11 YurySolovyov

@YurySolovyov Yes I'm using the dexie-2-support dist release without any problems.

nevf avatar Nov 29 '16 06:11 nevf

Thanks, I I'll do the release soon

YurySolovyov avatar Nov 29 '16 08:11 YurySolovyov

@nevf sorry it took so long. Released v1.2.3. Feel free to close this if it works for you

YurySolovyov avatar Dec 06 '16 19:12 YurySolovyov

@YurySolovyov Thanks, all good. Could you please include an unminified dexie.mongoify.js version in the dist folder. I'm not into Webpack yet.

nevf avatar Dec 08 '16 20:12 nevf

Sure, I'll ping you later

YurySolovyov avatar Dec 08 '16 20:12 YurySolovyov