diff options
| author | Joas Schilling <nickvergessen@gmx.de> | 2014-07-21 10:43:29 +0200 |
|---|---|---|
| committer | Vincent Petry <pvince81@owncloud.com> | 2014-07-21 20:45:27 +0200 |
| commit | e482ba60bcbbc2c5b8b143946bfe6ac14e7fdf7f (patch) | |
| tree | a34e3875e5d7c14200a28e714991ee3e8304ecd6 /settings | |
| parent | bad2d4d4080d11c9d187b7eca06fb0ca4fc6a29b (diff) | |
| download | nextcloud-server-e482ba60bcbbc2c5b8b143946bfe6ac14e7fdf7f.tar.gz nextcloud-server-e482ba60bcbbc2c5b8b143946bfe6ac14e7fdf7f.zip | |
Correctly use groups parameter only when its not empty
Fix #9745
Backport of c84c1f5 from master
Diffstat (limited to 'settings')
| -rw-r--r-- | settings/ajax/createuser.php | 16 |
1 files changed, 9 insertions, 7 deletions
diff --git a/settings/ajax/createuser.php b/settings/ajax/createuser.php index ae1d8856f43..463c15d59e8 100644 --- a/settings/ajax/createuser.php +++ b/settings/ajax/createuser.php @@ -5,21 +5,23 @@ OC_JSON::checkSubAdminUser(); if(OC_User::isAdminUser(OC_User::getUser())) { $groups = array(); - if( isset( $_POST["groups"] )) { + if (!empty($_POST["groups"])) { $groups = $_POST["groups"]; } }else{ - if(isset( $_POST["groups"] )) { + if (isset($_POST["groups"])) { $groups = array(); - foreach($_POST["groups"] as $group) { - if(OC_SubAdmin::isGroupAccessible(OC_User::getUser(), $group)) { - $groups[] = $group; + if (!empty($_POST["groups"])) { + foreach ($_POST["groups"] as $group) { + if (OC_SubAdmin::isGroupAccessible(OC_User::getUser(), $group)) { + $groups[] = $group; + } } } - if(count($groups) === 0) { + if (empty($groups)) { $groups = OC_SubAdmin::getSubAdminsGroups(OC_User::getUser()); } - }else{ + } else { $groups = OC_SubAdmin::getSubAdminsGroups(OC_User::getUser()); } } |