Squirrel/Squirrel.Windows

During update and uninstall some unrelated shortcuts pinned to the taskbar may get updated

Open

#1.238 aberto em 23 de jan. de 2018

Ver no GitHub
 (2 comments) (0 reactions) (0 assignees)C++ (1.061 forks)batch import
bug:failing-test-would-be-nicehelp wanted

Métricas do repositório

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

Description

Two, or more applications with names that begin the same for example:

MyApp and MyAppTwo

If I have both the apps pinned on the taskbar and I update or uninstall MyApp the shortcuts of the other app MyAppTwo pinned to the taskbar will be updated too.

Could this be the issue? https://github.com/Squirrel/Squirrel.Windows/blob/b25c9fa1e4d60f8969137532a95bd53bb5a8de1f/src/Squirrel/UpdateManager.ApplyReleases.cs#L454

Guia do colaborador

Direção de pesquisa
Investigue a lógica de atualização de atalhos em UpdateManager.ApplyReleases.cs por volta da linha 454. O problema é que, ao comparar nomes de atalhos, uma comparação simples de strings corresponde a nomes parciais (por exemplo, "MyApp" corresponde a "MyAppTwo"). Veja como os identificadores de atalho são comparados e considere usar correspondência exata ou separar por um delimitador.
Pilha de tecnologia
cpp
Domain
desktop
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
3
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
Meio dia
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.
Principalmente claro
Prerequisites
Windows developmentC++
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
40