From 9bd6d39a79ced84d0cb6b3f939aa86ef568754b6 Mon Sep 17 00:00:00 2001
From: tituspijean <tituspijean@outlook.com>
Date: Sat, 18 Apr 2020 10:53:22 +0200
Subject: [PATCH] [enh] add dynamic variables to systemd helper

---
 data/helpers.d/systemd | 23 +++++++++++++++++++----
 1 file changed, 19 insertions(+), 4 deletions(-)

diff --git a/data/helpers.d/systemd b/data/helpers.d/systemd
index 47e905f0f..5b1e3eaa8 100644
--- a/data/helpers.d/systemd
+++ b/data/helpers.d/systemd
@@ -2,9 +2,10 @@
 
 # Create a dedicated systemd config
 #
-# usage: ynh_add_systemd_config [--service=service] [--template=template]
-# | arg: -s, --service - Service name (optionnal, $app by default)
-# | arg: -t, --template - Name of template file (optionnal, this is 'systemd' by default, meaning ./conf/systemd.service will be used as template)
+# usage: ynh_add_systemd_config [--service=service] [--template=template] "list of others variables to replace"
+# | arg: -s, --service - Service name (optional, $app by default)
+# | arg: -t, --template - Name of template file (optional, this is 'systemd' by default, meaning ./conf/systemd.service will be used as template)
+# | arg: -v, --others_var=  - List of others variables to replace separeted by a space. For example : 'var_1 var_2 ...'
 #
 # This will use the template ../conf/<templatename>.service
 # to generate a systemd config, by replacing the following keywords
@@ -14,17 +15,22 @@
 #   __APP__       by  $app
 #   __FINALPATH__ by  $final_path
 #
+# And dynamic variables (from the last example) :
+#   __PATH_2__    by $path_2
+#   __PORT_2__    by $port_2
+#
 # Requires YunoHost version 2.7.2 or higher.
 ynh_add_systemd_config () {
 	# Declare an array to define the options of this helper.
 	local legacy_args=st
-	declare -Ar args_array=( [s]=service= [t]=template= )
+	declare -Ar args_array=( [s]=service= [t]=template= [v]=others_var=)
 	local service
 	local template
 	# Manage arguments with getopts
 	ynh_handle_getopts_args "$@"
 	local service="${service:-$app}"
 	local template="${template:-systemd.service}"
+	local others_var="${others_var:-}"
 
 	finalsystemdconf="/etc/systemd/system/$service.service"
 	ynh_backup_if_checksum_is_different --file="$finalsystemdconf"
@@ -38,6 +44,15 @@ ynh_add_systemd_config () {
 	if test -n "${app:-}"; then
 		ynh_replace_string --match_string="__APP__" --replace_string="$app" --target_file="$finalsystemdconf"
 	fi
+	
+	# Replace all other variable given as arguments
+	for var_to_replace in $others_var
+	do
+		# ${var_to_replace^^} make the content of the variable on upper-cases
+		# ${!var_to_replace} get the content of the variable named $var_to_replace 
+		ynh_replace_string --match_string="__${var_to_replace^^}__" --replace_string="${!var_to_replace}" --target_file="$finalnginxconf"
+	done
+	
 	ynh_store_file_checksum --file="$finalsystemdconf"
 
 	chown root: "$finalsystemdconf"