fix: ${files:N} will always be replaced with the first change path (#487)

love-linger · love-linger · commit cc66afe5e251 · 2024-10-14T15:31:29.000+08:00
* wrong loop index passed to `changes`
diff --git a/src/Models/CommitTemplate.cs b/src/Models/CommitTemplate.cs
@@ -55,7 +55,7 @@ public string Apply(List<Change> changes)
                 {
                     var count = Math.Min(int.Parse(countStr.Substring(1)), changes.Count);
                     for (int j = 0; j < count; j++)
-                        paths.Add(changes[i].Path);
+                        paths.Add(changes[j].Path);
 
                     if (count < changes.Count)
                         more = $" and {changes.Count - count} other files";

Original file line number	Diff line number	Diff line change
`@@ -55,7 +55,7 @@ public string Apply(List<Change> changes)`
`55`	`55`	`{`
`56`	`56`	`var count = Math.Min(int.Parse(countStr.Substring(1)), changes.Count);`
`57`	`57`	`for (int j = 0; j < count; j++)`
`58`		`- paths.Add(changes[i].Path);`
	`58`	`+ paths.Add(changes[j].Path);`
`59`	`59`
`60`	`60`	`if (count < changes.Count)`
`61`	`61`	`more = $" and {changes.Count - count} other files";`