Testing (#9)

* Fix linter

README.md

@@ -1,7 +1,7 @@
 # phpLDAPadmin for YunoHost
 
 [![Integration level](https://dash.yunohost.org/integration/phpldapadmin.svg)](https://dash.yunohost.org/appci/app/phpldapadmin) ![](https://ci-apps.yunohost.org/ci/badges/phpldapadmin.status.svg) ![](https://ci-apps.yunohost.org/ci/badges/phpldapadmin.maintain.svg)  
-[![Install phpLDAPadmin with YunoHost](https://install-app.yunohost.org/install-with-yunohost.png)](https://install-app.yunohost.org/?app=phpldapadmin)
+[![Install phpLDAPadmin with YunoHost](https://install-app.yunohost.org/install-with-yunohost.svg)](https://install-app.yunohost.org/?app=phpldapadmin)
 
 > *This package allow you to install phpLDAPadmin quickly and simply on a YunoHost server.  
 If you don't have YunoHost, please see [here](https://yunohost.org/#/install) to know how to install and enjoy it.*

check_process

@@ -17,11 +17,7 @@
 		upgrade=1	from_commit=785706fb2607e95fbc4876e1367a715a6b8543d2
 		backup_restore=1
 		multi_instance=1
-		incorrect_path=1
 		change_url=0
-;;; Levels
-	# If the level 5 (Package linter) is forced to 1. Please add justifications here.
-	Level 5=auto
 ;;; Options
 Email=aymhce@gmail.com
 Notification=none

manifest.json

@@ -14,41 +14,29 @@
         "email": "aymhce@gmail.com"
     },
     "requirements": {
-        "yunohost": ">= 4.0.0"
+        "yunohost": ">= 4.1.7"
     },
     "multi_instance": true,
     "services": [
         "nginx",
-        "php7.0-fpm"
+        "php7.3-fpm"
      ],
     "arguments": {
         "install" : [
             {
                 "name": "domain",
                 "type": "domain",
-                "ask": {
-                    "en": "Choose a domain for phpLDAPadmin",
-                    "fr": "Choisissez un domaine pour phpLDAPadmin"
-                },
                 "example": "domain.org"
             },
             {
                 "name": "path",
                 "type": "path",
-                "ask": {
-                    "en": "Choose a path for phpLDAPadmin",
-                    "fr": "Choisissez un chemin pour phpLDAPadmin"
-                },
                 "example": "/phpldapadmin",
                 "default": "/phpldapadmin"
             },
             {
                 "name": "admin",
                 "type": "user",
-                "ask": {
-                    "en": "Choose the only allowed admin user",
-                    "fr": "Choisissez l'unique utilisateur autorisé"
-                },
                 "example": "johndoe"
             }
         ]

pull_request_template.md

@@ -11,11 +11,6 @@
 - [ ] Upgrade from last version tested.
 - [ ] Can be reviewed and tested.
 
-## Validation
+## Package_check results
 ---
-- [ ] **Code review**
+* An automatic package_check will be launch at https://ci-apps-dev.yunohost.org/, when you add a specific comment to your Pull Request: "!testme", "!gogogadgetoci" or "By the power of systemd, I invoke The Great App CI to test this Pull Request!"*
-- [ ] **Approval (LGTM)**  
-
-- **CI succeeded** : 
-
-When the PR is marked as ready to merge, you have to wait for 3 days before really merging it.

scripts/backup

@@ -27,7 +27,10 @@ domain=$(ynh_app_setting_get --app=$app --key=domain)
 phpversion=$(ynh_app_setting_get --app=$app --key=phpversion)
 
 #=================================================
-# STANDARD BACKUP STEPS
+# DECLARE DATA AND CONF FILES TO BACKUP
+#=================================================
+ynh_print_info --message="Declaring files to be backed up..."
+
 #=================================================
 # BACKUP THE APP MAIN DIR
 #=================================================

scripts/restore

@@ -82,7 +82,7 @@ ynh_restore_file --origin_path="/etc/php/$phpversion/fpm/pool.d/$app.conf"
 # GRANT ACCESS RIGHT TO ADMIN 
 #=================================================
 
-yunohost app addaccess $app -u $admin
+ynh_permission_update --permission "main" --add $admin --remove "all_users"
 
 #=================================================
 # GENERIC FINALIZATION

scripts/upgrade

@@ -52,12 +52,6 @@ ynh_clean_setup () {
 # Exit if an error occurs during the execution of the script
 ynh_abort_if_errors
 
-#=================================================
-# CHECK THE PATH
-#=================================================
-
-path_url=$(ynh_normalize_url_path --path_url=$path_url)
-
 #=================================================
 # STANDARD UPGRADE STEPS
 #=================================================