From 8a7571e4717afd88702bb03e1bccdce8e813c91b Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?E=CC=81ric=20Gaspar?=
 <46165813+ericgaspar@users.noreply.github.com>
Date: Fri, 6 Oct 2023 19:13:26 +0200
Subject: [PATCH] cleaning

---
 conf/config.local.php          |  6 ++--
 conf/config.local.user.php     | 13 -------
 conf/config.local.yunohost.php | 64 ----------------------------------
 conf/extra_php-fpm.conf        |  4 +--
 conf/nginx.conf                |  2 +-
 manifest.toml                  |  2 ++
 scripts/backup                 |  7 ++++
 scripts/change_url             |  7 ++--
 scripts/install                |  9 -----
 scripts/restore                | 29 ++++++++++-----
 10 files changed, 39 insertions(+), 104 deletions(-)
 delete mode 100644 conf/config.local.user.php
 delete mode 100644 conf/config.local.yunohost.php

diff --git a/conf/config.local.php b/conf/config.local.php
index b59da7d..0ceb6af 100644
--- a/conf/config.local.php
+++ b/conf/config.local.php
@@ -168,7 +168,7 @@ const DB_FILE = DATA_ROOT . '__DATA_DIR__/association.sqlite';
  * Défaut : découverte automatique à partir de SCRIPT_NAME
  */
 
-//const WWW_URI = '/asso/';
+const WWW_URI = '__PATH__/';
 
 /**
  * Adresse URL HTTP(S) publique de Paheko
@@ -326,7 +326,7 @@ const DB_FILE = DATA_ROOT . '__DATA_DIR__/association.sqlite';
  * Défaut : 'TRUNCATE'
  * @var string
  */
-//const SQLITE_JOURNAL_MODE = 'TRUNCATE';
+const SQLITE_JOURNAL_MODE = 'WAL';
 
 /**
  * Activation du log HTTP (option de développement)
@@ -364,7 +364,7 @@ const DB_FILE = DATA_ROOT . '__DATA_DIR__/association.sqlite';
  * @var bool
  */
 
-//const ENABLE_UPGRADES = true;
+const ENABLE_UPGRADES = false;
 
 /**
  * Utilisation de cron pour les tâches automatiques
diff --git a/conf/config.local.user.php b/conf/config.local.user.php
deleted file mode 100644
index dbcdf62..0000000
--- a/conf/config.local.user.php
+++ /dev/null
@@ -1,13 +0,0 @@
-<?php
-
-/**
- * Ce fichier représente un exemple des constantes de configuration
- * disponibles pour Paheko.
- *
- * Modifiez ce fichier à votre guise, il ne sera pas modifié lors des mises à jour.
- * Consultez le fichier config.dist.php pour disposer d'un exemple.
- *
- */
-
-// Nécessaire pour situer les constantes dans le bon namespace
-namespace Paheko;
diff --git a/conf/config.local.yunohost.php b/conf/config.local.yunohost.php
deleted file mode 100644
index ecc707a..0000000
--- a/conf/config.local.yunohost.php
+++ /dev/null
@@ -1,64 +0,0 @@
-<?php
-
-/**
- * NE PAS MODIFIER CE FICHIER!
- *
- * Ce fichier sera mis à jour à chaque nouvelle version de Yunohost
- *
- * Pour ajouter vos configurations personnalisées, rendez-vous dans le fichier data/config.local.user.php
- *
- */
-
-// Nécessaire pour situer les constantes dans le bon namespace
-namespace Paheko;
-
-/**
- * Clé secrète, doit être unique à chaque instance de Garradin
- *
- * Ceci est utilisé afin de sécuriser l'envoi de formulaires
- * (protection anti-CSRF).
- *
- * Cette valeur peut être modifiée sans autre impact que la déconnexion des utilisateurs
- * actuellement connectés.
- *
- * Si cette constante n'est définie, Garradin ajoutera automatiquement
- * une valeur aléatoire dans le fichier config.local.php.
- */
-
-const SECRET_KEY = '__SECRET_KEY__';
-
-/**
- * Adresse URI de la racine du site Paheko
- * (doit se terminer par un slash)
- *
- * Défaut : découverte automatique à partir de SCRIPT_NAME
- */
-const WWW_URI = '__PATH__/';
-
-/**
- * Activer la possibilité de faire une mise à jour semi-automatisée
- * depuis fossil.kd2.org.
- *
- * Si mis à TRUE, alors un bouton sera accessible depuis le menu "Configuration"
- * pour faire une mise à jour en deux clics.
- *
- * Il est conseillé de désactiver cette fonctionnalité si vous ne voulez pas
- * permettre à un utilisateur de casser l'installation !
- *
- * Défaut : true, ajout pour l'environement Yunohost défaut : false
- *
- * @var bool
- */
-
-const ENABLE_UPGRADES = false;
-
-/**
- * Since 1.2.4, I downgraded the default SQLite journal mode to TRUNCATE instead of WAL because 
- * it might have been a cause of corruption on some hosting providers using NFS.
- *
- * I don't think that Yunohost can use NFS, so you should set it back to WAL 
- * by adding the following line to config.local.php when installing:
-*/
-
-const SQLITE_JOURNAL_MODE = 'WAL';
-
diff --git a/conf/extra_php-fpm.conf b/conf/extra_php-fpm.conf
index c27cac6..700c37c 100644
--- a/conf/extra_php-fpm.conf
+++ b/conf/extra_php-fpm.conf
@@ -1,4 +1,4 @@
 ; Additional php.ini defines, specific to this pool of workers.
 
-php_admin_value[upload_max_filesize] = 2M
-php_admin_value[post_max_size] = 2M
+php_admin_value[upload_max_filesize] = 50M
+php_admin_value[post_max_size] = 50M
diff --git a/conf/nginx.conf b/conf/nginx.conf
index 3be8ce5..e51cbbd 100644
--- a/conf/nginx.conf
+++ b/conf/nginx.conf
@@ -7,7 +7,7 @@ location __PATH__/ {
   index index.php /_route.php;
 
   # Increase size limit
-  client_max_body_size 2M;
+  client_max_body_size 50M;
 
   #try_files $uri $uri/ __PATH__/__PATH__/_route.php?$query_string;
   try_files $uri $uri/ __PATH__/_route.php?$query_string;
diff --git a/manifest.toml b/manifest.toml
index 81063cb..fa7be60 100644
--- a/manifest.toml
+++ b/manifest.toml
@@ -47,6 +47,8 @@ ram.runtime = "50M"
 
     [resources.install_dir]
 
+    [resources.data_dir]
+
     [resources.permissions]
     main.url = "/"
 
diff --git a/scripts/backup b/scripts/backup
index 18f83fe..c565227 100644
--- a/scripts/backup
+++ b/scripts/backup
@@ -21,6 +21,13 @@ ynh_print_info --message="Declaring files to be backed up..."
 
 ynh_backup --src_path="$install_dir"
 
+#=================================================
+# BACKUP THE DATA DIR
+#=================================================
+
+# Only relevant if there is a "data_dir" resource for this app
+ynh_backup --src_path="$data_dir" --is_big
+
 #=================================================
 # BACKUP THE NGINX CONFIGURATION
 #=================================================
diff --git a/scripts/change_url b/scripts/change_url
index f0a6298..414cc1c 100644
--- a/scripts/change_url
+++ b/scripts/change_url
@@ -25,9 +25,10 @@ ynh_change_url_nginx_config
 #=================================================
 ynh_script_progression --message="Updating a configuration file..." --weight=1
 
-domain=$new_domain
-path=$new_path
-ynh_add_config --template="config.local.yunohost.php" --destination="$install_dir/config.local.yunohost.php"
+ynh_add_config --template="../conf/config.local.php" --destination="$install_dir/config.local.php"
+
+chmod 650 "$install_dir/config.local.php"
+chown $app:$app "$install_dir/config.local.php"
 
 #=================================================
 # END OF SCRIPT
diff --git a/scripts/install b/scripts/install
index 5c9fa13..4d37886 100644
--- a/scripts/install
+++ b/scripts/install
@@ -46,15 +46,6 @@ ynh_add_config --template="../conf/config.local.php" --destination="$install_dir
 chmod 650 "$install_dir/config.local.php"
 chown $app:$app "$install_dir/config.local.php"
 
-#ynh_add_config --template="../conf/config.local.yunohost.php" --destination="$install_dir/config.local.yunohost.php"
-#chmod 400 "$install_dir/config.local.yunohost.php"
-#chown $app:$app "$install_dir/config.local.yunohost.php"
-
-#ynh_add_config --template="../conf/config.local.user.php" --destination="$install_dir/data/config.local.user.php"
-#chmod 400 "$install_dir/data/config.local.user.php"
-#chown $app:$app "$install_dir/data/config.local.user.php"
-
-
 # Permissions on files and directories
 
 #find "$install_dir" -type d -exec chmod 550 {} +
diff --git a/scripts/restore b/scripts/restore
index 9a43dad..9879a3d 100644
--- a/scripts/restore
+++ b/scripts/restore
@@ -20,15 +20,26 @@ ynh_restore_file --origin_path="$install_dir"
 # Restore permissions on app files
 chmod -R o-rwx "$install_dir"
 chown -R $app:www-data "$install_dir"
-find "$install_dir" -type d -exec chmod 550 {} +
-find "$install_dir" -type f -exec chmod 440 {} +
-find "$install_dir/data" -type d -exec chmod 770 {} +
-find "$install_dir/data" -type f -exec chmod 660 {} +
-chmod 440 "$install_dir/config.local.php"
-chmod 440 "$install_dir/config.local.yunohost.php"
-if [[ -f "$install_dir/config.local.user.php" ]]; then
-	chmod 660 "$install_dir/config.local.user.php"
-fi
+
+
+#=================================================
+# RESTORE THE DATA DIRECTORY
+#=================================================
+ynh_script_progression --message="Restoring the data directory..." --weight=1
+
+ynh_restore_file --origin_path="$data_dir" --not_mandatory
+
+# (Same as for install dir)
+# chown -R $app:www-data "$data_dir"
+# find "$install_dir" -type d -exec chmod 550 {} +
+# find "$install_dir" -type f -exec chmod 440 {} +
+# find "$install_dir/data" -type d -exec chmod 770 {} +
+# find "$install_dir/data" -type f -exec chmod 660 {} +
+# chmod 440 "$install_dir/config.local.php"
+# chmod 440 "$install_dir/config.local.yunohost.php"
+# if [[ -f "$install_dir/config.local.user.php" ]]; then
+# 	chmod 660 "$install_dir/config.local.user.php"
+# fi
 
 #=================================================
 # RESTORE SYSTEM CONFIGURATIONS