Merge pull request #15149 from owncloud/fix-enable-for-group
Correctly get array of groups and send OCP\IGroup objects to enable meth...
This commit is contained in:
commit
5e71d1558a
|
@ -269,7 +269,15 @@ class OC_App {
|
|||
|
||||
$appManager = \OC::$server->getAppManager();
|
||||
if (!is_null($groups)) {
|
||||
$appManager->enableAppForGroups($app, $groups);
|
||||
$groupManager = \OC::$server->getGroupManager();
|
||||
$groupsList = [];
|
||||
foreach ($groups as $group) {
|
||||
$groupItem = $groupManager->get($group);
|
||||
if ($groupItem instanceof \OCP\IGroup) {
|
||||
$groupsList[] = $groupManager->get($group);
|
||||
}
|
||||
}
|
||||
$appManager->enableAppForGroups($app, $groupsList);
|
||||
} else {
|
||||
$appManager->enableApp($app);
|
||||
}
|
||||
|
|
|
@ -413,7 +413,7 @@ OC.Settings.Apps = OC.Settings.Apps || {
|
|||
var element = $(this).parent().find('input.enable');
|
||||
var groups = $(this).val();
|
||||
if (groups && groups !== '') {
|
||||
groups = groups.split(',');
|
||||
groups = groups.split('|');
|
||||
} else {
|
||||
groups = [];
|
||||
}
|
||||
|
|
Loading…
Reference in New Issue