CommunityToolkit/WindowsCommunityToolkit

WeakEventListener in NetworkConnectionStateTrigger is duplicated

Open

#4059 aperta il 1 giu 2021

Vedi su GitHub
 (1 commento) (0 reazioni) (1 assegnatario)C# (1403 fork)batch import
Hacktoberfestbug :bug:good first issuehelp wantedimprovements :sparkles:

Metriche repository

Star
 (5708 star)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

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

Guida contributor

Direzione di ricerca
Il problema identifica un WeakEventListener duplicato in NetworkConnectionStateTrigger.cs (percorso: Microsoft.Toolkit.Uwp.UI/Triggers/NetworkConnectionStateTrigger.cs). La correzione comporta la rimozione della copia privata e la sostituzione del suo utilizzo con l'WeakEventListener esistente in Microsoft.Toolkit.Uwp/Helpers/WeakEventListener.cs. Controllare il file alla riga 73 per la classe duplicata, quindi aggiornare le direttive using e i riferimenti in modo che puntino all'helper condiviso. Assicurarsi che non ci siano modifiche di rottura e che il namespace sia importato correttamente.
Tech stack
csharp
Dominio
frontend
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
1
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Meno di 1 ora
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#Git
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
85