CommunityToolkit/WindowsCommunityToolkit

WeakEventListener in NetworkConnectionStateTrigger is duplicated

Open

#4.059 geöffnet am 1. Juni 2021

Auf GitHub ansehen
 (1 Kommentar) (0 Reaktionen) (1 zugewiesene Person)C# (1.403 Forks)batch import
Hacktoberfestbug :bug:good first issuehelp wantedimprovements :sparkles:

Repository-Metriken

Stars
 (5.708 Stars)
PR-Merge-Metriken
 (Keine gemergten PRs in 30 T)

Beschreibung

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

Contributor Guide

Research-Richtung
Das Problem identifiziert einen duplizierten WeakEventListener in NetworkConnectionStateTrigger.cs (Pfad: Microsoft.Toolkit.Uwp.UI/Triggers/NetworkConnectionStateTrigger.cs). Die Behebung umfasst das Entfernen der privaten Kopie und das Ersetzen ihrer Verwendung durch den vorhandenen WeakEventListener in Microsoft.Toolkit.Uwp/Helpers/WeakEventListener.cs. Überprüfen Sie die Datei in Zeile 73 auf die duplizierte Klasse, aktualisieren Sie dann die using Direktiven und Referenzen, damit sie auf den gemeinsamen Helfer verweisen. Stellen Sie sicher, dass es keine Breaking Changes gibt und der Namespace korrekt importiert wird.
Tech Stack
csharp
Domain
frontend
Issue Type
Refactoring
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
1
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
Unter 1 Stunde
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
C#Git
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
85