fix: Fix logging typo, add TODO comment for process width updates, block final send for deletel operation

Author: lazysegtree

internal/ui/metadata/model.go:35

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

src/internal/file_operations.go

@@ -40,7 +40,7 @@ func zipSources(sources []string, target string, processBar *processbar.Model) e
 		p.DoneTime = time.Now()
 		pSendErr := processBar.SendUpdateProcessMsg(p, true)
 		if pSendErr != nil {
-			slog.Error("Error sending process udpate", "error", pSendErr)
+			slog.Error("Error sending process update", "error", pSendErr)
 		}
 		return errors.New("file already exists")
 	}
@@ -107,7 +107,7 @@ func zipSources(sources []string, target string, processBar *processbar.Model) e
 	p.DoneTime = time.Now()
 	pSendErr := processBar.SendUpdateProcessMsg(p, true)
 	if pSendErr != nil {
-		slog.Error("Error sending process udpate", "error", pSendErr)
+		slog.Error("Error sending process update", "error", pSendErr)
 	}
 	return nil
 }

src/internal/file_operations_compress.go

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

src/internal/file_operations_extract.go

@@ -133,7 +133,7 @@ func deleteOperation(processBarModel *processbar.Model, items []string, useTrash
 		deleteFunc = trashMacOrLinux
 	}
 	for _, item := range items {
-		err := deleteFunc(item)
+		err = deleteFunc(item)
 
 		if err != nil {
 			p.State = processbar.Failed
@@ -149,7 +149,10 @@ func deleteOperation(processBarModel *processbar.Model, items []string, useTrash
 		p.State = processbar.Successful
 	}
 	p.DoneTime = time.Now()
-	processBarModel.TrySendingUpdateProcessMsg(p)
+	err = processBarModel.SendUpdateProcessMsg(p, true)
+	if err != nil {
+		slog.Error("Failed to send final delete operation update", "error", err)
+	}
 	return p.State
 }
 

src/internal/handle_file_operations.go

@@ -32,7 +32,7 @@ func (m *Model) SetDimensions(width int, height int) {
 func (m *Model) SetMetadata(metadata Metadata) {
 	m.metadata = metadata
 	// Note : Dont always reset render to 0
-	// We would have udpate requests coming in during user scrolling through metadata
+	// We would have update requests coming in during user scrolling through metadata
 	m.ResetRenderIfInvalid()
 }
 

src/internal/ui/metadata/model.go

@@ -115,6 +115,10 @@ func (m *Model) Render(processBarFocussed bool) string {
 		}
 		renderedHeight += 3
 
+		// Note : We will be updating this on each Render, although harmless from performance
+		// perspective. We are rendering modified version of the data.
+		// TODO: We could, save pointer of process in map and update progressbar of each
+		// map on each SetWidth. This would be cleaner and more efficient.
 		curProcess := processes[i]
 		curProcess.Progress.Width = m.viewWidth() - 3