CommunityToolkit/WindowsCommunityToolkit

WeakEventListener in NetworkConnectionStateTrigger is duplicated

Open

#4.059 aberto em 1 de jun. de 2021

Ver no GitHub
 (1 comment) (0 reactions) (1 assignee)C# (1.403 forks)batch import
Hacktoberfestbug :bug:good first issuehelp wantedimprovements :sparkles:

Métricas do repositório

Stars
 (5.708 stars)
Métricas de merge de PR
 (Nenhuma PRs mesclada em 30d)

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

Guia do colaborador

Direção de pesquisa
O problema identifica um WeakEventListener duplicado em NetworkConnectionStateTrigger.cs (caminho: Microsoft.Toolkit.Uwp.UI/Triggers/NetworkConnectionStateTrigger.cs). A correção envolve remover a cópia privada e substituir seu uso pelo WeakEventListener existente em Microsoft.Toolkit.Uwp/Helpers/WeakEventListener.cs. Verifique o arquivo na linha 73 para a classe duplicada, depois atualize as diretivas using e as referências para apontar para o auxiliar compartilhado. Certifique se de que não haja alterações que quebrem a compatibilidade e que o namespace seja importado corretamente.
Pilha de tecnologia
csharp
Domain
frontend
Tipo Issue
Refatoração
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
1
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
Menos de 1 hora
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
C#Git
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
85