[event-viewer] adds debug loging when command execution throws error

Author: loukaspd

changelog.md

@@ -11,6 +11,7 @@ and this project adheres to [Semantic Versioning](http://semver.org/).
 
 ### Changed
 - **[AppLogger]:**: introduce debug instance
+- **[event-viewer]:**: adds debug loging when command execution throws error
 
 ### Fixed
 - **[log-selection]:** event-logs command parsing when thousands separator is '.' instead of ','

src/app/services/powershell/powershell-command-executor.ts

@@ -1,5 +1,6 @@
 import Shell from 'node-powershell';
 import { GlobalUtils } from '../global-utils';
+import { AppLogger } from '../AppLogger';
 
 
 
@@ -35,7 +36,13 @@ export class PsCommandExecutor {
         .then(() => _ps.addCommand(command))
         .then(() => _ps.invoke())
         .catch(e => {
-            if (!ignoreError) console.error(e);
+            if (!ignoreError) {
+                console.error('The following command threw an error.\n' + command);
+                console.error(e);
+                AppLogger.getDebug().logErrorMessage('The following command threw an error.\n' + command);
+                AppLogger.getDebug().logError(e);
+            }
+
             return '';
         })
         .finally(() => {

src/app/services/powershell/powershell-commands.ts

@@ -50,7 +50,6 @@ export class PowershellCommands {
         if (computerName) command += ` -ComputerName "${computerName}"`;
         command += ' -List';
 
-        AppLogger.getDebug().logDebug(command);
         return PsCommandExecutor.executeCommand(command, false, 90)
         .then(output => PowershellCommands._parseEventViewersList(output))
         .then(events => {
@@ -96,7 +95,7 @@ export class PowershellCommands {
         //add select part
         command += PowershellCommands._selectEventCommand;
         //execute and parse
-        return PsCommandExecutor.executeCommand(command, true)
+        return PsCommandExecutor.executeCommand(command, lastEvent == null)
         .then(output => PowershellCommands._parseEventsCommandOutput(output));
     }