summaryrefslogtreecommitdiffstats
path: root/settings
diff options
context:
space:
mode:
authorThomas Müller <thomas.mueller@tmit.eu>2015-10-01 15:37:55 +0200
committerThomas Müller <thomas.mueller@tmit.eu>2015-10-01 15:37:55 +0200
commitb2dd5cb61655b6ca82e8fd1fb117b42301d28c2b (patch)
tree98e25e54bcefd00d0169cbf18cf930bce17228d3 /settings
parent64994facce890e53a50bc75047e502e434c9c749 (diff)
downloadnextcloud-server-b2dd5cb61655b6ca82e8fd1fb117b42301d28c2b.tar.gz
nextcloud-server-b2dd5cb61655b6ca82e8fd1fb117b42301d28c2b.zip
save excluded groups in json format - fixes #10983
Diffstat (limited to 'settings')
-rw-r--r--settings/admin.php2
-rw-r--r--settings/js/admin.js6
2 files changed, 2 insertions, 6 deletions
diff --git a/settings/admin.php b/settings/admin.php
index 31c9b8c1376..ec49b3e823f 100644
--- a/settings/admin.php
+++ b/settings/admin.php
@@ -87,7 +87,7 @@ $template->assign('shareEnforceExpireDate', $appConfig->getValue('core', 'sharea
$excludeGroups = $appConfig->getValue('core', 'shareapi_exclude_groups', 'no') === 'yes' ? true : false;
$template->assign('shareExcludeGroups', $excludeGroups);
$excludedGroupsList = $appConfig->getValue('core', 'shareapi_exclude_groups_list', '');
-$excludedGroupsList = explode(',', $excludedGroupsList); // FIXME: this should be JSON!
+$excludedGroupsList = json_decode($excludedGroupsList);
$template->assign('shareExcludedGroupsList', implode('|', $excludedGroupsList));
$template->assign('encryptionEnabled', \OC::$server->getEncryptionManager()->isEnabled());
$backends = \OC::$server->getUserManager()->getBackends();
diff --git a/settings/js/admin.js b/settings/js/admin.js
index 7117c7b46cf..9f7133c6571 100644
--- a/settings/js/admin.js
+++ b/settings/js/admin.js
@@ -23,11 +23,7 @@ $(document).ready(function(){
OC.Settings.setupGroupsSelect($(element));
$(element).change(function(ev) {
var groups = ev.val || [];
- if (groups.length > 0) {
- groups = ev.val.join(','); // FIXME: make this JSON
- } else {
- groups = '';
- }
+ groups = JSON.stringify(groups);
OC.AppConfig.setValue('core', $(this).attr('name'), groups);
});
});