Squirrel/Squirrel.Windows

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

Open

#1.238 geöffnet am 23. Jan. 2018

Auf GitHub ansehen
 (2 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)C++ (1.061 Forks)batch import
bug:failing-test-would-be-nicehelp wanted

Repository-Metriken

Stars
 (7.025 Stars)
PR-Merge-Metriken
 (Keine gemergten PRs in 30 T)

Beschreibung

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

Contributor Guide

Research-Richtung
Untersuche die Logik zur Aktualisierung von Verknüpfungen in UpdateManager.ApplyReleases.cs um Zeile 454. Das Problem besteht darin, dass beim Vergleichen von Verknüpfungsnamen ein einfacher String Vergleich partielle Namen erfasst (z.B. "MyApp" stimmt mit "MyAppTwo" überein). Überprüfe, wie Verknüpfungskennungen verglichen werden, und erwäge die Verwendung exakter Übereinstimmungen oder die Trennung durch ein Trennzeichen.
Tech Stack
cpp
Domain
desktop
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
Halber Tag
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Meist klar
Voraussetzungen
Windows developmentC++
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
40