fix(simulator): detect string emptiness properly without helper usage
authorJérôme Benoit <jerome.benoit@sap.com>
Tue, 7 Feb 2023 21:05:01 +0000 (22:05 +0100)
committerJérôme Benoit <jerome.benoit@sap.com>
Tue, 7 Feb 2023 21:05:01 +0000 (22:05 +0100)
Signed-off-by: Jérôme Benoit <jerome.benoit@sap.com>
src/utils/Configuration.ts

index 2cba674d3dd523589e087c7143753588bf26efb3..c71b3ee5098dd87113fe46695f570e860a42f2e2 100644 (file)
@@ -339,13 +339,13 @@ export default class Configuration {
     ) {
       console.error(
         chalk`{green ${Configuration.logPrefix()}} {red Deprecated configuration key '${key}' usage in section '${sectionName}'${
-          logMsgToAppend.trim().length !== 0 && `. ${logMsgToAppend}`
+          logMsgToAppend.trim().length > 0 && `. ${logMsgToAppend}`
         }}`
       );
     } else if (!Configuration.isUndefined(Configuration.getConfig()[key])) {
       console.error(
         chalk`{green ${Configuration.logPrefix()}} {red Deprecated configuration key '${key}' usage${
-          logMsgToAppend.trim().length !== 0 && `. ${logMsgToAppend}`
+          logMsgToAppend.trim().length > 0 && `. ${logMsgToAppend}`
         }}`
       );
     }
@@ -376,7 +376,7 @@ export default class Configuration {
   private static getConfigurationFileWatcher(): fs.FSWatcher | undefined {
     try {
       return fs.watch(Configuration.configurationFile, (event, filename): void => {
-        if (filename?.trim().length !== 0 && event === 'change') {
+        if (filename?.trim().length > 0 && event === 'change') {
           // Nullify to force configuration file reading
           Configuration.configuration = null;
           if (!Configuration.isUndefined(Configuration.configurationChangeCallback)) {
@@ -432,7 +432,7 @@ export default class Configuration {
     logPrefix: string,
     params: HandleErrorParams<EmptyObject> = { throwError: true }
   ): void {
-    const prefix = logPrefix?.trim().length !== 0 ? `${logPrefix} ` : '';
+    const prefix = logPrefix?.trim().length > 0 ? `${logPrefix} ` : '';
     let logMsg: string;
     switch (error.code) {
       case 'ENOENT':