Code Quality: Changed the count check condition in HotKey.cs to avoid IndexOutOfRangeException (#17373)

hishitetsu · web-flow · commit 8cf17eb0e03d · 2025-07-30T21:02:34.000-04:00
diff --git a/src/Files.App/Data/Commands/HotKey/HotKey.cs b/src/Files.App/Data/Commands/HotKey/HotKey.cs
@@ -308,7 +308,7 @@ public static HotKey Parse(string code, bool localized = true)
 				{
 					parts = [code];
 				}
-				else if (parts.Count > 0 && string.IsNullOrEmpty(parts.Last()))
+				else if (parts.Count > 1 && string.IsNullOrEmpty(parts.Last()))
 				{
 					// If the last part is empty, remove it and add a "+" to the last non-empty part
 					parts.RemoveAt(parts.Count - 1);

Original file line number	Diff line number	Diff line change
`@@ -308,7 +308,7 @@ public static HotKey Parse(string code, bool localized = true)`
`308`	`308`	`{`
`309`	`309`	`parts = [code];`
`310`	`310`	`}`
`311`		`- else if (parts.Count > 0 && string.IsNullOrEmpty(parts.Last()))`
	`311`	`+ else if (parts.Count > 1 && string.IsNullOrEmpty(parts.Last()))`
`312`	`312`	`{`
`313`	`313`	`// If the last part is empty, remove it and add a "+" to the last non-empty part`
`314`	`314`	`parts.RemoveAt(parts.Count - 1);`