Coding style, double quotes

scripts/install

@@ -11,21 +11,8 @@ ynh_script_progression "Setting up source files..."
 ynh_setup_source --dest_dir="$install_dir/app"
 
 chmod -R 750 "$install_dir"
-chown -R $app:www-data "$install_dir"
+chown -R "$app:www-data" "$install_dir"
 
-#=================================================
-# SYSTEM CONFIGURATION
-#=================================================
-ynh_script_progression "Adding system configurations related to $app..."
-
-ynh_config_add_phpfpm
-
-ynh_config_add_nginx
-
-ynh_config_add_logrotate
-
-#=================================================
-# APP INITIAL CONFIGURATION
 #=================================================
 # ADD A CONFIGURATION
 #=================================================
@@ -43,9 +30,20 @@ chmod -R 600 "$install_dir/app/data"
 ynh_script_progression "Applying SSO patch..."
 
 ynh_config_add --template="sso.php" --destination="$install_dir/index.php"
-chown $app:www-data "$install_dir/index.php"
+chown "$app:www-data" "$install_dir/index.php"
 chmod 750 "$install_dir/index.php"
 
+#=================================================
+# SYSTEM CONFIGURATION
+#=================================================
+ynh_script_progression "Adding system configurations related to $app..."
+
+ynh_config_add_phpfpm
+
+ynh_config_add_nginx
+
+ynh_config_add_logrotate
+
 #=================================================
 # END OF SCRIPT
 #=================================================

scripts/restore

@@ -12,9 +12,9 @@ ynh_script_progression "Restoring the app main directory..."
 ynh_restore "$install_dir"
 
 chmod -R 750 "$install_dir"
-chown -R $app:www-data "$install_dir"
+chown -R "$app:www-data" "$install_dir"
 chmod 600 "$install_dir/app/data/_data_/_default_/configs/application.ini"
-chown -R $app:$app "$install_dir/app/data/_data_"
+chown -R "$app:$app" "$install_dir/app/data/_data_"
 
 #=================================================
 # RESTORE SYSTEM CONFIGURATIONS
@@ -32,7 +32,7 @@ ynh_restore "/etc/logrotate.d/$app"
 #=================================================
 ynh_script_progression "Reloading NGINX web server and $app's service..."
 
-ynh_systemctl --service=php$php_version-fpm --action=reload
+ynh_systemctl --service="php$php_version-fpm" --action=reload
 ynh_systemctl --service=nginx --action=reload
 
 #=================================================

scripts/upgrade

@@ -9,13 +9,12 @@ source /usr/share/yunohost/helpers
 ynh_script_progression "Ensuring downward compatibility..."
 
 # Do something when upgrading from 2.3.2~ynh1 or lower
-if ynh_app_upgrading_from_version_before_or_equal_to 2.28.4~ynh1
+if ynh_app_upgrading_from_version_before_or_equal_to 2.28.4~ynh1; then
-then
     # Move everything inside a $install_dir/app/ subfolder
     # This allows to have a $install_dir/index.php handling the SSO
-    mkdir -p $install_dir/app
+    mkdir -p "$install_dir/app"
     # Ugly way to not return an error when moving everything to a subfolter of the same folder https://stackoverflow.com/a/43262922
-    find $install_dir -maxdepth 1 -mindepth 1 -not -name app -exec mv -t $install_dir/app {} +
+    find "$install_dir" -maxdepth 1 -mindepth 1 -not -name app -exec mv -t "$install_dir/app" {} +
 fi
 
 #=================================================
@@ -26,7 +25,26 @@ ynh_script_progression "Upgrading source files..."
 ynh_setup_source --dest_dir="$install_dir/app" --full_replace=1 --keep="data/_data_/_default_/configs/application.ini"
 
 chmod -R 750 "$install_dir"
-chown -R $app:www-data "$install_dir"
+chown -R "$app:www-data" "$install_dir"
+
+#=================================================
+# UPDATE A CONFIG FILE
+#=================================================
+#ynh_script_progression "Updating configuration..."
+
+#ynh_config_add --template="application.ini" --destination="$install_dir/app/data/_data_/_default_/configs/application.ini"
+
+chmod -R 600 "$install_dir/app/data"
+chown -R "$app:$app" "$install_dir/app/data"
+
+#=================================================
+# SETUP SSO
+#=================================================
+ynh_script_progression "Applying SSO patch..."
+
+ynh_config_add --template="sso.php" --destination="$install_dir/index.php"
+chown "$app:www-data" "$install_dir/index.php"
+chmod 750 "$install_dir/index.php"
 
 #=================================================
 # REAPPLY SYSTEM CONFIGURATIONS
@@ -39,27 +57,6 @@ ynh_config_add_nginx
 
 ynh_config_add_logrotate
 
-#=================================================
-# RECONFIGURE THE APP (UPDATE CONF, APPLY MIGRATIONS...)
-#=================================================
-# UPDATE A CONFIG FILE
-#=================================================
-#ynh_script_progression "Updating configuration..."
-
-#ynh_config_add --template="application.ini" --destination="$install_dir/app/data/_data_/_default_/configs/application.ini"
-
-chmod -R 600 "$install_dir/app/data"
-chown -R $app:$app "$install_dir/app/data"
-
-#=================================================
-# SETUP SSO
-#=================================================
-ynh_script_progression "Applying SSO patch..."
-
-ynh_config_add --template="sso.php" --destination="$install_dir/index.php"
-chown $app:www-data "$install_dir/index.php"
-chmod 750 "$install_dir/index.php"
-
 #=================================================
 # END OF SCRIPT
 #=================================================