Squirrel/Squirrel.Windows

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

Open

#1,238 创建于 2018年1月23日

在 GitHub 查看
 (2 评论) (0 反应) (0 负责人)C++ (1,061 fork)batch import
bug:failing-test-would-be-nicehelp wanted

仓库指标

Star
 (7,025 star)
PR 合并指标
 (30 天内没有已合并 PR)

描述

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

贡献者指南

研究方向
调查 UpdateManager.ApplyReleases.cs 中大约第454行的快捷方式更新逻辑。问题在于比较快捷方式名称时，简单的字符串比较会匹配部分名称（例如“MyApp”匹配“MyAppTwo”）。查看快捷键标识符的比较方式，考虑使用精确匹配或通过分隔符区分。
技术栈
cpp
领域
desktop
议题类型
缺陷
难度面向新贡献者的预计实现难度，1 表示很小改动，5 表示专家级工作。
3
预计时间有经验贡献者完成调查、实现、测试并准备 pull request 的粗略时间范围。
半天
活动状态议题当前的可参与程度：新鲜、活跃、陈旧、阻塞或等待维护者输入。
活跃
清晰度议题是否清楚说明期望改动、验收标准和下一步。
基本清晰
前置要求
Windows developmentC++
新手友好度1-100 的估计分数，表示该议题对首次贡献者的友好程度。
40