Remove another unnecessary refresh

stefanhaller · stefanhaller · commit 5be97c61456c · 2025-07-02T15:58:03.000+02:00
This was added after this PR comment: #3276 (comment) > Can we do a refresh after this reset so that the screen shows that the patch > has been cancelled? That way, if we cancel on the next popup, the screen will > be in a valid state. I don't understand what "cancel on the next popup" means; there is no further popup after this code.
diff --git a/pkg/gui/controllers/commits_files_controller.go b/pkg/gui/controllers/commits_files_controller.go
@@ -313,9 +313,6 @@ func (self *CommitFilesController) discard(selectedNodes []*filetree.CommitFileN
 				// Reset the current patch if there is one.
 				if self.c.Git().Patch.PatchBuilder.Active() {
 					self.c.Git().Patch.PatchBuilder.Reset()
-					if err := self.c.Refresh(types.RefreshOptions{Mode: types.BLOCK_UI}); err != nil {
-						return err
-					}
 				}
 
 				for _, node := range selectedNodes {

Original file line number	Diff line number	Diff line change
`@@ -313,9 +313,6 @@ func (self CommitFilesController) discard(selectedNodes []filetree.CommitFileN`
`313`	`313`	`// Reset the current patch if there is one.`
`314`	`314`	`if self.c.Git().Patch.PatchBuilder.Active() {`
`315`	`315`	`self.c.Git().Patch.PatchBuilder.Reset()`
`316`		`- if err := self.c.Refresh(types.RefreshOptions{Mode: types.BLOCK_UI}); err != nil {`
`317`		`- return err`
`318`		`- }`
`319`	`316`	`}`
`320`	`317`
`321`	`318`	`for _, node := range selectedNodes {`