fix: comments

Author: lazysegtree

src/internal/file_operations.go

@@ -210,15 +210,15 @@ func pasteDir(src, dst string, p *processbar.Process, cut bool, processBarModel
 
 		if err != nil {
 			p.State = processbar.Failed
-			pSendErr := processBarModel.SendUpdateProcessNameMsg(*p, true)
+			pSendErr := processBarModel.SendUpdateProcessMsg(*p, true)
 			if pSendErr != nil {
 				slog.Error("Error sending process udpate", "error", pSendErr)
 			}
 			return err
 		}
 
 		p.Done++
-		processBarModel.TrySendingUpdateProcessNameMsg(*p)
+		processBarModel.TrySendingUpdateProcessMsg(*p)
 		return nil
 	})
 

src/internal/file_operations_compress.go

@@ -38,7 +38,7 @@ func zipSources(sources []string, target string, processBar *processbar.Model) e
 		p.Name = icon.CompressFile + icon.Space + "File already exist"
 		p.State = processbar.Cancelled
 		p.DoneTime = time.Now()
-		pSendErr := processBar.SendUpdateProcessNameMsg(p, true)
+		pSendErr := processBar.SendUpdateProcessMsg(p, true)
 		if pSendErr != nil {
 			slog.Error("Error sending process udpate", "error", pSendErr)
 		}
@@ -90,7 +90,7 @@ func zipSources(sources []string, target string, processBar *processbar.Model) e
 				return err
 			}
 			p.Done++
-			processBar.TrySendingUpdateProcessNameMsg(p)
+			processBar.TrySendingUpdateProcessMsg(p)
 			return nil
 		})
 		if err != nil {
@@ -105,7 +105,7 @@ func zipSources(sources []string, target string, processBar *processbar.Model) e
 		p.Done = totalFiles
 	}
 	p.DoneTime = time.Now()
-	pSendErr := processBar.SendUpdateProcessNameMsg(p, true)
+	pSendErr := processBar.SendUpdateProcessMsg(p, true)
 	if pSendErr != nil {
 		slog.Error("Error sending process udpate", "error", pSendErr)
 	}

src/internal/file_operations_extract.go

@@ -34,7 +34,7 @@ func extractCompressFile(src, dest string, processBar *processbar.Model) error {
 	}
 
 	p.DoneTime = time.Now()
-	pSendErr := processBar.SendUpdateProcessNameMsg(p, true)
+	pSendErr := processBar.SendUpdateProcessMsg(p, true)
 	if pSendErr != nil {
 		slog.Error("Error sending process udpate", "error", pSendErr)
 	}

src/internal/handle_file_operations.go

@@ -142,14 +142,14 @@ func deleteOperation(processBarModel *processbar.Model, items []string, useTrash
 		}
 		p.Name = icon.Delete + icon.Space + filepath.Base(item)
 		p.Done++
-		processBarModel.TrySendingUpdateProcessNameMsg(p)
+		processBarModel.TrySendingUpdateProcessMsg(p)
 	}
 
 	if p.State != processbar.Failed {
 		p.State = processbar.Successful
 	}
 	p.DoneTime = time.Now()
-	processBarModel.TrySendingUpdateProcessNameMsg(p)
+	processBarModel.TrySendingUpdateProcessMsg(p)
 	return p.State
 }
 
@@ -339,14 +339,14 @@ func executePasteOperation(processBarModel *processbar.Model,
 		}
 
 		p.Done++
-		processBarModel.TrySendingUpdateProcessNameMsg(p)
+		processBarModel.TrySendingUpdateProcessMsg(p)
 	}
 
 	if p.State != processbar.Failed {
 		p.State = processbar.Successful
 	}
 	p.DoneTime = time.Now()
-	err = processBarModel.SendUpdateProcessNameMsg(p, true)
+	err = processBarModel.SendUpdateProcessMsg(p, true)
 	if err != nil {
 		slog.Error("Could not send final update for process Bar", "error", err)
 	}

src/internal/ui/processbar/model_update.go

@@ -9,7 +9,11 @@ func (m *Model) ListenForChannelUpdates() {
 	// A goroutine running forever
 	go func() {
 		for {
-			msg := <-m.msgChan
+			msg, ok := <-m.msgChan
+			if !ok {
+				slog.Debug("Channel closed, stopping listener")
+				return
+			}			
 			slog.Debug("Received message", "id", msg.GetReqID())
 			if _, ok := msg.(stopListeningMsg); ok {
 				return
@@ -67,13 +71,13 @@ func (m *Model) SendAddProcessMsg(name string, total int, blockingSend bool) (Pr
 	return p, nil
 }
 
-func (m *Model) SendUpdateProcessNameMsg(p Process, blockingSend bool) error {
+func (m *Model) SendUpdateProcessMsg(p Process, blockingSend bool) error {
 	msg := updateProcessMsg{NewProcess: p, BaseMsg: BaseMsg{reqID: m.newReqCnt()}}
 	return m.sendMsgToChannel(msg, blockingSend)
 }
 
 // Non Blocking and can fail
-func (m *Model) TrySendingUpdateProcessNameMsg(p Process) {
+func (m *Model) TrySendingUpdateProcessMsg(p Process) {
 	msg := updateProcessMsg{NewProcess: p, BaseMsg: BaseMsg{reqID: m.newReqCnt()}}
 	err := m.sendMsgToChannel(msg, false)
 	if err != nil {

src/internal/ui/processbar/model_utils.go

@@ -41,8 +41,8 @@ func (m *Model) getSortedProcesses() []Process {
 	}
 	// sort by the process
 	sort.Slice(processes, func(i, j int) bool {
-		doneI := (processes[i].State == Successful)
-		doneJ := (processes[j].State == Successful)
+		doneI := (processes[i].State == Successful || processes[i].State == Failed)
+		doneJ := (processes[j].State == Successful || processes[j].State == Failed)
 
 		// sort by done or not
 		if doneI != doneJ {