Update logging to use payload.Messages consistently

ziagham · ziagham · commit 642e334c1ac5 · 2025-11-14T23:58:17.000+01:00
Modified logging behavior in InstallPluginCommandOptionsHandler, UninstallPluginCommandOptionsHandler, and UpdatePluginCommandOptionsHandler classes. Replaced `_flowCtlLogger.Write(payload.Data)` with `_flowCtlLogger.Write(payload.Messages)` in the `else` block of the `try` statement to ensure consistent logging of payload messages regardless of operation success. #132
diff --git a/src/FlowCtl/Commands/Plugins/Install/InstallPluginCommandOptionsHandler.cs b/src/FlowCtl/Commands/Plugins/Install/InstallPluginCommandOptionsHandler.cs
@@ -61,7 +61,7 @@ private async Task Execute(InstallPluginCommandOptions options, CancellationToke
             if (payload is { Succeeded: false })
                 _flowCtlLogger.WriteError(payload.Messages);
             else
-                _flowCtlLogger.Write(payload.Data);
+                _flowCtlLogger.Write(payload.Messages);
         }
         catch (Exception ex)
         {
diff --git a/src/FlowCtl/Commands/Plugins/Uninstall/UninstallPluginCommandOptionsHandler.cs b/src/FlowCtl/Commands/Plugins/Uninstall/UninstallPluginCommandOptionsHandler.cs
@@ -51,7 +51,7 @@ private async Task Execute(UninstallPluginCommandOptions options, CancellationTo
             if (payload is { Succeeded: false })
                 _flowCtlLogger.WriteError(payload.Messages);
             else
-                _flowCtlLogger.Write(payload.Data);
+                _flowCtlLogger.Write(payload.Messages);
         }
         catch (Exception ex)
         {
diff --git a/src/FlowCtl/Commands/Plugins/Update/UpdatePluginCommandOptionsHandler.cs b/src/FlowCtl/Commands/Plugins/Update/UpdatePluginCommandOptionsHandler.cs
@@ -56,7 +56,7 @@ private async Task Execute(UpdatePluginCommandOptions options, CancellationToken
             if (payload is { Succeeded: false })
                 _flowCtlLogger.WriteError(payload.Messages);
             else
-                _flowCtlLogger.Write(payload.Data);
+                _flowCtlLogger.Write(payload.Messages);
         }
         catch (Exception ex)
         {

Original file line number	Diff line number	Diff line change
`@@ -61,7 +61,7 @@ private async Task Execute(InstallPluginCommandOptions options, CancellationToke`
`61`	`61`	`if (payload is { Succeeded: false })`
`62`	`62`	`_flowCtlLogger.WriteError(payload.Messages);`
`63`	`63`	`else`
`64`		`- _flowCtlLogger.Write(payload.Data);`
	`64`	`+ _flowCtlLogger.Write(payload.Messages);`
`65`	`65`	`}`
`66`	`66`	`catch (Exception ex)`
`67`	`67`	`{`
Original file line number	Diff line number	Diff line change
`@@ -51,7 +51,7 @@ private async Task Execute(UninstallPluginCommandOptions options, CancellationTo`
`51`	`51`	`if (payload is { Succeeded: false })`
`52`	`52`	`_flowCtlLogger.WriteError(payload.Messages);`
`53`	`53`	`else`
`54`		`- _flowCtlLogger.Write(payload.Data);`
	`54`	`+ _flowCtlLogger.Write(payload.Messages);`
`55`	`55`	`}`
`56`	`56`	`catch (Exception ex)`
`57`	`57`	`{`
Original file line number	Diff line number	Diff line change
`@@ -56,7 +56,7 @@ private async Task Execute(UpdatePluginCommandOptions options, CancellationToken`
`56`	`56`	`if (payload is { Succeeded: false })`
`57`	`57`	`_flowCtlLogger.WriteError(payload.Messages);`
`58`	`58`	`else`
`59`		`- _flowCtlLogger.Write(payload.Data);`
	`59`	`+ _flowCtlLogger.Write(payload.Messages);`
`60`	`60`	`}`
`61`	`61`	`catch (Exception ex)`
`62`	`62`	`{`