xamarin/Xamarin.Forms

Crash when calling ChangeCanExecute() from non-UI thread

Ouverte

#5 679 ouverte le 25 mars 2019

Voir sur GitHubGuide contributeur
 (2 commentaires) (0 réaction) (0 personne assignée)C# (1 926 forks)batch import
e/5 :clock5:help wantedi/highinactivet/bug :bug:up-for-grabs

Métriques du dépôt

Stars
 (5 644 étoiles)
Métriques de merge PR
 (Aucune PR mergée en 30 j)

Description

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

Guide contributeur

Direction de recherche
Le plantage se produit lorsque ChangeCanExecute est appelé depuis un thread non UI. Le problème recommande d'encapsuler les appels dans Device.BeginInvokeOnMainThread. Une correction appropriée consisterait à s'assurer que l'événement CanExecuteChanged est déclenché sur le thread principal, probablement dans la classe Command. Vérifiez le code source de Xamarin.Forms, en particulier le fichier Command.cs, et envisagez d'utiliser SynchronizationContext ou similaire pour marshaler l'événement vers le thread UI.
Stack technique
csharp
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.
Claire
Prérequis
C#Xamarin.Forms
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
75