diff options
author | Morris Jobke <hey@morrisjobke.de> | 2018-08-20 17:08:18 +0200 |
---|---|---|
committer | GitHub <noreply@github.com> | 2018-08-20 17:08:18 +0200 |
commit | 37869d9b2f7bf5e9f3779ccd344114649ce459c1 (patch) | |
tree | 336dc41c77bb07406537379339aa8f780f344dc0 /core/js/setupchecks.js | |
parent | 080572993e891a76721f4f7a7b76c85bc0e4c65d (diff) | |
parent | 3f790bb85b3544680f4af2e3e005d736a5aff8a0 (diff) | |
download | nextcloud-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.js | 17 |
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'), |