Coding style, reorder steps

scripts/backup

@@ -1,7 +1,5 @@
 #!/bin/bash
 
-#=================================================
-# GENERIC START
 #=================================================
 # IMPORT GENERIC HELPERS
 #=================================================
@@ -22,23 +20,13 @@ ynh_print_info --message="Declaring files to be backed up..."
 ynh_backup --src_path="$install_dir"
 
 #=================================================
-# BACKUP THE NGINX CONFIGURATION
+# BACKUP THE SYSTEM CONFIGURATION
 #=================================================
 
 ynh_backup --src_path="/etc/nginx/conf.d/$domain.d/$app.conf"
 
-#=================================================
-# BACKUP THE PHP-FPM CONFIGURATION
-#=================================================
-
 ynh_backup --src_path="/etc/php/$phpversion/fpm/pool.d/$app.conf"
 
-#=================================================
-# SPECIFIC BACKUP
-#=================================================
-# BACKUP VARIOUS FILES
-#=================================================
-
 ynh_backup --src_path="/etc/cron.d/$app"
 
 #=================================================

scripts/change_url

@@ -1,7 +1,5 @@
 #!/bin/bash
 
-#=================================================
-# GENERIC STARTING
 #=================================================
 # IMPORT GENERIC HELPERS
 #=================================================
@@ -9,8 +7,6 @@
 source _common.sh
 source /usr/share/yunohost/helpers
 
-#=================================================
-# STANDARD MODIFICATIONS
 #=================================================
 # MODIFY URL IN NGINX CONF
 #=================================================
@@ -18,15 +14,13 @@ ynh_script_progression --message="Updating NGINX web server configuration..." --
 
 ynh_change_url_nginx_config
 
-#=================================================
-# SPECIFIC MODIFICATIONS
 #=================================================
 # ADD A CONFIGURATION
 #=================================================
-ynh_script_progression --message="Adding a configuration file..." --weight=1
+ynh_script_progression --message="Configuring $app..." --weight=1
 
-pushd $install_dir
+pushd "$install_dir"
-	php$phpversion bin/console glpi:config:set url_base https://$domain$path
+	"php$phpversion" bin/console glpi:config:set url_base "https://$domain$path"
 popd
 
 #=================================================

scripts/install

@@ -1,7 +1,5 @@
 #!/bin/bash
 
-#=================================================
-# GENERIC START
 #=================================================
 # IMPORT GENERIC HELPERS
 #=================================================
@@ -20,6 +18,18 @@ ynh_setup_source --dest_dir="$install_dir"
 chmod -R o-rwx "$install_dir"
 chown -R $app:www-data "$install_dir"
 
+#=================================================
+# INSTALL THE DATABASE
+#=================================================
+ynh_script_progression --message="Installing $app..." --weight=1
+
+pushd "$install_dir"
+	"php$phpversion" bin/console glpi:database:install --db-host="localhost" \
+		--db-name="$db_name" --db-password="$db_pwd" --db-user="$db_user" --reconfigure --no-interaction
+	"php$phpversion" bin/console glpi:config:set url_base "https://$domain$path"
+	ynh_secure_remove --file="$install_dir/install/install.php"
+popd
+
 #=================================================
 # SYSTEM CONFIGURATION
 #=================================================
@@ -33,19 +43,6 @@ ynh_add_nginx_config
 
 ynh_add_config --template="cron" --destination="/etc/cron.d/$app"
 
-#=================================================
-# SPECIFIC SETUP
-#=================================================
-# INSTALL THE DATABASE
-#=================================================
-ynh_script_progression --message="Installing $app..." --weight=1
-
-pushd $install_dir
-	php$phpversion bin/console glpi:database:install --db-host="localhost" --db-name="$db_name" --db-password="$db_pwd" --db-user="$db_user" --reconfigure --no-interaction
-	php$phpversion bin/console glpi:config:set url_base https://$domain$path
-	ynh_secure_remove --file="$install_dir/install/install.php"
-popd
-
 #=================================================
 # END OF SCRIPT
 #=================================================

scripts/remove

@@ -1,7 +1,5 @@
 #!/bin/bash
 
-#=================================================
-# GENERIC START
 #=================================================
 # IMPORT GENERIC HELPERS
 #=================================================

scripts/restore

@@ -1,7 +1,5 @@
 #!/bin/bash
 
-#=================================================
-# GENERIC START
 #=================================================
 # IMPORT GENERIC HELPERS
 #=================================================
@@ -18,14 +16,14 @@ ynh_script_progression --message="Restoring the app main directory..." --weight=
 ynh_restore_file --origin_path="$install_dir"
 
 chmod -R o-rwx "$install_dir"
-chown -R $app:www-data "$install_dir"
+chown -R "$app:www-data" "$install_dir"
 
 #=================================================
 # RESTORE THE MYSQL DATABASE
 #=================================================
 ynh_script_progression --message="Restoring the MySQL database..." --weight=1
 
-ynh_mysql_connect_as --user=$db_user --password=$db_pwd --database=$db_name < ./db.sql
+ynh_mysql_connect_as --user="$db_user" --password="$db_pwd" --database="$db_name" < ./db.sql
 
 #=================================================
 # RESTORE THE PHP-FPM CONFIGURATION
@@ -38,14 +36,12 @@ ynh_restore_file --origin_path="/etc/nginx/conf.d/$domain.d/$app.conf"
 
 ynh_restore_file --origin_path="/etc/cron.d/$app"
 
-#=================================================
-# GENERIC FINALIZATION
 #=================================================
 # RELOAD NGINX AND PHP-FPM
 #=================================================
 ynh_script_progression --message="Reloading NGINX web server and PHP-FPM..." --weight=1
 
-ynh_systemd_action --service_name=php$phpversion-fpm --action=reload
+ynh_systemd_action --service_name="php$phpversion-fpm" --action=reload
 ynh_systemd_action --service_name=nginx --action=reload
 
 #=================================================

scripts/upgrade

@@ -1,7 +1,5 @@
 #!/bin/bash
 
-#=================================================
-# GENERIC START
 #=================================================
 # IMPORT GENERIC HELPERS
 #=================================================
@@ -9,12 +7,6 @@
 source _common.sh
 source /usr/share/yunohost/helpers
 
-#=================================================
-# CHECK VERSION
-#=================================================
-
-upgrade_type=$(ynh_check_app_version_changed)
-
 #=================================================
 # DOWNLOAD, CHECK AND UNPACK SOURCE
 #=================================================
@@ -24,7 +16,17 @@ ynh_script_progression --message="Upgrading source files..." --weight=1
 ynh_setup_source --dest_dir="$install_dir" #--full_replace=1
 
 chmod -R o-rwx "$install_dir"
-chown -R $app:www-data "$install_dir"
+chown -R "$app:www-data" "$install_dir"
+
+#=================================================
+# UPDATE THE DATABASE
+#=================================================
+ynh_script_progression --message="Updating the database..." --weight=1
+
+pushd "$install_dir"
+	"php$phpversion" bin/console system:check_requirements
+	"php$phpversion" bin/console db:update --no-interaction --no-telemetry --force
+popd
 
 #=================================================
 # REAPPLY SYSTEM CONFIGURATIONS
@@ -39,18 +41,6 @@ ynh_add_nginx_config
 
 ynh_add_config --template="cron" --destination="/etc/cron.d/$app"
 
-#=================================================
-# SPECIFIC UPGRADE
-#=================================================
-# UPDATE THE DATABASE
-#=================================================
-ynh_script_progression --message="Updating the database..." --weight=1
-
-pushd $install_dir
-	php$phpversion bin/console system:check_requirements
-	php$phpversion bin/console db:update --no-interaction --no-telemetry --force
-popd
-
 #=================================================
 # END OF SCRIPT
 #=================================================