From 7c6d37e2d31a49e1c180b063f245536e0d507a9d Mon Sep 17 00:00:00 2001
From: WatskeBart <bartistic@gmail.com>
Date: Thu, 28 Feb 2019 16:07:34 +0100
Subject: [PATCH 1/3] Update _default.cfg

-hostname="${servername}" should be -hostname='${servername}'
When applying the wrong quotes, spaces are not detected and the server name will be cut off after the first space and showing incorrectly in the server browser.
---
 lgsm/config-default/config-lgsm/inssserver/_default.cfg | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/lgsm/config-default/config-lgsm/inssserver/_default.cfg b/lgsm/config-default/config-lgsm/inssserver/_default.cfg
index 06886ed21..1ba1df679 100644
--- a/lgsm/config-default/config-lgsm/inssserver/_default.cfg
+++ b/lgsm/config-default/config-lgsm/inssserver/_default.cfg
@@ -23,7 +23,7 @@ maxplayers="28"
 fn_parms(){
 	# Allows serverpassword to work with parameters
 	if [ "${serverpassword}" != "NOT SET" ]; then
-		parms="${defaultmap}?Scenario=${defaultscenario}?MaxPlayers=${maxplayers}?Port=${port}?QueryPort=${queryport}?password="${serverpassword}" -hostname="${servername}" -log"
+		parms="${defaultmap}?Scenario=${defaultscenario}?MaxPlayers=${maxplayers}?Port=${port}?QueryPort=${queryport}?password="${serverpassword}" -hostname='${servername}' -log"
 	else
 		parms="${defaultmap}?Scenario=${defaultscenario}?MaxPlayers=${maxplayers}?Port=${port}?QueryPort=${queryport} -hostname='${servername}' -log"
 	fi

From a98b5fdd4b7ca8dbabbd98ad20718c5819b6e032 Mon Sep 17 00:00:00 2001
From: Jonathan Rosewood <jonathan-rosewood@yandex.ru>
Date: Sat, 2 Mar 2019 00:32:12 +0300
Subject: [PATCH 2/3] fix(fctrserver): resolve factorio update bug

---
 lgsm/functions/update_factorio.sh | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/lgsm/functions/update_factorio.sh b/lgsm/functions/update_factorio.sh
index 887bdd22b..794a0aac4 100644
--- a/lgsm/functions/update_factorio.sh
+++ b/lgsm/functions/update_factorio.sh
@@ -81,10 +81,10 @@ fn_update_factorio_arch(){
 
 fn_update_factorio_availablebuild(){
 	# Gets latest build info.
-	if [ "${branch}" != "stable" ]; then
-		availablebuild=$(${curlpath} -s https://factorio.com/get-download/stable/headless/linux64 | grep -o '[0-9]\.[0-9]\{2\}\.[0-9]\{2\}' | head -1)
+	if [ "${branch}" == "stable" ]; then
+		availablebuild=$(${curlpath} -s https://factorio.com/get-download/stable/headless/linux64 | grep -o '[0-9]\.[0-9]\{1,\}\.[0-9]\{1,\}' | head -1)
 	else
-		availablebuild=$(${curlpath} -s https://factorio.com/get-download/latest/headless/linux64 | grep -o '[0-9]\.[0-9]\{2\}\.[0-9]\{2\}' | head -1)
+		availablebuild=$(${curlpath} -s https://factorio.com/get-download/latest/headless/linux64 | grep -o '[0-9]\.[0-9]\{1,\}\.[0-9]\{1,\}' | head -1)
 	fi
 	# Checks if availablebuild variable has been set
 	if [ -z "${availablebuild}" ]; then

From cdf0d4ec1ee69d42b66fc313e95b7f740c6d416a Mon Sep 17 00:00:00 2001
From: Daniel Gibbs <me@danielgibbs.co.uk>
Date: Fri, 1 Mar 2019 22:05:00 +0000
Subject: [PATCH 3/3] fix(fctrserver): refactor update_factorio.sh

general tidy up and optimise of code
---
 lgsm/functions/update_factorio.sh | 78 ++++++++++++++-----------------
 1 file changed, 36 insertions(+), 42 deletions(-)

diff --git a/lgsm/functions/update_factorio.sh b/lgsm/functions/update_factorio.sh
index 794a0aac4..a1aa9653d 100644
--- a/lgsm/functions/update_factorio.sh
+++ b/lgsm/functions/update_factorio.sh
@@ -10,11 +10,6 @@ local commandaction="Update"
 local function_selfname="$(basename "$(readlink -f "${BASH_SOURCE[0]}")")"
 
 fn_update_factorio_dl(){
-	if [ "${branch}" == "stable" ]; then
-		downloadbranch="stable"
-	elif [ "${branch}" == "experimental" ]; then
-		downloadbranch="latest"
-	fi
 	fn_fetch_file "https://factorio.com/get-download/${downloadbranch}/headless/${factorioarch}" "${tmpdir}" "factorio_headless_${factorioarch}-${availablebuild}.tar.xz"
 	fn_dl_extract "${tmpdir}" "factorio_headless_${factorioarch}-${availablebuild}.tar.xz" "${tmpdir}"
 	echo -e "copying to ${serverfiles}...\c"
@@ -29,13 +24,23 @@ fn_update_factorio_dl(){
 }
 
 fn_update_factorio_currentbuild(){
-	# Gets current build info
-	# Checks if current build info is available. If it fails, then a server restart will be forced to generate logs.
-	if [ ! -f "${serverfiles}/factorio-current.log" ]; then
+	# Get current build info.
+	# If log file is missing, the server will restart to generate logs.
+
+	if [ -f "${serverfiles}/factorio-current.log" ]; then
+		currentbuild=$(grep "Loading mod base" "${serverfiles}/factorio-current.log" 2> /dev/null | awk '{print $5}' | tail -1)
+	fi
+
+	if [ -z "${currentbuild}" ]; then
 		fn_print_error "Checking for update: factorio.com"
 		sleep 0.5
-		fn_print_error_nl "Checking for update: factorio.com: No logs with server version found"
-		fn_script_log_error "Checking for update: factorio.com: No logs with server version found"
+		if [ ! -f "${serverfiles}/factorio-current.log" ]; then
+			fn_print_error_nl "Checking for update: factorio.com: No logs with server version found"
+			fn_script_log_error "Checking for update: factorio.com: No logs with server version found"
+		elif [ -z "${currentbuild}" ]; then	
+			fn_print_error_nl "Checking for update: factorio.com: Current build version not found"
+			fn_script_log_error "Checking for update: factorio.com: Current build version not found"
+		fi
 		sleep 0.5
 		fn_print_info_nl "Checking for update: factorio.com: Forcing server restart"
 		fn_script_log_info "Checking for update: factorio.com: Forcing server restart"
@@ -45,28 +50,17 @@ fn_update_factorio_currentbuild(){
 		exitbypass=1
 		command_start.sh
 		sleep 0.5
-		# Check again and exit on failure.
+
+		# Check again and exit if failure.
+		if [ -f "${serverfiles}/factorio-current.log" ]; then
+			currentbuild=$(grep "Loading mod base" "${serverfiles}/factorio-current.log" 2> /dev/null | awk '{print $5}' | tail -1)
+		fi	
+
 		if [ ! -f "${serverfiles}/factorio-current.log" ]; then
 			fn_print_fail_nl "Checking for update: factorio.com: Still No logs with server version found"
 			fn_script_log_fatal "Checking for update: factorio.com: Still No logs with server version found"
 			core_exit.sh
-		fi
-	fi
-
-	# Get current build from logs
-	currentbuild=$(grep "Loading mod base" "${serverfiles}/factorio-current.log" 2> /dev/null | awk '{print $5}' | tail -1)
-	if [ -z "${currentbuild}" ]; then
-		fn_print_error_nl "Checking for update: factorio.com: Current build version not found"
-		fn_script_log_error "Checking for update: factorio.com: Current build version not found"
-		sleep 0.5
-		fn_print_info_nl "Checking for update: factorio.com: Forcing server restart"
-		fn_script_log_info "Checking for update: factorio.com: Forcing server restart"
-		exitbypass=1
-		command_stop.sh
-		exitbypass=1
-		command_start.sh
-		currentbuild=$(grep "Loading mod base" "${serverfiles}/factorio-current.log" 2> /dev/null | awk '{print $5}' | tail -1)
-		if [ -z "${currentbuild}" ]; then
+		elif [ -z "${currentbuild}" ]; then
 			fn_print_fail_nl "Checking for update: factorio.com: Current build version still not found"
 			fn_script_log_fatal "Checking for update: factorio.com: Current build version still not found"
 			core_exit.sh
@@ -74,20 +68,12 @@ fn_update_factorio_currentbuild(){
 	fi
 }
 
-fn_update_factorio_arch(){
-	# Factorio is linux64 only for now
-	factorioarch="linux64"
-}
-
 fn_update_factorio_availablebuild(){
 	# Gets latest build info.
-	if [ "${branch}" == "stable" ]; then
-		availablebuild=$(${curlpath} -s https://factorio.com/get-download/stable/headless/linux64 | grep -o '[0-9]\.[0-9]\{1,\}\.[0-9]\{1,\}' | head -1)
-	else
-		availablebuild=$(${curlpath} -s https://factorio.com/get-download/latest/headless/linux64 | grep -o '[0-9]\.[0-9]\{1,\}\.[0-9]\{1,\}' | head -1)
-	fi
-	# Checks if availablebuild variable has been set
-	if [ -z "${availablebuild}" ]; then
+		availablebuild=$(${curlpath} -s https://factorio.com/get-download/${downloadbranch}/headless/${factorioarch} | grep -o '[0-9]\.[0-9]\{1,\}\.[0-9]\{1,\}' | head -1)
+	
+	# Checks if availablebuild variable has been set.
+	if [ -v "${availablebuild}" ]; then
 		fn_print_fail "Checking for update: factorio.com"
 		sleep 0.5
 		fn_print_fail "Checking for update: factorio.com: Not returning version info"
@@ -103,7 +89,7 @@ fn_update_factorio_availablebuild(){
 }
 
 fn_update_factorio_compare(){
-	# Removes dots so if can compare version numbers
+	# Removes dots so if statement can compare version numbers.
 	currentbuilddigit=$(echo "${currentbuild}" | tr -cd '[:digit:]')
 	availablebuilddigit=$(echo "${availablebuild}" | tr -cd '[:digit:]')
 
@@ -157,7 +143,15 @@ fn_update_factorio_compare(){
 	fi
 }
 
-fn_update_factorio_arch
+# Factorio is linux64 only for now.
+factorioarch="linux64"
+
+if [ "${branch}" == "stable" ]; then
+	downloadbranch="stable"
+elif [ "${branch}" == "experimental" ]; then
+	downloadbranch="latest"
+fi
+
 if [ "${installer}" == "1" ]; then
 	fn_update_factorio_availablebuild
 	fn_update_factorio_dl