From 1a40ff1eed837f2e777e79d62efd3fb01f6eb7df Mon Sep 17 00:00:00 2001
From: Pierre de La Morinerie <kemenaran@gmail.com>
Date: Tue, 10 Oct 2017 17:42:41 +0530
Subject: [PATCH] remove: use more variables instead of hardcoded values

---
 scripts/remove | 17 ++++++++++-------
 1 file changed, 10 insertions(+), 7 deletions(-)

diff --git a/scripts/remove b/scripts/remove
index 637d490..a1055e5 100755
--- a/scripts/remove
+++ b/scripts/remove
@@ -13,9 +13,12 @@ source /usr/share/yunohost/helpers
 # LOAD SETTINGS
 #=================================================
 
+app=$YNH_APP_INSTANCE_NAME
 domain=$(ynh_app_setting_get mattermost domain)
 db_name="mattermost"
 db_user="mmuser"
+final_path="/var/www/$app"
+data_path="/home/yunohost.app/$app"
 
 #=================================================
 # STANDARD REMOVE
@@ -23,8 +26,8 @@ db_user="mmuser"
 # STOP AND REMOVE SERVICE
 #=================================================
 
-sudo supervisorctl stop mattermost
-sudo rm -f /etc/supervisor/conf.d/mattermost.conf
+sudo supervisorctl stop "$app"
+sudo rm -f "/etc/supervisor/conf.d/${app}.conf"
 
 #=================================================
 # REMOVE THE MYSQL DATABASE
@@ -37,23 +40,23 @@ ynh_mysql_execute_as_root "DROP DATABASE IF EXISTS $db_name ; DROP USER $db_user
 # REMOVE APP MAIN DIR
 #=================================================
 
-sudo rm -rf /var/www/mattermost
-sudo rm -rf /home/yunohost.app/mattermost
+sudo rm -rf "$final_path"
+sudo rm -rf "$data_path"
 
 #=================================================
 # REMOVE NGINX CONFIGURATION
 #=================================================
 
-sudo rm -f /etc/nginx/conf.d/$domain.d/mattermost.conf
+sudo rm -f "/etc/nginx/conf.d/${domain}.d/${app}.conf"
 
 #=================================================
 # REMOVE LOG FILE
 #=================================================
 
-sudo rm -f "/var/log/mattermost.log"
+sudo rm -f "/var/log/${app}.log"
 
 #=================================================
 # REMOVE DEDICATED USER
 #=================================================
 
-sudo userdel mattermost
+sudo userdel "$app"