refactor: add default value + template script is no longer DRY

README-FR.md

@@ -148,7 +148,7 @@ Une tâche cron peut être mise en place pour automatiser la vérification des m
   Pour en savoir plus sur les _special times_, cliquez [ici](https://docs.ansible.com/ansible/latest/collections/ansible/builtin/cron_module.html).
   - `ynh_autoupdate.apps` : est indispensable. Activez la mise à jour automatique des applications Yunohost en mettant la valeur à `True`.
   - `ynh_autoupdate.system` : est indispensable. Activez la mise à jour automatique du système Yunohost en mettant la valeur à `True`.
-  - `ynh_autoupdate.dest_script` est obligatoire. Il s'agit du répertoire où le fichier de template va être stocké.
+  - `ynh_autoupdate.dest_script` : c'est le chemin du répertoire où le script de mise à jour sera installé sur le serveur. La valeur par défaut est `/usr/local/bin`.
 
 Si des mises à jour sont disponibles, elles sont faites automatiquement. En cas de problème suite à la mise à jour d'une application, vous pouvez lire les logs qui sont disponibles ici `/var/log/yunohost/categories/operation`. Vous avez aussi la possibilité de revenir à la version précédente car Yunohost fait toujours une sauvegarde automatique d'une application lorsqu'elle est mise à jour. 
 

README.md

@@ -148,7 +148,7 @@ A cron job can been set up to automate the check for system and application upda
   - `ynh_autoupdate.special_time`: it is mandatory. It allows you to specify when you want this task to be executed. Possible values: (`annually`,`daily`,`hourly`,`monthly`,`reboot`,`weekly`,`yearly`). To learn more about special times, click [here](https://docs.ansible.com/ansible/latest/collections/ansible/builtin/cron_module.html).
   - `ynh_autoupdate.apps`: is required. Enable automatic updating of Yunohost applications by setting the value to `True`.
   - `ynh_autoupdate.system`: is required. Enable automatic updating of the Yunohost system by setting the value to `True`.
-  - `ynh_autoupdate.dest_script` is mandatory. This is the directory where the template script will be stored.
+  - `ynh_autoupdate.dest_script`: it is the path to the directory where the update script will be installed on the server. The default value is `/usr/local/bin`.
 
 If available, updates are done automatically. In case of problems following an application update, you can read logs located in `/var/log/yunohost/categories/operation` . You also have the possibility to rollback to the previous version since Yunohost always makes an automatic backup of an application when it is updated. 
 

tasks/autoupdate.yml

@@ -21,7 +21,7 @@
 - name: Creates Yunohost autoupdate script
   ansible.builtin.template:
     src: "templates/ynh_autoupdate.sh.j2"
-    dest: "{{ ynh_autoupdate.dest_script }}"
+    dest: "{{ ynh_autoupdate.dest_script | default('/usr/local/bin') }}"
     owner: root
     group: root
     mode: '0740'
@@ -32,5 +32,5 @@
           Logs can be found in /var/log/yunohost/categories/operation"
     special_time: "{{ ynh_autoupdate.special_time }}"
     user: root
-    job: "{{ ynh_autoupdate.dest_script }}"
+    job: "{{ ynh_autoupdate.dest_script | default('/usr/local/bin') }}"
     cron_file: ynh_autoupdate_cron

templates/ynh_autoupdate.sh.j2

 #!/bin/sh
 
-{% if ynh_autoupdate.apps and ynh_autoupdate.system %}
-yunohost tools update && \
-yunohost tools upgrade system && \
-yunohost tools upgrade apps
-
-{% elif ynh_autoupdate.apps %}
-yunohost tools update && \
-yunohost tools upgrade apps
-
-{% elif ynh_autoupdate.system %}
-yunohost tools update && \
-yunohost tools upgrade system
-
-{% else %}
-echo "Read the readme to know more about ynh_autoupdate.apps and ynh_autoupdate.system"
-echo "https://lab.frogg.it/lydra/yunohost/ansible-yunohost/-/blob/main/README.md"
-exit 1
+yunohost tools update
+{% if ynh_autoupdate.system %}
+    yunohost tools upgrade system
+{% endif %}
+{% if ynh_autoupdate.apps %}
+    yunohost tools upgrade apps
+{% endif %}
+{% if ynh_autoupdate.system is false and ynh_autoupdate.apps is false %}
+    echo "Read the readme to know more about ynh_autoupdate.apps and ynh_autoupdate.system"
+    echo "https://lab.frogg.it/lydra/yunohost/ansible-yunohost/-/blob/main/README.md"
+    exit 1
 {% endif %}