Fix a stupid issue in which a custom arg ending with _path would get its value replaced instead of the actual path é_è

This commit is contained in:
Alexandre Aubin 2023-12-31 18:08:18 +01:00
parent fee310b0b2
commit b8606ecc5e

View file

@ -81,6 +81,7 @@ _INSTALL_APP () {
# Make sure we have a trailing & because that assumption is used in some sed regex later # Make sure we have a trailing & because that assumption is used in some sed regex later
[[ ${install_args: -1} == '&' ]] || install_args+="&" [[ ${install_args: -1} == '&' ]] || install_args+="&"
[[ ${install_args:0:1} == '&' ]] || install_args="&$install_args"
# We have default values for domain, admin and is_public, but these # We have default values for domain, admin and is_public, but these
# may still be overwritten by the args ($@) # may still be overwritten by the args ($@)
@ -95,7 +96,7 @@ _INSTALL_APP () {
&& [[ "$(jq -r '.arguments.install[] | select(.name=="is_public") | .type' $package_path/manifest.json)" != "boolean" ]] \ && [[ "$(jq -r '.arguments.install[] | select(.name=="is_public") | .type' $package_path/manifest.json)" != "boolean" ]] \
&& continue && continue
install_args=$(echo $install_args | sed "s@$key=[^&]*\&@$key=$value\&@") install_args=$(echo $install_args | sed "s@\&$key=[^&]*\&@\&$key=$value\&@")
done done
# Note : we do this at this stage and not during the parsing of check_process # Note : we do this at this stage and not during the parsing of check_process