Merge pull request #558 from mountaindude/522

Enhance logging for app replacement in scrambleField function
ptarmiganlabs · Nov 26, 2024 · 33450a4 · 33450a4
2 parents dc039ba + 2567249
commit 33450a4
Showing 1 changed file with 13 additions and 0 deletions.
diff --git a/src/lib/cmd/qseow/scramblefield.js b/src/lib/cmd/qseow/scramblefield.js
@@ -274,6 +274,19 @@ export async function scrambleField(options) {
                         resultReplace = await replaceApp(newAppId, options.newAppCmdId, options);
                         scrambleResult.status = 'success';
                     }
+
+                    if (resultReplace) {
+                        logger.info(
+                            `Replaced existing app "${options.newAppCmdId}" with new, scrambled app "${options.newAppName}" (app ID: ${newAppId})`
+                        );
+                        scrambleResult.cmdDone = 'replace';
+                        scrambleResult.status = 'success';
+                    } else {
+                        logger.error(
+                            `Error replacing existing app "${options.newAppCmdName}" with new, scrambled app "${options.newAppName}"`
+                        );
+                        scrambleResult.status = 'error';
+                    }
                 } else if (options.newAppCmdName) {
                     // Replace by app name
                     // First look up app ID by name