Merge branch '216-replace-bash-empty-sting-test-with-existence-test-of-variable' into 'master'

Resolve "replace bash empty-sting test with existence test of variable"

Closes #216

See merge request Pmodules/src!193
This commit is contained in:
2023-07-03 17:20:10 +00:00
+2 -2
View File
@@ -1026,7 +1026,7 @@ get_available_modules() {
# - in same overlay as first found
# - new version and not hidden by overlay
local name="${mod%/*}"
if [[ -z "${modulenames[${name}]}" ]]; then
if [[ ! -v modulenames[${name}] ]]; then
if [[ "${OverlayInfo[${ol}:type]}" == "${ol_hiding}" ]]; then
modulenames[${name}]="${ol}"
else
@@ -1036,7 +1036,7 @@ get_available_modules() {
elif [[ "${modulenames[${name}]}" == "${ol}" ]]; then
add='yes'
elif [[ "${modulenames[${name}]}" == '0' ]] \
&& [[ -z ${dict[${mod}]} ]]; then
&& [[ ! -v dict[${mod}] ]]; then
add='yes'
fi
else