From da87ba417cd8aa88ccfd8a8f6c10fce182ade71d Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?E=CC=81ric=20Gaspar?=
 <46165813+ericgaspar@users.noreply.github.com>
Date: Mon, 2 Sep 2024 09:05:10 +0200
Subject: [PATCH] cleaning

---
 manifest.toml   |  2 +-
 scripts/backup  | 10 ----------
 scripts/install |  4 ----
 scripts/restore |  4 ----
 scripts/upgrade |  5 -----
 5 files changed, 1 insertion(+), 24 deletions(-)

diff --git a/manifest.toml b/manifest.toml
index ba137cf..bd6e92b 100644
--- a/manifest.toml
+++ b/manifest.toml
@@ -18,7 +18,7 @@ admindoc = "https://wiki.servarr.com/prowlarr"
 code = "https://github.com/Prowlarr/Prowlarr"
 
 [integration]
-yunohost = ">= 11.2.18"
+yunohost = ">= 11.2.29"
 helpers_version = "2.1"
 architectures = "all"
 multi_instance = true
diff --git a/scripts/backup b/scripts/backup
index 543c3fc..cdfd314 100755
--- a/scripts/backup
+++ b/scripts/backup
@@ -29,18 +29,8 @@ ynh_backup "$data_dir"
 
 ynh_backup "/etc/nginx/conf.d/$domain.d/$app.conf"
 
-#=================================================
-# SPECIFIC BACKUP
-#=================================================
-# BACKUP LOGROTATE
-#=================================================
-
 ynh_backup "/etc/logrotate.d/$app"
 
-#=================================================
-# BACKUP SYSTEMD
-#=================================================
-
 ynh_backup "/etc/systemd/system/$app.service"
 
 #=================================================
diff --git a/scripts/install b/scripts/install
index acf1502..d455a98 100755
--- a/scripts/install
+++ b/scripts/install
@@ -22,16 +22,12 @@ ynh_script_progression "Setting up source files..."
 # Download, check integrity, uncompress and patch the source from app.src
 ynh_setup_source --dest_dir="$install_dir"
 
-#REMOVEME? Assuming the install dir is setup using ynh_setup_source, the proper chmod/chowns are now already applied and it shouldn't be necessary to tweak perms | chmod -R o-rwx "$install_dir"
-#REMOVEME? Assuming the install dir is setup using ynh_setup_source, the proper chmod/chowns are now already applied and it shouldn't be necessary to tweak perms | chown -R "$app:" "$install_dir"
 #=================================================
 # APP INITIAL CONFIGURATION
 #=================================================
 ynh_script_progression "Adding $app's configuration files..."
 
 ynh_config_add --template="config.xml" --destination="$data_dir/config.xml"
-#REMOVEME? Assuming the file is setup using ynh_config_add, the proper chmod/chowns are now already applied and it shouldn't be necessary to tweak perms | chmod 400 "$data_dir/config.xml"
-#REMOVEME? Assuming the file is setup using ynh_config_add, the proper chmod/chowns are now already applied and it shouldn't be necessary to tweak perms | chown "$app:$app" "$data_dir/config.xml"
 
 #=================================================
 # YUNOHOST MULTIMEDIA INTEGRATION
diff --git a/scripts/restore b/scripts/restore
index b965fa7..be9cbee 100755
--- a/scripts/restore
+++ b/scripts/restore
@@ -15,8 +15,6 @@ ynh_script_progression "Restoring the app main directory..."
 
 ynh_restore "$install_dir"
 
-#REMOVEME? Assuming the install dir is setup using ynh_setup_source, the proper chmod/chowns are now already applied and it shouldn't be necessary to tweak perms | chmod -R o-rwx "$install_dir"
-#REMOVEME? Assuming the install dir is setup using ynh_setup_source, the proper chmod/chowns are now already applied and it shouldn't be necessary to tweak perms | chown -R "$app:" "$install_dir"
 #=================================================
 # RESTORE THE DATA DIRECTORY
 #=================================================
@@ -28,8 +26,6 @@ chown -R "$app:www-data" "$data_dir"
 
 if [ ! -f "$data_dir/config.xml" ]; then
     ynh_config_add --template="config.xml" --destination="$data_dir/config.xml"
-    #REMOVEME? Assuming the file is setup using ynh_config_add, the proper chmod/chowns are now already applied and it shouldn't be necessary to tweak perms | chmod 400 "$data_dir/config.xml"
-    #REMOVEME? Assuming the file is setup using ynh_config_add, the proper chmod/chowns are now already applied and it shouldn't be necessary to tweak perms | chown "$app:$app" "$data_dir/config.xml"
 fi
 
 #=================================================
diff --git a/scripts/upgrade b/scripts/upgrade
index 694c843..9389782 100755
--- a/scripts/upgrade
+++ b/scripts/upgrade
@@ -22,8 +22,6 @@ ynh_script_progression "Upgrading source files..."
 # Download, check integrity, uncompress and patch the source from app.src
 ynh_setup_source --dest_dir="$install_dir" --full_replace
 
-#REMOVEME? Assuming the install dir is setup using ynh_setup_source, the proper chmod/chowns are now already applied and it shouldn't be necessary to tweak perms | chmod -R o-rwx "$install_dir"
-#REMOVEME? Assuming the install dir is setup using ynh_setup_source, the proper chmod/chowns are now already applied and it shouldn't be necessary to tweak perms | chown -R "$app:" "$install_dir"
 #=================================================
 # UPDATE A CONFIG FILE
 #=================================================
@@ -34,9 +32,6 @@ sed -i -e '$a\' "$data_dir/config.xml"
 
 ynh_config_add --template="config.xml" --destination="$data_dir/config.xml"
 
-#REMOVEME? Assuming the file is setup using ynh_config_add, the proper chmod/chowns are now already applied and it shouldn't be necessary to tweak perms | chmod 400 "$data_dir/config.xml"
-#REMOVEME? Assuming the file is setup using ynh_config_add, the proper chmod/chowns are now already applied and it shouldn't be necessary to tweak perms | chown "$app:$app" "$data_dir/config.xml"
-
 #=================================================
 # REAPPLY SYSTEM CONFIGURATIONS
 #=================================================