CommunityToolkit/WindowsCommunityToolkit

WeakEventListener in NetworkConnectionStateTrigger is duplicated

Open

#4,059 opened on Jun 1, 2021

View on GitHub
 (1 comment) (0 reactions) (1 assignee)C# (5,708 stars) (1,403 forks)batch import
Hacktoberfestbug :bug:good first issuehelp wantedimprovements :sparkles:

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

Contributor guide

Tech stack
csharp
Domain
frontend
Issue type
refactor
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
2
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
under 1 hour
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
stale
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
clear
Prerequisites
C#Windows Community Toolkit structure
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
80
Research direction
The issue identifies a duplicated WeakEventListener in NetworkConnectionStateTrigger.cs (path: Microsoft.Toolkit.Uwp.UI/Triggers/NetworkConnectionStateTrigger.cs). The fix involves removing the private copy and replacing its usage with the existing WeakEventListener in Microsoft.Toolkit.Uwp/Helpers/WeakEventListener.cs. Check the file at line 73 for the duplicated class, then update using directives and references to point to the shared helper. Ensure no breaking changes and that the namespace is correctly imported.