refactor: drop unused value_exists from report functions
The `value_exists` variable in the report info-flag loop is no longer read after the `not deployed` failure was removed, and was already unused in domains, haproxy-vhosts, traefik-vhosts, caddy-vhosts, and openresty-vhosts. Drop the declaration and the trailing assignment so the loop reads cleanly across all plugins.
This commit is contained in:
@@ -74,14 +74,13 @@ cmd-builder-dockerfile-report-single() {
|
|||||||
done
|
done
|
||||||
else
|
else
|
||||||
local match=false
|
local match=false
|
||||||
local value_exists=false
|
|
||||||
for flag in "${flag_map[@]}"; do
|
for flag in "${flag_map[@]}"; do
|
||||||
valid_flags="${valid_flags} $(echo "$flag" | cut -d':' -f1)"
|
valid_flags="${valid_flags} $(echo "$flag" | cut -d':' -f1)"
|
||||||
if [[ "$flag" == "${INFO_FLAG}:"* ]]; then
|
if [[ "$flag" == "${INFO_FLAG}:"* ]]; then
|
||||||
value=${flag#*: }
|
value=${flag#*: }
|
||||||
size="${#value}"
|
size="${#value}"
|
||||||
if [[ "$size" -ne 0 ]]; then
|
if [[ "$size" -ne 0 ]]; then
|
||||||
echo "$value" && match=true && value_exists=true
|
echo "$value" && match=true
|
||||||
else
|
else
|
||||||
match=true
|
match=true
|
||||||
fi
|
fi
|
||||||
|
|||||||
@@ -74,14 +74,13 @@ cmd-builder-herokuish-report-single() {
|
|||||||
done
|
done
|
||||||
else
|
else
|
||||||
local match=false
|
local match=false
|
||||||
local value_exists=false
|
|
||||||
for flag in "${flag_map[@]}"; do
|
for flag in "${flag_map[@]}"; do
|
||||||
valid_flags="${valid_flags} $(echo "$flag" | cut -d':' -f1)"
|
valid_flags="${valid_flags} $(echo "$flag" | cut -d':' -f1)"
|
||||||
if [[ "$flag" == "${INFO_FLAG}:"* ]]; then
|
if [[ "$flag" == "${INFO_FLAG}:"* ]]; then
|
||||||
value=${flag#*: }
|
value=${flag#*: }
|
||||||
size="${#value}"
|
size="${#value}"
|
||||||
if [[ "$size" -ne 0 ]]; then
|
if [[ "$size" -ne 0 ]]; then
|
||||||
echo "$value" && match=true && value_exists=true
|
echo "$value" && match=true
|
||||||
else
|
else
|
||||||
match=true
|
match=true
|
||||||
fi
|
fi
|
||||||
|
|||||||
@@ -74,14 +74,13 @@ cmd-builder-lambda-report-single() {
|
|||||||
done
|
done
|
||||||
else
|
else
|
||||||
local match=false
|
local match=false
|
||||||
local value_exists=false
|
|
||||||
for flag in "${flag_map[@]}"; do
|
for flag in "${flag_map[@]}"; do
|
||||||
valid_flags="${valid_flags} $(echo "$flag" | cut -d':' -f1)"
|
valid_flags="${valid_flags} $(echo "$flag" | cut -d':' -f1)"
|
||||||
if [[ "$flag" == "${INFO_FLAG}:"* ]]; then
|
if [[ "$flag" == "${INFO_FLAG}:"* ]]; then
|
||||||
value=${flag#*: }
|
value=${flag#*: }
|
||||||
size="${#value}"
|
size="${#value}"
|
||||||
if [[ "$size" -ne 0 ]]; then
|
if [[ "$size" -ne 0 ]]; then
|
||||||
echo "$value" && match=true && value_exists=true
|
echo "$value" && match=true
|
||||||
else
|
else
|
||||||
match=true
|
match=true
|
||||||
fi
|
fi
|
||||||
|
|||||||
@@ -74,14 +74,13 @@ cmd-builder-nixpacks-report-single() {
|
|||||||
done
|
done
|
||||||
else
|
else
|
||||||
local match=false
|
local match=false
|
||||||
local value_exists=false
|
|
||||||
for flag in "${flag_map[@]}"; do
|
for flag in "${flag_map[@]}"; do
|
||||||
valid_flags="${valid_flags} $(echo "$flag" | cut -d':' -f1)"
|
valid_flags="${valid_flags} $(echo "$flag" | cut -d':' -f1)"
|
||||||
if [[ "$flag" == "${INFO_FLAG}:"* ]]; then
|
if [[ "$flag" == "${INFO_FLAG}:"* ]]; then
|
||||||
value=${flag#*: }
|
value=${flag#*: }
|
||||||
size="${#value}"
|
size="${#value}"
|
||||||
if [[ "$size" -ne 0 ]]; then
|
if [[ "$size" -ne 0 ]]; then
|
||||||
echo "$value" && match=true && value_exists=true
|
echo "$value" && match=true
|
||||||
else
|
else
|
||||||
match=true
|
match=true
|
||||||
fi
|
fi
|
||||||
|
|||||||
@@ -74,14 +74,13 @@ cmd-builder-pack-report-single() {
|
|||||||
done
|
done
|
||||||
else
|
else
|
||||||
local match=false
|
local match=false
|
||||||
local value_exists=false
|
|
||||||
for flag in "${flag_map[@]}"; do
|
for flag in "${flag_map[@]}"; do
|
||||||
valid_flags="${valid_flags} $(echo "$flag" | cut -d':' -f1)"
|
valid_flags="${valid_flags} $(echo "$flag" | cut -d':' -f1)"
|
||||||
if [[ "$flag" == "${INFO_FLAG}:"* ]]; then
|
if [[ "$flag" == "${INFO_FLAG}:"* ]]; then
|
||||||
value=${flag#*: }
|
value=${flag#*: }
|
||||||
size="${#value}"
|
size="${#value}"
|
||||||
if [[ "$size" -ne 0 ]]; then
|
if [[ "$size" -ne 0 ]]; then
|
||||||
echo "$value" && match=true && value_exists=true
|
echo "$value" && match=true
|
||||||
else
|
else
|
||||||
match=true
|
match=true
|
||||||
fi
|
fi
|
||||||
|
|||||||
@@ -74,14 +74,13 @@ cmd-builder-railpack-report-single() {
|
|||||||
done
|
done
|
||||||
else
|
else
|
||||||
local match=false
|
local match=false
|
||||||
local value_exists=false
|
|
||||||
for flag in "${flag_map[@]}"; do
|
for flag in "${flag_map[@]}"; do
|
||||||
valid_flags="${valid_flags} $(echo "$flag" | cut -d':' -f1)"
|
valid_flags="${valid_flags} $(echo "$flag" | cut -d':' -f1)"
|
||||||
if [[ "$flag" == "${INFO_FLAG}:"* ]]; then
|
if [[ "$flag" == "${INFO_FLAG}:"* ]]; then
|
||||||
value=${flag#*: }
|
value=${flag#*: }
|
||||||
size="${#value}"
|
size="${#value}"
|
||||||
if [[ "$size" -ne 0 ]]; then
|
if [[ "$size" -ne 0 ]]; then
|
||||||
echo "$value" && match=true && value_exists=true
|
echo "$value" && match=true
|
||||||
else
|
else
|
||||||
match=true
|
match=true
|
||||||
fi
|
fi
|
||||||
|
|||||||
@@ -107,14 +107,13 @@ cmd-caddy-report-single() {
|
|||||||
done
|
done
|
||||||
else
|
else
|
||||||
local match=false
|
local match=false
|
||||||
local value_exists=false
|
|
||||||
for flag in "${flag_map[@]}"; do
|
for flag in "${flag_map[@]}"; do
|
||||||
valid_flags="${valid_flags} $(echo "$flag" | cut -d':' -f1)"
|
valid_flags="${valid_flags} $(echo "$flag" | cut -d':' -f1)"
|
||||||
if [[ "$flag" == "${INFO_FLAG}:"* ]]; then
|
if [[ "$flag" == "${INFO_FLAG}:"* ]]; then
|
||||||
value=${flag#*: }
|
value=${flag#*: }
|
||||||
size="${#value}"
|
size="${#value}"
|
||||||
if [[ "$size" -ne 0 ]]; then
|
if [[ "$size" -ne 0 ]]; then
|
||||||
echo "$value" && match=true && value_exists=true
|
echo "$value" && match=true
|
||||||
else
|
else
|
||||||
match=true
|
match=true
|
||||||
fi
|
fi
|
||||||
|
|||||||
@@ -75,14 +75,13 @@ cmd-certs-report-single() {
|
|||||||
done
|
done
|
||||||
else
|
else
|
||||||
local match=false
|
local match=false
|
||||||
local value_exists=false
|
|
||||||
for flag in "${flag_map[@]}"; do
|
for flag in "${flag_map[@]}"; do
|
||||||
valid_flags="${valid_flags} $(echo "$flag" | cut -d':' -f1)"
|
valid_flags="${valid_flags} $(echo "$flag" | cut -d':' -f1)"
|
||||||
if [[ "$flag" == "${INFO_FLAG}:"* ]]; then
|
if [[ "$flag" == "${INFO_FLAG}:"* ]]; then
|
||||||
value=${flag#*: }
|
value=${flag#*: }
|
||||||
size="${#value}"
|
size="${#value}"
|
||||||
if [[ "$size" -ne 0 ]]; then
|
if [[ "$size" -ne 0 ]]; then
|
||||||
echo "$value" && match=true && value_exists=true
|
echo "$value" && match=true
|
||||||
else
|
else
|
||||||
match=true
|
match=true
|
||||||
fi
|
fi
|
||||||
|
|||||||
@@ -77,14 +77,13 @@ cmd-checks-report-single() {
|
|||||||
done
|
done
|
||||||
else
|
else
|
||||||
local match=false
|
local match=false
|
||||||
local value_exists=false
|
|
||||||
for flag in "${flag_map[@]}"; do
|
for flag in "${flag_map[@]}"; do
|
||||||
valid_flags="${valid_flags} $(echo "$flag" | cut -d':' -f1)"
|
valid_flags="${valid_flags} $(echo "$flag" | cut -d':' -f1)"
|
||||||
if [[ "$flag" == "${INFO_FLAG}:"* ]]; then
|
if [[ "$flag" == "${INFO_FLAG}:"* ]]; then
|
||||||
value=${flag#*: }
|
value=${flag#*: }
|
||||||
size="${#value}"
|
size="${#value}"
|
||||||
if [[ "$size" -ne 0 ]]; then
|
if [[ "$size" -ne 0 ]]; then
|
||||||
echo "$value" && match=true && value_exists=true
|
echo "$value" && match=true
|
||||||
else
|
else
|
||||||
match=true
|
match=true
|
||||||
fi
|
fi
|
||||||
|
|||||||
@@ -77,14 +77,13 @@ cmd-domains-report-single() {
|
|||||||
done
|
done
|
||||||
else
|
else
|
||||||
local match=false
|
local match=false
|
||||||
local value_exists=false
|
|
||||||
for flag in "${flag_map[@]}"; do
|
for flag in "${flag_map[@]}"; do
|
||||||
valid_flags="${valid_flags} $(echo "$flag" | cut -d':' -f1)"
|
valid_flags="${valid_flags} $(echo "$flag" | cut -d':' -f1)"
|
||||||
if [[ "$flag" == "${INFO_FLAG}:"* ]]; then
|
if [[ "$flag" == "${INFO_FLAG}:"* ]]; then
|
||||||
value=${flag#*: }
|
value=${flag#*: }
|
||||||
size="${#value}"
|
size="${#value}"
|
||||||
if [[ "$size" -ne 0 ]]; then
|
if [[ "$size" -ne 0 ]]; then
|
||||||
echo "$value" && match=true && value_exists=true
|
echo "$value" && match=true
|
||||||
else
|
else
|
||||||
match=true
|
match=true
|
||||||
fi
|
fi
|
||||||
|
|||||||
@@ -407,14 +407,13 @@ cmd-git-report-single() {
|
|||||||
done
|
done
|
||||||
else
|
else
|
||||||
local match=false
|
local match=false
|
||||||
local value_exists=false
|
|
||||||
for flag in "${flag_map[@]}"; do
|
for flag in "${flag_map[@]}"; do
|
||||||
valid_flags="${valid_flags} $(echo "$flag" | cut -d':' -f1)"
|
valid_flags="${valid_flags} $(echo "$flag" | cut -d':' -f1)"
|
||||||
if [[ "$flag" == "${INFO_FLAG}:"* ]]; then
|
if [[ "$flag" == "${INFO_FLAG}:"* ]]; then
|
||||||
value=${flag#*: }
|
value=${flag#*: }
|
||||||
size="${#value}"
|
size="${#value}"
|
||||||
if [[ "$size" -ne 0 ]]; then
|
if [[ "$size" -ne 0 ]]; then
|
||||||
echo "$value" && match=true && value_exists=true
|
echo "$value" && match=true
|
||||||
else
|
else
|
||||||
match=true
|
match=true
|
||||||
fi
|
fi
|
||||||
|
|||||||
@@ -97,14 +97,13 @@ cmd-haproxy-report-single() {
|
|||||||
done
|
done
|
||||||
else
|
else
|
||||||
local match=false
|
local match=false
|
||||||
local value_exists=false
|
|
||||||
for flag in "${flag_map[@]}"; do
|
for flag in "${flag_map[@]}"; do
|
||||||
valid_flags="${valid_flags} $(echo "$flag" | cut -d':' -f1)"
|
valid_flags="${valid_flags} $(echo "$flag" | cut -d':' -f1)"
|
||||||
if [[ "$flag" == "${INFO_FLAG}:"* ]]; then
|
if [[ "$flag" == "${INFO_FLAG}:"* ]]; then
|
||||||
value=${flag#*: }
|
value=${flag#*: }
|
||||||
size="${#value}"
|
size="${#value}"
|
||||||
if [[ "$size" -ne 0 ]]; then
|
if [[ "$size" -ne 0 ]]; then
|
||||||
echo "$value" && match=true && value_exists=true
|
echo "$value" && match=true
|
||||||
else
|
else
|
||||||
match=true
|
match=true
|
||||||
fi
|
fi
|
||||||
|
|||||||
@@ -164,14 +164,13 @@ cmd-nginx-report-single() {
|
|||||||
done
|
done
|
||||||
else
|
else
|
||||||
local match=false
|
local match=false
|
||||||
local value_exists=false
|
|
||||||
for flag in "${flag_map[@]}"; do
|
for flag in "${flag_map[@]}"; do
|
||||||
valid_flags="${valid_flags} $(echo "$flag" | cut -d':' -f1)"
|
valid_flags="${valid_flags} $(echo "$flag" | cut -d':' -f1)"
|
||||||
if [[ "$flag" == "${INFO_FLAG}:"* ]]; then
|
if [[ "$flag" == "${INFO_FLAG}:"* ]]; then
|
||||||
value=${flag#*: }
|
value=${flag#*: }
|
||||||
size="${#value}"
|
size="${#value}"
|
||||||
if [[ "$size" -ne 0 ]]; then
|
if [[ "$size" -ne 0 ]]; then
|
||||||
echo "$value" && match=true && value_exists=true
|
echo "$value" && match=true
|
||||||
else
|
else
|
||||||
match=true
|
match=true
|
||||||
fi
|
fi
|
||||||
|
|||||||
@@ -134,14 +134,13 @@ cmd-openresty-report-single() {
|
|||||||
done
|
done
|
||||||
else
|
else
|
||||||
local match=false
|
local match=false
|
||||||
local value_exists=false
|
|
||||||
for flag in "${flag_map[@]}"; do
|
for flag in "${flag_map[@]}"; do
|
||||||
valid_flags="${valid_flags} $(echo "$flag" | cut -d':' -f1)"
|
valid_flags="${valid_flags} $(echo "$flag" | cut -d':' -f1)"
|
||||||
if [[ "$flag" == "${INFO_FLAG}:"* ]]; then
|
if [[ "$flag" == "${INFO_FLAG}:"* ]]; then
|
||||||
value=${flag#*: }
|
value=${flag#*: }
|
||||||
size="${#value}"
|
size="${#value}"
|
||||||
if [[ "$size" -ne 0 ]]; then
|
if [[ "$size" -ne 0 ]]; then
|
||||||
echo "$value" && match=true && value_exists=true
|
echo "$value" && match=true
|
||||||
else
|
else
|
||||||
match=true
|
match=true
|
||||||
fi
|
fi
|
||||||
|
|||||||
@@ -69,14 +69,13 @@ cmd-scheduler-docker-local-report-single() {
|
|||||||
done
|
done
|
||||||
else
|
else
|
||||||
local match=false
|
local match=false
|
||||||
local value_exists=false
|
|
||||||
for flag in "${flag_map[@]}"; do
|
for flag in "${flag_map[@]}"; do
|
||||||
valid_flags="${valid_flags} $(echo "$flag" | cut -d':' -f1)"
|
valid_flags="${valid_flags} $(echo "$flag" | cut -d':' -f1)"
|
||||||
if [[ "$flag" == "${INFO_FLAG}:"* ]]; then
|
if [[ "$flag" == "${INFO_FLAG}:"* ]]; then
|
||||||
value=${flag#*: }
|
value=${flag#*: }
|
||||||
size="${#value}"
|
size="${#value}"
|
||||||
if [[ "$size" -ne 0 ]]; then
|
if [[ "$size" -ne 0 ]]; then
|
||||||
echo "$value" && match=true && value_exists=true
|
echo "$value" && match=true
|
||||||
else
|
else
|
||||||
match=true
|
match=true
|
||||||
fi
|
fi
|
||||||
|
|||||||
@@ -124,14 +124,13 @@ cmd-traefik-report-single() {
|
|||||||
done
|
done
|
||||||
else
|
else
|
||||||
local match=false
|
local match=false
|
||||||
local value_exists=false
|
|
||||||
for flag in "${flag_map[@]}"; do
|
for flag in "${flag_map[@]}"; do
|
||||||
valid_flags="${valid_flags} $(echo "$flag" | cut -d':' -f1)"
|
valid_flags="${valid_flags} $(echo "$flag" | cut -d':' -f1)"
|
||||||
if [[ "$flag" == "${INFO_FLAG}:"* ]]; then
|
if [[ "$flag" == "${INFO_FLAG}:"* ]]; then
|
||||||
value=${flag#*: }
|
value=${flag#*: }
|
||||||
size="${#value}"
|
size="${#value}"
|
||||||
if [[ "$size" -ne 0 ]]; then
|
if [[ "$size" -ne 0 ]]; then
|
||||||
echo "$value" && match=true && value_exists=true
|
echo "$value" && match=true
|
||||||
else
|
else
|
||||||
match=true
|
match=true
|
||||||
fi
|
fi
|
||||||
|
|||||||
Reference in New Issue
Block a user