nolanlawson/cordova-plugin-sqlite-2

Android: should throw meaningful errors when the 1MB CursorWindow limit is reached

Open

#46 aperta il 18 dic 2016

Vedi su GitHub
 (3 commenti) (0 reazioni) (0 assegnatari)JavaScript (29 fork)github user discovery
enhancementhelp wanted

Metriche repository

Star
 (168 star)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

Currently these errors just get swallowed, e.g.: https://github.com/pouchdb/pouchdb/issues/5413

We just return empty results which causes bugs later on down the line that are hard to diagnose: https://github.com/nolanlawson/cordova-plugin-sqlite-2/blob/33afdef8409b8782b7a82ca7d59270e7e11541dd/src/android/SQLitePlugin.java#L93-L96

Guida contributor

Direzione di ricerca
Esamina il file SQLitePlugin.java, in particolare il metodo execute, e identifica dove l'eccezione del limite CursorWindow viene inghiottita. Implementa un blocco try catch attorno alla query del database per catturare l'eccezione e lanciare un messaggio di errore significativo.
Tech stack
javascriptjava
Dominio
mobile
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Datata
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
GitAndroid SDK
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70