From: jimtng <2554958+jimtng@users.noreply.github.com>
Date: Mon, 28 Feb 2022 17:09:57 +0000 (+1000)
Subject: [jrubyscripting] Filter out empty require options (#12386)
X-Git-Url: https://git.basschouten.com/?a=commitdiff_plain;h=166e4ec1b6e487d24865d4feafd21c70e79d49b9;p=openhab-addons.git

[jrubyscripting] Filter out empty require options (#12386)

Signed-off-by: Jimmy Tanagra <jcode@tanagra.id.au>
---

diff --git a/bundles/org.openhab.automation.jrubyscripting/src/main/java/org/openhab/automation/jrubyscripting/internal/JRubyScriptEngineConfiguration.java b/bundles/org.openhab.automation.jrubyscripting/src/main/java/org/openhab/automation/jrubyscripting/internal/JRubyScriptEngineConfiguration.java
index 9e8b8ea1ff..d9f0e29450 100644
--- a/bundles/org.openhab.automation.jrubyscripting/src/main/java/org/openhab/automation/jrubyscripting/internal/JRubyScriptEngineConfiguration.java
+++ b/bundles/org.openhab.automation.jrubyscripting/src/main/java/org/openhab/automation/jrubyscripting/internal/JRubyScriptEngineConfiguration.java
@@ -198,16 +198,16 @@ public class JRubyScriptEngineConfiguration {
             return;
         }
 
-        String[] scripts = requireConfigElement.getValue().get().split(",");
-        for (String script : scripts) {
-            final String requireStatement = String.format("require '%s'", script.trim());
-            try {
-                logger.trace("Injecting require statement: {}", requireStatement);
-                engine.eval(requireStatement);
-            } catch (ScriptException e) {
-                logger.warn("Error evaluating statement {}: {}", requireStatement, e.getMessage());
-            }
-        }
+        Stream.of(requireConfigElement.getValue().get().split(",")).map(s -> s.trim()).filter(s -> !s.isEmpty())
+                .forEach(script -> {
+                    final String requireStatement = String.format("require '%s'", script);
+                    try {
+                        logger.trace("Injecting require statement: {}", requireStatement);
+                        engine.eval(requireStatement);
+                    } catch (ScriptException e) {
+                        logger.warn("Error evaluating statement {}: {}", requireStatement, e.getMessage());
+                    }
+                });
     }
 
     /**