fix empty value in rule and fix some styling

Signed-off-by: Markus Blaschke <[email protected]>

Author: mblaschke

templates/report.css

@@ -115,13 +115,14 @@ h3 { font-size: 1.2rem; }
 }
 
 
-.ms-sel-item {
+body .ms-sel-item,
+body .ms-ctn .ms-sel-item {
     background: #f3f3f3;
-    color: #999;
+    color: var(--bs-body-color);
     float: left;
-    font-size: 12px;
-    padding: 0 5px;
-    border-radius: 3px;
+    font-size: 0.9rem;
+    padding: 0.2rem 0.9rem;
+    border-radius: var(--bs-border-radius);
     border: 1px solid #DDD;
     margin: 1px 5px 1px 0;
 }

templates/report.js

@@ -7,10 +7,26 @@ let globalSettingsKey = "auditor-global-settings";
 let globalSettings = {};
 const resultListDelimiter = ";";
 
-function onlyUnique(value, index, array) {
-    return array.indexOf(value) === index;
+let  onlyUnique = (val, index, array) => {
+    return array.indexOf(val) === index;
 }
 
+let convertListToString = (val) => {
+    return val.join(resultListDelimiter)
+}
+
+let convertStringToList = (val) => {
+    let ret = [];
+
+    val = val.trim();
+    val.split(resultListDelimiter).forEach((item) => {
+        item = item.trim();
+        if (item) {
+            ret.push(item);
+        }
+    })
+    return ret
+}
 
 // restore settings
 try {
@@ -241,7 +257,8 @@ let refreshTableFilter = () => {
                     });
                     break;
                 case "rule":
-                    reportFilter.push({field:fieldName, type:"in", value:el.val().split(resultListDelimiter)});
+                    let valueList = convertStringToList(el.val());
+                    reportFilter.push({field:fieldName, type:"in", value:valueList});
                     break;
                 default:
                     reportFilter.push({field:fieldName, type:"like", value:el.val()});
@@ -326,7 +343,7 @@ table.on("tableBuilt", () => {
 
 
     let ruleSelectorData = [];
-    $("#reportFilterRule").val().split(resultListDelimiter).forEach((val) => {
+    convertStringToList($("#reportFilterRule").val()).forEach((val) => {
         ruleSelectorData.push({
             id: val,
             name: val
@@ -341,7 +358,7 @@ table.on("tableBuilt", () => {
         resultAsStringDelimiter: resultListDelimiter,
     });
     $(ruleMs).on('selectionchange', function(e,m) {
-        $("#reportFilterRule").val(this.getValue().join(resultListDelimiter));
+        $("#reportFilterRule").val(convertListToString(this.getValue()));
         formSaveToHash();
         refreshTableFilter();
     });