Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

res.rows.length undefined when exceed 1000 #41

Closed
alaaapp opened this issue Aug 12, 2016 · 5 comments
Closed

res.rows.length undefined when exceed 1000 #41

alaaapp opened this issue Aug 12, 2016 · 5 comments

Comments

@alaaapp
Copy link

alaaapp commented Aug 12, 2016

Hi,
First thank you for this great plugin, its helping me alot.
The issue is described in the title. it's really strange...although i can get all data using LIMIT statement by slicing, any help?
thx, Al

@nolanlawson
Copy link
Owner

Can you provide more detail or a test case to reproduce?

@gevyaniv
Copy link

gevyaniv commented Aug 4, 2017

Hello Nolan,

I am a big fan of your work and I wanted to thank you.

Maybe I can provide a few more details.
I have been using this plugin for a while, and recently, while working with a big document in the DB, I got this error message

`Uncaught TypeError: Cannot read property 'length' of null cordova.js:314
safeJsonParse @ pouchdb.js:3979

(anonymous function) @ pouchdb.js:6865

(anonymous function) @ sqlite-plugin.js:4439

onSuccess @ sqlite-plugin.js:4741

callbackFromNative @ cordova.js:293

processMessage @ cordova.js:1081

processMessages @ cordova.js:1104

pollOnce @ cordova.js:973

pollOnceFromOnlineEvent @ cordova.js:960`

It seems like when reading a big document, the success callback is being called with empty result which causes this error.

This happens when I use the plugin with Android and works well on iOS.

I tried a different plugin and it worked, but I was really hoping this can be fixed since we have been using your plugin for a while and we prefer it.

I hope this helps

@nolanlawson
Copy link
Owner

Which version of Android? Also could be related to #39 .

@nolanlawson
Copy link
Owner

Closing old issues, please reopen if there is a test case or details that can reproduce this.

@nolanlawson
Copy link
Owner

also this may be the core issue: #46

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants