ankidroid/Anki-Android

DialogHandler assumes it was started by DeckPicker, may crash otherwise

Open

#5 075 ouverte le 21 oct. 2018

Voir sur GitHub
 (18 commentaires) (0 réactions) (1 assigné)Kotlin (2 789 forks)batch import
2.8.x2.9.xBugHelp WantedKeep OpenPriority-HighRelease process

Métriques du dépôt

Stars
 (11 132 stars)
Métriques de merge PR
 (Merge moyen 9j 12h) (93 PRs mergées en 30 j)

Description

Reproduction Steps
  1. Open AnkiDroid via a Reviewer-first pathway?
  2. request file import?
Expected Result

Import dialog is shown

Actual Result

ClassCastException because import dialog is spawned from Reviewer somehow, instead of DeckPicker?

java.lang.ClassCastException: com.ichi2.anki.Reviewer cannot be cast to com.ichi2.anki.DeckPicker
at com.ichi2.anki.dialogs.DialogHandler.handleMessage(DialogHandler.java:82)
at android.os.Handler.dispatchMessage(Handler.java:106)
at android.os.Looper.loop(Looper.java:164)
at android.app.ActivityThread.main(ActivityThread.java:6501)
at java.lang.reflect.Method.invoke(Native Method)
at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:438)
at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:807)

Guide contributeur

Direction de recherche
Examinez DialogHandler.handleMessage() pour comprendre pourquoi il effectue un cast vers DeckPicker et comment gérer en toute sécurité les messages provenant d'autres activités comme Reviewer. Déterminez si la boîte de dialogue d'importation doit être affichée à partir d'un parent générique ou si le gestionnaire doit être refactorisé pour prendre en charge plusieurs types d'activités.
Stack technique
kotlin
Domaine
mobile
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Plutôt claire
Prérequis
Android development
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70