CommunityToolkit/WindowsCommunityToolkit

WeakEventListener in NetworkConnectionStateTrigger is duplicated

Open

#4 059 ouverte le 1 juin 2021

Voir sur GitHub
 (1 commentaire) (0 réactions) (1 assigné)C# (1 403 forks)batch import
Hacktoberfestbug :bug:good first issuehelp wantedimprovements :sparkles:

Métriques du dépôt

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

Description

Describe the bug

Looks like NetworkConnectionStateTrigger is using a private copy of WeakEventListener, it's identical to the one we already have in the Toolkit, so we should update the source to use the existing code within the Toolkit.

https://github.com/windows-toolkit/WindowsCommunityToolkit/blob/ebb12e0c38c22ec01849e202f8e477d8c1a67061/Microsoft.Toolkit.Uwp.UI/Triggers/NetworkConnectionStateTrigger.cs#L73

Expected behavior

Use existing code over shadow copy.

https://github.com/windows-toolkit/WindowsCommunityToolkit/blob/ebb12e0c38c22ec01849e202f8e477d8c1a67061/Microsoft.Toolkit.Uwp/Helpers/WeakEventListener.cs#L17

Guide contributeur

Direction de recherche
Le problème identifie un WeakEventListener dupliqué dans NetworkConnectionStateTrigger.cs (chemin : Microsoft.Toolkit.Uwp.UI/Triggers/NetworkConnectionStateTrigger.cs). La correction implique de supprimer la copie privée et de remplacer son utilisation par le WeakEventListener existant dans Microsoft.Toolkit.Uwp/Helpers/WeakEventListener.cs. Vérifiez le fichier à la ligne 73 pour la classe dupliquée, puis mettez à jour les directives using et les références pour pointer vers l'assistant partagé. Assurez vous qu'il n'y a pas de changements cassants et que l'espace de noms est correctement importé.
Stack technique
csharp
Domaine
frontend
Type d'issue
Refactorisation
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
1
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Moins d'une heure
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#Git
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
85