only chown if existing

Signed-off-by: Daniel Hansson <mailto@danielhansson.nu>
nextcloud · Oct 14, 2022 · b048bb5 · b048bb5
1 parent 99b612a
commit b048bb5
Showing 1 changed file with 5 additions and 2 deletions.
diff --git a/static/setup_secure_permissions_nextcloud.sh b/static/setup_secure_permissions_nextcloud.sh
@@ -73,7 +73,7 @@ then
 fi
 
 # Nextcloud datafolder
-if [ -d "$NCDATA" ]
+if [ -d "${NCDATA}" ]
 then
     # Always chown root dir
     chown "${htuser}":"${htgroup}" "${NCDATA}"/
@@ -83,7 +83,10 @@ then
         chown -R "${htuser}":"${htgroup}" "${NCDATA}"/
     fi
     # Also always chown files_external (https://github.com/nextcloud/vm/issues/2398)
-    chown -R "${htuser}":"${htgroup}" "${NCDATA}"/files_external
+    if [ -d "${NCDATA}"/files_external ]
+    then
+        chown -R "${htuser}":"${htgroup}" "${NCDATA}"/files_external
+    fi
 fi
 
 chmod +x "${NCPATH}"/occ