Empty string handling fixes
authorJérôme Benoit <jerome.benoit@sap.com>
Wed, 25 Jan 2023 22:42:33 +0000 (23:42 +0100)
committerJérôme Benoit <jerome.benoit@sap.com>
Wed, 25 Jan 2023 22:42:33 +0000 (23:42 +0100)
Signed-off-by: Jérôme Benoit <jerome.benoit@sap.com>
src/charging-station/AuthorizedTagsCache.ts
src/charging-station/ChargingStation.ts
src/charging-station/ChargingStationUtils.ts
src/utils/Configuration.ts

index 5808f02b57f9a01b6fde9c1d75b281751b68fedd..a2b180e330600b37873d7c37b0e01c52edf2e3c6 100644 (file)
@@ -35,7 +35,7 @@ export default class AuthorizedTagsCache {
             file,
             null,
             (event, filename) => {
-              if (filename && event === 'change') {
+              if (!Utils.isEmptyString(filename) && event === 'change') {
                 try {
                   logger.debug(
                     `${this.logPrefix(file)} ${FileType.Authorization} file have changed, reload`
index ddaff652478e6d6898eef112dd3fcc6b2713add9..276c836866970585c2cef6923c6dfd1dcb89a746 100644 (file)
@@ -511,7 +511,7 @@ export default class ChargingStation {
           this.templateFile,
           null,
           (event, filename): void => {
-            if (filename && event === 'change') {
+            if (!Utils.isEmptyString(filename) && event === 'change') {
               try {
                 logger.debug(
                   `${this.logPrefix()} ${FileType.ChargingStationTemplate} ${
index 9b371d140732db0bbcb7cff08b864ea9f2a1cd8d..e73f4d50670628cd61dfc82a2604ef7f9ae2d9fe 100644 (file)
@@ -210,7 +210,7 @@ export class ChargingStationUtils {
     if (!Utils.isUndefined(template[key])) {
       logger.warn(
         `${logPrefix} Deprecated template key '${key}' usage in file '${templateFile}'${
-          logMsgToAppend && `. ${logMsgToAppend}`
+          !Utils.isEmptyString(logMsgToAppend) && `. ${logMsgToAppend}`
         }`
       );
     }
index 5df7bdc5521debb1091388c38620497fa1eee96b..b1926e16dad7425d3cd7ea8a257163c41a4a7e14 100644 (file)
@@ -336,13 +336,13 @@ export default class Configuration {
     ) {
       console.error(
         chalk`{green ${Configuration.logPrefix()}} {red Deprecated configuration key '${key}' usage in section '${sectionName}'${
-          logMsgToAppend && `. ${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 && `. ${logMsgToAppend}`
+          logMsgToAppend.trim().length !== 0 && `. ${logMsgToAppend}`
         }}`
       );
     }