ankidroid/Anki-Android

DialogHandler assumes it was started by DeckPicker, may crash otherwise

Open

#5.075 aberto em 21 de out. de 2018

Ver no GitHub
 (18 comments) (0 reactions) (1 assignee)Kotlin (2.789 forks)batch import
2.8.x2.9.xBugHelp WantedKeep OpenPriority-HighRelease process

Métricas do repositório

Stars
 (11.132 stars)
Métricas de merge de PR
 (Mesclagem média 9d 12h) (93 fundiu PRs em 30d)

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)

Guia do colaborador

Direção de pesquisa
Investigue o DialogHandler.handleMessage() para entender por que ele faz cast para DeckPicker e como lidar com segurança com mensagens de outras atividades, como Reviewer. Determine se a caixa de diálogo de importação deve ser exibida a partir de um pai genérico ou se o manipulador precisa ser refatorado para suportar vários tipos de atividade.
Pilha de tecnologia
kotlin
Domain
mobile
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Principalmente claro
Prerequisites
Android development
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
70