xamarin/Xamarin.Forms

Crash when calling ChangeCanExecute() from non-UI thread

Aperta

#5679 aperta il 25 mar 2019

Vedi su GitHubGuida contributor
 (2 commenti) (0 reazioni) (0 assegnatari)C# (1926 fork)batch import
e/5 :clock5:help wantedi/highinactivet/bug :bug:up-for-grabs

Metriche repository

Star
 (5644 stelle)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

Description

App crashes when running async method which triggers ChangeCanExecute(). I think this should the responsibility of the renderers to trigger any visual updates on the UI thread. The workaround for this problem is to wrap all ChangeCanExecute() in Device.BeginInvokeOnMainThread()

Steps to Reproduce

Run async methods which triggers ChangeCanExecute(). You can i.e. create a button with async Command which changes one of the binding property and this binding property raise ChangeCanExecute() in the setter.

Expected Behavior

App should not crash.

Actual Behavior

App crashes when running async method which triggers ChangeCanExecute(). The problem exists both on Android and iOS.

Basic Information

  • Version with issue: Xamarin Forms 3.6.0.264807
  • Last known good version:
  • IDE: Visual Studio 2017 15.9.4
  • Platform Target Frameworks:
    • iOS: 8.0
    • Android: 8.1

Reproduction Project

Here is a project with a repro:

App1.zip

Guida contributor

Direzione di ricerca
Il crash si verifica quando ChangeCanExecute viene chiamato da un thread non UI. Il problema consiglia di racchiudere le chiamate in Device.BeginInvokeOnMainThread. Una correzione appropriata comporterebbe garantire che l'evento CanExecuteChanged venga sollevato sul thread principale, probabilmente nella classe Command. Controlla il codice sorgente di Xamarin.Forms, in particolare il file Command.cs, e considera l'uso di SynchronizationContext o simili per trasferire l'evento al thread UI.
Tech stack
csharp
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.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
C#Xamarin.Forms
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
75