4
0
Эх сурвалжийг харах

refactor: improve conditional statements in install_config.sh

The code changes refactor the conditional statements in the `install_config.sh` file. Instead of checking if `${shortname}` is equal to "dst" or "arma3", it now checks if `${clustercfgdir}` or `${networkcfgdir}` are not empty, respectively. This improves readability and maintainability of the code.

a
Daniel Gibbs 2 жил өмнө
parent
commit
e499c1c450

+ 33 - 12
lgsm/modules/install_config.sh

@@ -10,26 +10,47 @@ moduleselfname="$(basename "$(readlink -f "${BASH_SOURCE[0]}")")"
 # Checks if server cfg dir exists, creates it if it doesn't.
 # Checks if server cfg dir exists, creates it if it doesn't.
 fn_check_cfgdir() {
 fn_check_cfgdir() {
 	changes=""
 	changes=""
-	if [ "${shortname}" == "dst" ]; then
+	if [ -n "${clustercfgdir}" ]; then
 		echo -en "creating config directory [ ${italic}${clustercfgdir}${default} ]"
 		echo -en "creating config directory [ ${italic}${clustercfgdir}${default} ]"
 		changes+=$(mkdir -pv "${clustercfgdir}")
 		changes+=$(mkdir -pv "${clustercfgdir}")
-	elif [ "${shortname}" == "arma3" ]; then
+		if [ "$?" -ne 0 ]; then # shellcheck disable=SC2181
+			fn_print_fail_eol_nl
+			fn_script_log_fatal "creating ${servercfgdir} config directory"
+			core_exit.sh
+		elif [ "${changes}" != "" ]; then
+			fn_print_ok_eol_nl
+			fn_script_log_pass "creating ${servercfgdir} config directory"
+		else
+			fn_print_skip_eol_nl
+		fi
+	elif [ -n "${networkcfgdir}" ]; then
 		echo -en "creating config directory [ ${italic}${networkcfgdir}${default} ]"
 		echo -en "creating config directory [ ${italic}${networkcfgdir}${default} ]"
 		changes+=$(mkdir -pv "${networkcfgdir}")
 		changes+=$(mkdir -pv "${networkcfgdir}")
+		if [ "$?" -ne 0 ]; then # shellcheck disable=SC2181
+			fn_print_fail_eol_nl
+			fn_script_log_fatal "creating ${servercfgdir} config directory"
+			core_exit.sh
+		elif [ "${changes}" != "" ]; then
+			fn_print_ok_eol_nl
+			fn_script_log_pass "creating ${servercfgdir} config directory"
+		else
+			fn_print_skip_eol_nl
+		fi
 	else
 	else
 		echo -en "creating config directory [ ${italic}${servercfgdir}${default} ]"
 		echo -en "creating config directory [ ${italic}${servercfgdir}${default} ]"
 		changes+=$(mkdir -pv "${servercfgdir}")
 		changes+=$(mkdir -pv "${servercfgdir}")
+		if [ "$?" -ne 0 ]; then # shellcheck disable=SC2181
+			fn_print_fail_eol_nl
+			fn_script_log_fatal "creating ${servercfgdir} config directory"
+			core_exit.sh
+		elif [ "${changes}" != "" ]; then
+			fn_print_ok_eol_nl
+			fn_script_log_pass "creating ${servercfgdir} config directory"
+		else
+			fn_print_skip_eol_nl
+		fi
 	fi
 	fi
-	if [ "$?" -ne 0 ]; then # shellcheck disable=SC2181
-		fn_print_fail_eol_nl
-		fn_script_log_fatal "creating ${servercfgdir} config directory"
-		core_exit.sh
-	elif [ "${changes}" != "" ]; then
-		fn_print_ok_eol_nl
-		fn_script_log_pass "creating ${servercfgdir} config directory"
-	else
-		fn_print_skip_eol_nl
-	fi
+
 	unset changes
 	unset changes
 }
 }