Android not saving the inserts [workaround needed]

[chintanop]

@brodybits Seems like a bunch of folks are facing this issue: on Android: https://groups.google.com/forum/#!topic/Cordova-SQLitePlugin/oq8wMDSH4M4

Cordova: 3.5.0
Android API Level: 19

Any idea?

[brodycj]

It sounds like an issue introduced by recent updates to the Android sqlite classes. SQLite itself should never do this: https://www.sqlite.org/transactional.html

[brodycj]

From December 2014:

On Tue, Dec 16, 2014 at 1:48 PM, mladenpetrovic88@gmail.com wrote:

I have been using this plugin for some time now, and recently i noticed that
sometimes plugin just doesn't insert anything in database. From the several
things that need to be initialized on app start, nothing is inserted, and of
course my whole app breaks.
I noticed this after updates to android support jar, cordova and few other
things.
I can't think of any reason why something like this would occur so please
any tips would be helpful.
Thank you

On Wed, Dec 17, 2014 at 1:59 PM, mladenpetrovic88@gmail.com wrote:

After hours of trying to figure out why/when does db fails i finally figured
it out.
When i start the app then exit it (not close/swipe it) and return back to
app sqlite plugin notifies me again in console that database is opened
again

SQLitePlugin openargs: {"name":"insideoutdb","bgType":1}
I am using cordova 4.0.0, Ionic 1.2.8 and sqlite 3.8.6 on lollipop and
kitkat.
My understanding is that database isn't closed until you explicitly close
the app ( or android kills it ) so when i come back to my app it tries to
open already opened db and it overwrites it or crash.

From February 2015:

On Sun, Feb 15, 2015 at 12:48 AM, Cesar Fernandes cesar.fa@gmail.com wrote:

I have the same problem.... I am working with latest versions of cordova
and plugin...

I manage 3 sqlite tables. I make some inserts in one of the tables, and when
i close the app and come back, the inserts get lost... then for some reason
that i can not find out, i populate the second table with lots of entries
from one web service, and this one does not get lost.... i already tried
lots of things to solve this problem, but i am almost givin up...

I alway get messages that database is locked , or database was closed bu
remained 1 without closing....

On Sun, Mar 1, 2015 at 1:54 AM, COP chintan@trialx.com wrote:

Hi, I'm also having the same problem on Android

The inserts work fine till the APP is on, i.e. you can insert and select the
data.

Once, you KILL the app. All the inserts are gone! Works fine in iOS - the
Android is not working.

[brodycj]

What I am suspecting now is the following commit in the Android database classes: aosp-mirror/platform_external_sqlite@d4f30d0

It references the following sqlite fossil commit: http://www.sqlite.org/src/info/6c4c2b7dba

[brodycj]

I made a possible workaround in the branch xb-android-lock-workaround. Can you please let us know if this will solve the problem?

[chintanop]

@brodybits Thanks a lot for quick turn-around! this worked!

[brodycj]

Happy that it solves your problem. But it points to an issue with the Android sqlite library. My idea is to make this workaround an option for those who need it.

[brodycj]

The workaround as proposed was validated by @chintanop and is now optional, as described in README.md.

[cesar1983]

What files should I overwrite from the workaround branch? I replaced only SQLitePlugin.java and SQLitePlugin.js , but seems to be still wrong

[brodycj]

The workaround branch is no longer supported. Please completely remove your app from the Android simulator or device, install the latest update from the master branch of this plugin, and use the androidLockWorkaround option in sqlitePlugin.openDatabase, as described in the readme.

[cesar1983]

Ok! thank you very much! I will play with that now...

[brodycj]

Also note that this is really a bug in the Android database libraries and not this project. Unfortunately I have not reproduced this issue myself. I have implemented a very crude workaround but this should really be fixed in the Android code itself!

As I said before, I have a theory that the issue was introduced in the following commit in the Android database classes: android/platform_external_sqlite@d4f30d0, which references the following sqlite fossil commit: http://www.sqlite.org/src/info/6c4c2b7dba

It would really help if someone could file a bug on the Android project itself at: https://code.google.com/p/android/issues/list

[cesar1983]

I really appreciate your help. I will write to Android developers like you
told...

Tks again
Em 20/03/2015 22:19, "Chris Brody" notifications@github.com escreveu:

Also note that this is really a bug in the Android database libraries and
not this project. Unfortunately I have not reproduced this issue
myself. I have implemented a very crude workaround but this should
really be fixed in the Android code itself!

As I said before, I have a theory that the issue was introduced in the
following commit in the Android database classes:
aosp-mirror/platform_external_sqlite@d4f30d0
aosp-mirror/platform_external_sqlite@d4f30d0,
which references the following sqlite fossil commit:
http://www.sqlite.org/src/info/6c4c2b7dba

It would really help if someone could file a bug on the Android project
itself at: https://code.google.com/p/android/issues/list

Reply to this email directly or view it on GitHub
#193 (comment)
.

[cesar1983]

Yeah, i think i ll have to write to android developers.. sorry because i
was hopefull.. i have to finish this project soon... :/

I re-installed the plugin it is still not working.. i can see in the catlog
the workaround is being called "Android db closing/locking workaround
applied" ...

I use a moto G android 4.4 ...

When i emulate it in eclipse, it is working , but not on my device.

On Fri, Mar 20, 2015 at 10:18 PM, a Cesar Fernandes cesar.fa@gmail.com
wrote:

I really appreciate your help. I will write to Android developers like you
told...

Tks again
Em 20/03/2015 22:19, "Chris Brody" notifications@github.com escreveu:

Also note that this is really a bug in the Android database libraries and

not this project. Unfortunately I have not reproduced this issue
myself. I have implemented a very crude workaround but this should
really be fixed in the Android code itself!

As I said before, I have a theory that the issue was introduced in the
following commit in the Android database classes:
aosp-mirror/platform_external_sqlite@d4f30d0
aosp-mirror/platform_external_sqlite@d4f30d0,
which references the following sqlite fossil commit:
http://www.sqlite.org/src/info/6c4c2b7dba

It would really help if someone could file a bug on the Android project
itself at: https://code.google.com/p/android/issues/list

Reply to this email directly or view it on GitHub
#193 (comment)
.

Café Hostel
Rua Agissê, 152 - Vl. Madalena
+55 11 2649-7217
http://www.cafehostel.com.br
https://www.facebook.com/cafehostelsp

[brodycj]

That would really help me.

[cherrypickercherrypicker]

Does this issue occurs only when you use db.transaction or on db.executeSql too?

[brodycj]

Does this issue occurs only when you use db.transaction or on db.executeSql too?

Unfortunately I have not reproduced this issue myself so far. The workaround is only implemented in the case of using db.transaction().

[mdbw3]

I'm running into this issue also. Does it also occur when opening the database with androidDatabaseImplementation: 2?

As it occurs occasionally, testing / debugging is quite a challenge.

[brodycj]

I'm running into this issue also. Does it also occur when opening the database with androidDatabaseImplementation: 2?

As it occurs occasionally, testing / debugging is quite a challenge.

This should only be an issue if you open the database with androidDatabaseImplementation: 2 setting. If you see this issue with the default settings then please raise a new issue along with some details that can help me reproduce the problem.

As documented, there is an androidLockWorkaround that you can use in case you encounter this issue. Please read and follow the documentation very carefully if you need to use this setting.

[mdbw3]

Ok, thanks. I'll look into it and open a new issue if I can't find a related item or solution.

Edit: the example code at the 'Workaround for Android db locking issue' section is a little bit confusing:
var db = window.sqlitePlugin.openDatabase({name: 'my.db', location: 'default', androidLockWorkaround: 1});

The 'androidDatabaseImplementation: 2 ' setting isn't set, so the example code doesn't make sense, right?

[brodycj]

The 'androidDatabaseImplementation: 2 ' setting isn't set, so the example code doesn't make sense, right?

@mdbw3 the README.md has been updated, thanks.

[craig-at-rsg]

I'm having this problem, but I don't use androidDatabaseImplementation: 2, and the data is lost without closing the app. Specifically, I insert a row and get back the inserted id, then the next insert to that table gets back the same insert id.

One potential cause is that I'm opening the database read-only in some Android plugin code. My understanding was that read-only wouldn't affect a read-write connection, but maybe that's just for Sqlite itself and not for something that wraps it (Android code or this plugin).

What info would be useful in tracking this down? Or is there already a workaround (can I use androidLockWorkaround: 1 without androidDatabaseImplementation: 2)? The docs seem to say that won't help.

[brodycj]

Hi @craig-at-rsg,

That really sounds like a different problem. I suggest we discuss this in a new issue. Thanks!

[craig-at-rsg]

@brodybits Thanks for getting back to me so fast. I just opened #601 for this.

[dex252]

I am writing through a translator. Hello, I will raise this problem again. It occurs when UPDATE records in the database. I would like to use the solution proposed in the documentation in the "Workaround for Android db locking issue" section, but the constructor lacks androidDatabaseProvider and androidLockWorkaround. Are there any other solutions to this problem?
I am sure that the method works without exception and the input data is correct, but in practice nothing happens. After UPDATE it follows: .then, I don’t know what it returns ... I know that catch does not work out. It is strange that this does not happen for all records, and only after the first successful UPDATE record. Other records work fine, but favorites will work until the first change, the second and the next ones no longer work.

[brodycj]

Please raise a new issue with some information about what is happening, how you open the database, how you store data, and what kind of devices you see the the issue on. I generally do not support discussions in closed issues.