fix: include value type check for numbers

decipher3114 · Dec 22, 2024 · 4841a61 · 4841a61
1 parent c0476a6
commit 4841a61
Showing 1 changed file with 4 additions and 0 deletions.
diff --git a/modules/json/options.sh b/modules/json/options.sh
@@ -176,6 +176,10 @@ editOptions() {
                 if [ "${NEW_VALUE[*]}" == "${CURRENT_VALUE[*]}" ]; then
                     break
                 fi
+                if [[ $TYPE == "Number" &&  ! "${NEW_VALUE[*]}" =~ ^[0-9]+$ ]]; then
+                    notify msg "This field should contain only numbers."
+                    continue
+                fi
                 if UPDATED_OPTIONS=$(jq -e --arg SELECTED_OPTION "$SELECTED_OPTION" --arg TYPE "$TYPE" '
                         map(
                             .key as $KEY |