CommunityToolkit/WindowsCommunityToolkit

WeakEventListener in NetworkConnectionStateTrigger is duplicated

Open

#4,059 opened on 2021年6月1日

GitHub で見る
 (1 comment) (0 reactions) (1 assignee)C# (1,403 forks)batch import
Hacktoberfestbug :bug:good first issuehelp wantedimprovements :sparkles:

Repository metrics

Stars
 (5,708 stars)
PR merge metrics
 (30d に merged PR はありません)

説明

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

コントリビューターガイド

調査方針
この問題は、NetworkConnectionStateTrigger.cs（パス：Microsoft.Toolkit.Uwp.UI/Triggers/NetworkConnectionStateTrigger.cs）内に重複する WeakEventListener を特定しています。修正には、そのプライベートコピーを削除し、その使用を Microsoft.Toolkit.Uwp/Helpers/WeakEventListener.cs 内の既存の WeakEventListener に置き換えることが含まれます。73行目にある重複クラスを確認し、using ディレクティブと参照を更新して共有ヘルパーを指すようにしてください。破壊的変更がないこと、および名前空間が正しくインポートされていることを確認してください。
技術スタック
csharp
領域
frontend
Issue 種別
リファクタリング
難度新しい貢献者にとっての実装難度です。1 は非常に小さな変更、5 は専門的な作業です。
1
推定時間経験ある貢献者が調査、実装、テスト、pull request 準備にかけるおおよその時間範囲です。
1時間未満
活動状況Issue が今どれだけ取り組みやすいかを示します。新しい、活発、古い、ブロック中、またはメンテナー入力待ちなどです。
アクティブ
明確さ期待される変更、受け入れ条件、次の手順がどれだけ明確かを示します。
明確
前提条件
C#Git
初心者向け度初回貢献者にどれだけ取り組みやすいかを 1-100 で推定したスコアです。
85