chore: review updates

RRashmit · RRashmit · commit 8c3e473412ed · 2025-03-13T14:52:34.000Z
diff --git a/internal/file/file_manager_service.go b/internal/file/file_manager_service.go
@@ -313,9 +313,10 @@ func (fms *FileManagerService) ConfigApply(ctx context.Context,
 	if fileErr != nil {
 		return model.RollbackRequired, fileErr
 	}
+	fileOverviewFiles := files.ConvertToMapOfFiles(fileOverview.GetFiles())
 	// Update map of current files on disk
-	fms.UpdateCurrentFilesOnDisk(files.ConvertToMapOfFiles(fileOverview.GetFiles()))
-	manifestFileErr := fms.UpdateManifestFile(files.ConvertToMapOfFiles(fileOverview.GetFiles()))
+	fms.UpdateCurrentFilesOnDisk(fileOverviewFiles)
+	manifestFileErr := fms.UpdateManifestFile(fileOverviewFiles)
 	if manifestFileErr != nil {
 		return model.RollbackRequired, manifestFileErr
 	}

Original file line number	Diff line number	Diff line change
`@@ -313,9 +313,10 @@ func (fms *FileManagerService) ConfigApply(ctx context.Context,`
`313`	`313`	`if fileErr != nil {`
`314`	`314`	`return model.RollbackRequired, fileErr`
`315`	`315`	`}`
	`316`	`+ fileOverviewFiles := files.ConvertToMapOfFiles(fileOverview.GetFiles())`
`316`	`317`	`// Update map of current files on disk`
`317`		`- fms.UpdateCurrentFilesOnDisk(files.ConvertToMapOfFiles(fileOverview.GetFiles()))`
`318`		`- manifestFileErr := fms.UpdateManifestFile(files.ConvertToMapOfFiles(fileOverview.GetFiles()))`
	`318`	`+ fms.UpdateCurrentFilesOnDisk(fileOverviewFiles)`
	`319`	`+ manifestFileErr := fms.UpdateManifestFile(fileOverviewFiles)`
`319`	`320`	`if manifestFileErr != nil {`
`320`	`321`	`return model.RollbackRequired, manifestFileErr`
`321`	`322`	`}`