From fc969ae1d448b4209c27c2cae8222fbcc12a5b64 Mon Sep 17 00:00:00 2001
From: Kay0u <pierre.moltess@gmail.com>
Date: Wed, 29 Jan 2020 14:24:59 +0700
Subject: [PATCH] remove visitors only for if current value is /

---
 data/helpers.d/setting | 9 +++++++--
 1 file changed, 7 insertions(+), 2 deletions(-)

diff --git a/data/helpers.d/setting b/data/helpers.d/setting
index 5bcd7af32..9f68cb5d9 100644
--- a/data/helpers.d/setting
+++ b/data/helpers.d/setting
@@ -158,7 +158,12 @@ ynh_add_protected_uris() {
 #
 ynh_app_setting()
 {
-    ACTION="$1" APP="$2" KEY="$3" VALUE="${4:-}" python - <<EOF
+    if [[ "$1" == "delete" ]] && [[ "$3" =~ ^(unprotected|skipped)_ ]] 
+    then
+        current_value=$(ynh_app_setting_get --app=$app --key=$3)
+    fi
+
+    ACTION="$1" APP="$2" KEY="$3" VALUE="${4:-}" python2.7 - <<EOF
 import os, yaml, sys
 app, action = os.environ['APP'], os.environ['ACTION'].lower()
 key, value = os.environ['KEY'], os.environ.get('VALUE', None)
@@ -192,7 +197,7 @@ EOF
         if [[ "$1" == "set" ]] && [[ "${4:-}" == "/" ]]
         then
             ynh_permission_update --permission "main" --add "visitors"
-        elif [[ "$1" == "delete" ]]
+        elif [[ "$1" == "delete" ]] && [[ "${current_value:-}" == "/" ]]
         then
             ynh_permission_update --permission "main" --remove "visitors"
         fi