capacitor-community / sqlite

⚡Capacitor plugin for native & electron SQLite databases.
MIT License
495 stars 118 forks source link

Error CapacitorSQLitePlugin :null only in production #494

Closed wimZ closed 10 months ago

wimZ commented 10 months ago

Discussed in https://github.com/capacitor-community/sqlite/discussions/493

Originally posted by **wimZ** December 11, 2023 My app used to run fine in producton. After various modifications and plugin updates, it now runs fine in debug mode on a Samsung A53 5G (Android 13) but when I publish to play-store and install on the same device I get error : CapacitorSQLitePlugin :null The error occurs on createConnection Install from Playstore to an older device with Android 9 and it runs fine. I have rolled back recent updates cleaned project reinstalled SQLite including some previous versions I would appreciate any suggestions how to proceed.
jepiqueau commented 10 months ago

@wimZ Not really an idea. May be you should try to invalidate the cache File->Invalidate Caches select all caches and invalidate and restart. and redo the full sequence

npx cap sync
npm run build
npx cap copy
npx cap open android

in Android Studio do

i never published an app to the play-store, i only develop capacitor plugins, so here i cannot help

Hope this will help

Which version of Android Studio ?

wimZ commented 10 months ago

@jepiqueau: The rebuild as described was not successful.

Android Studio is 2023.1.1

;)

ileosebastian commented 10 months ago

I had the same problem, but not in production but with the app-debug that generates the ionic capacitor run android command. Using the app made by Jepiqueau, called ionic7-angular-sqlite-starter, I have been able to solve the problem, to tell the truth, without much insight into how exactly I did it. I thought it was a problem with the capacitor.config.ts file, but it's not. Today I needed to install Capacitor's SplashScreen for my app, installing it as the official Capacitor website suggests, and when I used it only once and re-did the app-debug, I got this message again, that is, CapacitorSQLitePlugin:null. I have uninstalled the SplashScreen plugin and returned to my previous functional build and the problem still exists.

I have been able to analyze that this only happens on certain Android devices, specifically on my physical device that has Android 12, but on Android Studio emulated devices this problem does not occur (I tried APIs 30, 31 and 32).

I've also noticed ambiguities depending on what OS I run my Ionic project on. Since it works perfectly on the physical device if I run it on Manjaro OS.

Post data: This is the first time I speak in a discussion forum here on Git hub, if I have been impertinent in any community rule, I apologize to everyone.

I have attached my package.json below: package.json

ileosebastian commented 10 months ago

@jepiqueau: The rebuild as described was not successful.

Android Studio is 2023.1.1

;)

I did the same process and I got the exact same thing.

rijeshpk commented 10 months ago

@wimZ @ileosebastian I had similar issue when I uninstall and reinstall app. It restores old data and creates the plugin issue you mentioned. Setting allowbackup='false' fixed the issue, which excludes the old app data when reinstalling the app. Below link has changes required to be done for older version of android and new. https://stackoverflow.com/questions/70365809/how-to-specify-to-not-allow-any-data-backup-with-androiddataextractionrules

@jepiqueau

jepiqueau commented 10 months ago

@rijeshpk thnks for this, @wimZ @ileosebastian Can you try what @rijeshpk proposed and tell me if it works. otherwise there is solution to clean the cache programmatically when you re-install the app

wimZ commented 10 months ago

@jepiqueau, I already made these settings with same result.

BTW I use plain capacitor without framework

In order to figure out where it starts to fail, I wanted to create a minimal app and rebuild from there while phased testing Unluckily I run into the following:

After initial setup as per: https://capacitorjs.com/docs/getting-started then npm i @capacitor-community/sqlite This creates a vite build.

Then I get the following result

my index.js (stripped out non-essentials)

import {CapacitorSQLite,SQLiteConnection) from '@capacitor-community/sqlite' const _db = CapacitorSQLite const dbName='abc'

//1 await _db.createConnection({database: dbName,encrypted: false,mode: 'no-encryption',version: 1,})

//2 await _db.open({ database: dbName })

//3 await _db.execute({database: dbName, statements: [DROP TABLE IF NOT EXIST test], values: [],}) Error: Execute: Attempt to invoke virtual method 'java.lang.String java.lang.String.replace(java.lang.CharSequence, java.lang.CharSequence)' on a null object reference at returnResult ((index):917:32) at cap.fromNative ((index):899:17) at :1:18

//4 await _db.query({database: dbName, statement: 'Select from test', values: [], }) Error: Query: in selectSQL cursor no such table: test: , while compiling: Select from test

This result expected, since the table does not exist

jepiqueau commented 10 months ago

@wimZ can you share your minimal app

wimZ commented 10 months ago

How do I share?

Whole folder in zip?

18 dec 2023 17:46:10 QUEAU Jean Pierre @.***>:

@wimZ[https://github.com/wimZ] can you share your minimal app

— Reply to this email directly, view it on GitHub[https://github.com/capacitor-community/sqlite/issues/494#issuecomment-1861021227], or unsubscribe[https://github.com/notifications/unsubscribe-auth/AD2YDNPRRJQRJHA7SSLEOGTYKBXNBAVCNFSM6AAAAABAR6PYW6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNRRGAZDCMRSG4]. You are receiving this because you were mentioned. [Tracking afbeelding][https://github.com/notifications/beacon/AD2YDNMG6DPYV4UMGVJ2DHDYKBXNBA5CNFSM6AAAAABAR6PYW6WGG33NNVSW45C7OR4XAZNMJFZXG5LFINXW23LFNZ2KUY3PNVWWK3TUL5UWJTTO5TXCW.gif]

eideard-hm commented 10 months ago

Any solution, it does not work for me at all

ileosebastian commented 10 months ago

@rijeshpk thnks for this, @wimZ @ileosebastian Can you try what @rijeshpk proposed and tell me if it works. otherwise there is solution to clean the cache programmatically when you re-install the app

In fact @jepiqueau, I can say that what @rijeshpk says works perfectly for my case. That is, I applied everything mentioned in android-quirks (I didn't do it before because I was testing it only in a web environment with SQL queries) and then reinstalled all the packages and added android back to the ionic project to see if it worked. It is indeed an error in the cache and/or permissions that the application has that is causing the CapacitorSQLitePlugin to not be instantiated.

ileosebastian commented 10 months ago

import {CapacitorSQLite,SQLiteConnection) from '@capacitor-community/sqlite' const _db = CapacitorSQLite const dbName='abc'

//1 await _db.createConnection({database: dbName,encrypted: false,mode: 'no-encryption',version: 1,})

//2 await _db.open({ database: dbName })

//3 await _db.execute({database: dbName, statements: [DROP TABLE IF NOT EXIST test], values: [],}) Error: Execute: Attempt to invoke virtual method 'java.lang.String java.lang.String.replace(java.lang.CharSequence, java.lang.CharSequence)' on a null object reference at returnResult ((index):917:32) at cap.fromNative ((index):899:17) at :1:18

//4 await _db.query({database: dbName, statement: 'Select from test', values: [], }) Error: Query: in selectSQL cursor no such table: test: , while compiling: Select from test

This result expected, since the table does not exist

Following only the lines of code that execute the queries from the plugin (but uncommented), the 'test' table is never created in the first instance, so it is correct that the query cannot be made.

wimZ commented 10 months ago

Maybe you noticed the error on 3 the DROP TABLE.

I get the same error on each execute. create and drop query seems to work, since it gives an expected 'no such table'

I removed the CREATE TABLE should have removed the Drop.


19 dec 2023 01:45:13 Leo Intriago @.***>:

import {CapacitorSQLite,SQLiteConnection) from @.***/sqlite' const _db = CapacitorSQLite const dbName='abc'

//1 await _db.createConnection({database: dbName,encrypted: false,mode: 'no-encryption',version: 1,})

//2 await _db.open({ database: dbName })

//3 await _db.execute({database: dbName, statements: [DROP TABLE IF NOT EXIST test], values: [],}) Error: Execute: Attempt to invoke virtual method 'java.lang.String java.lang.String.replace(java.lang.CharSequence, java.lang.CharSequence)' on a null object reference at returnResult ((index):917:32) at cap.fromNative ((index):899:17) at :1:18

//4 await _db.query({database: dbName, statement: 'Select from test', values: [], }) Error: Query: in selectSQL cursor no such table: test: , while compiling: Select from test

This result expected, since the table does not exist

Following only the lines of code that execute the queries from the plugin (but uncommented), the 'test' table is never created in the first instance, so it is correct that the query cannot be made.

— Reply to this email directly, view it on GitHub[https://github.com/capacitor-community/sqlite/issues/494#issuecomment-1861921047], or unsubscribe[https://github.com/notifications/unsubscribe-auth/AD2YDNPHZ3RXWP3SYPOYBRDYKDPRPAVCNFSM6AAAAABAR6PYW6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNRRHEZDCMBUG4]. You are receiving this because you were mentioned. [Tracking afbeelding][https://github.com/notifications/beacon/AD2YDNK24TXFIR3NRS62ESTYKDPRPA5CNFSM6AAAAABAR6PYW6WGG33NNVSW45C7OR4XAZNMJFZXG5LFINXW23LFNZ2KUY3PNVWWK3TUL5UWJTTO7KURO.gif]

jepiqueau commented 10 months ago

@wimZ can you share it on github so i can clone it and test it

wimZ commented 10 months ago

@jepiqueau

You should now have access

Appreciate your support!!!

jepiqueau commented 10 months ago

@wimZ can you give me the link

jepiqueau commented 10 months ago

@wimZ i mean the link to your github repository where you have updated your app so i can clone it quickly

wimZ commented 10 months ago

my 1st time sharing.

https://github.com/wimZ/sqlitetest/invitations

jepiqueau commented 10 months ago

@wimZ i got it and i will have a look

jepiqueau commented 10 months ago

@wimZ Did the bug happens on Android as you do not implement the web part

wimZ commented 10 months ago

Yes its android where it fails

My original app is only android.

jepiqueau commented 10 months ago

@wimZ modify your index.js file as below:

/** @format */

import { SplashScreen } from '@capacitor/splash-screen'
import {
   CapacitorSQLite,
   SQLiteConnection,
} from '@capacitor-community/sqlite'
const _sqlite = CapacitorSQLite
const sqliteConnection = new SQLiteConnection(_sqlite);
let _db
let dbName = 'abcdefghijk',
   result = false

const main = () => {
   SplashScreen.hide()
   document
      .querySelector('#connect-db')
      .addEventListener('click', async function (e) {
         try {
            console.log('creating connection')
            _db = await sqliteConnection.createConnection(
               dbName, false, 'no-encryption', 1, false)
            console.log("$$$$ _db: ",_db)
         } catch (e) {
            console.info(e.message)
         }
      })
   document
      .querySelector('#open-db')
      .addEventListener('click', async function (e) {
         try {
            console.info('opening db')
            await _db.open()
         } catch (e) {
            console.info(e.message)
         }
      })
   document
      .querySelector('#drop-db')
      .addEventListener('click', async function (e) {
         try {
            result = await _db.execute(`DROP TABLE IF EXISTS test;`)
            console.log(result)
         } catch (e) {
            console.info(e.message)
            console.log(e.stack)
         }
      })
   document
      .querySelector('#create-db')
      .addEventListener('click', async function (e) {
         try {
            result = await _db.execute(
               `CREATE TABLE IF NOT EXISTS test ('sleutel','naam')`)
            console.log(result)
         } catch (e) {
            console.info(e.message)
         }
      })
   document
      .querySelector('#insert-db')
      .addEventListener('click', async function (e) {
         try {
            result = await _db.execute(
                  `insert into test (sleutel, naam) VALUES (1, 'truus');`)
            console.log(result)
         } catch (e) {
            console.info(e.message)
         }
         try {
            result = await _db.execute('insert into test (sleutel, naam) VALUES (2, "kees");')
            console.log(result)
         } catch (e) {
            console.info(e.message)
         }
      })
   document
      .querySelector('#query-db')
      .addEventListener('click', async function (e) {
         try {
            result = await _db.query('Select * from test;',[])
            console.log(JSON.stringify(result.values))
            console.log(result)
         } catch (e) {
            console.info(e.message)
         }
      })
}

main()

this will work i will advise you to look at the tutorials Good luck for your development

wimZ commented 10 months ago

Works! Thanks.

The app was migrated from Cordova, and as far as I recall I never modified the way I instantiated sqlite db

I'll check if problem in the app can be solved the same way.

wimZ commented 10 months ago

Tests so far show that my issue is resolved.

Thanks for supporting this.