summaryrefslogtreecommitdiffstats
path: root/core/js/setupchecks.js
diff options
context:
space:
mode:
authorMorris Jobke <hey@morrisjobke.de>2018-08-20 17:08:18 +0200
committerGitHub <noreply@github.com>2018-08-20 17:08:18 +0200
commit37869d9b2f7bf5e9f3779ccd344114649ce459c1 (patch)
tree336dc41c77bb07406537379339aa8f780f344dc0 /core/js/setupchecks.js
parent080572993e891a76721f4f7a7b76c85bc0e4c65d (diff)
parent3f790bb85b3544680f4af2e3e005d736a5aff8a0 (diff)
downloadnextcloud-server-37869d9b2f7bf5e9f3779ccd344114649ce459c1.tar.gz
nextcloud-server-37869d9b2f7bf5e9f3779ccd344114649ce459c1.zip
Merge pull request #10628 from nextcloud/feature/10154/app-directory-permission-check
Adds a permission check for app directories
Diffstat (limited to 'core/js/setupchecks.js')
-rw-r--r--core/js/setupchecks.js17
1 files changed, 17 insertions, 0 deletions
diff --git a/core/js/setupchecks.js b/core/js/setupchecks.js
index 93072981e99..13e351445e9 100644
--- a/core/js/setupchecks.js
+++ b/core/js/setupchecks.js
@@ -316,6 +316,23 @@
type: OC.SetupChecks.MESSAGE_TYPE_WARNING
});
}
+
+ if(data.appDirsWithDifferentOwner.length > 0) {
+ var appDirsWithDifferentOwner = data.appDirsWithDifferentOwner.reduce(
+ function(appDirsWithDifferentOwner, directory) {
+ return appDirsWithDifferentOwner + '<li>' + directory + '</li>';
+ },
+ ''
+ );
+ messages.push({
+ msg: t('core', 'Some app directories are owned by a different user than the web server one. ' +
+ 'This may be the case if apps have been installed manually. ' +
+ 'Check the permissions of the following app directories:')
+ + '<ul>' + appDirsWithDifferentOwner + '</ul>',
+ type: OC.SetupChecks.MESSAGE_TYPE_WARNING
+ });
+ }
+
} else {
messages.push({
msg: t('core', 'Error occurred while checking server setup'),