Skip to content

Commit

Permalink
Added select2 on the apps page
Browse files Browse the repository at this point in the history
Moved setupGroupsSelect() from admin.js to a common settings.js
as OC.Settings.setupGoupsSelect().

Now using select2 as well on the apps page.
  • Loading branch information
Vincent Petry committed Sep 9, 2014
1 parent 39f5580 commit 0d28ba0
Show file tree
Hide file tree
Showing 6 changed files with 99 additions and 88 deletions.
3 changes: 2 additions & 1 deletion settings/admin.php
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,8 @@

OC_Util::checkAdminUser();

OC_Util::addStyle( "settings", "settings" );
OCP\Util::addStyle('settings', 'settings');
OCP\Util::addScript('settings', 'settings');
OC_Util::addScript( "settings", "admin" );
OC_Util::addScript( "settings", "log" );
OC_Util::addScript( 'core', 'multiselect' );
Expand Down
8 changes: 4 additions & 4 deletions settings/apps.php
Original file line number Diff line number Diff line change
Expand Up @@ -24,17 +24,17 @@
OC_Util::checkAdminUser();

// Load the files we need
OC_Util::addStyle( "settings", "settings" );
OC_Util::addScript("core", "multiselect");
OCP\Util::addStyle('settings', 'settings' );
OCP\Util::addScript('settings', 'settings');
OCP\Util::addScript('core', 'select2/select2');
OCP\Util::addStyle('core', 'select2/select2');
OC_App::setActiveNavigationEntry( "core_apps" );

$combinedApps = OC_App::listAllApps();
$groups = \OC_Group::getGroups();

$tmpl = new OC_Template( "settings", "apps", "user" );

$tmpl->assign('apps', $combinedApps);
$tmpl->assign('groups', $groups);

$appid = (isset($_GET['appid'])?strip_tags($_GET['appid']):'');

Expand Down
63 changes: 1 addition & 62 deletions settings/js/admin.js
Original file line number Diff line number Diff line change
@@ -1,64 +1,3 @@
var SharingGroupList = {
setupGroupsSelect: function($elements) {
if ($elements.length > 0) {
// note: settings are saved through a "change" event registered
// on all input fields
$elements.select2({
placeholder: t('core', 'Groups'),
allowClear: true,
multiple: true,
ajax: {
url: OC.generateUrl('/settings/ajax/grouplist'),
dataType: 'json',
quietMillis: 100,
data: function (term) {
return {
pattern: term, //search term
};
},
results: function (data) {
if (data.status === "success") {
var results = [];

// add groups
$.each(data.data.adminGroups, function(i, group) {
results.push({id:group.id, displayname:group.name});
});
$.each(data.data.groups, function(i, group) {
results.push({id:group.id, displayname:group.name});
});

return {results: results};
} else {
//FIXME add error handling
}
}
},
id: function(element) {
return element.id;
},
initSelection: function(element, callback) {
var selection =
_.map(($(element).val() || []).split(',').sort(),
function(groupName) {
return {
id: groupName,
displayname: groupName
};
});
callback(selection);
},
formatResult: function (element) {
return element.displayname;
},
formatSelection: function (element) {
return element.displayname;
}
});
}
}
};

$(document).ready(function(){
var params = OC.Util.History.parseUrlQuery();

Expand All @@ -80,7 +19,7 @@ $(document).ready(function(){


$('#excludedGroups').each(function (index, element) {
SharingGroupList.setupGroupsSelect($(element));
OC.Settings.setupGroupsSelect($(element));
});


Expand Down
35 changes: 19 additions & 16 deletions settings/js/apps.js
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,11 @@

OC.Settings = OC.Settings || {};
OC.Settings.Apps = OC.Settings.Apps || {
setupGroupsSelect: function() {
OC.Settings.setupGroupsSelect($('#group_select'), {
placeholder: t('core', 'All')
});
},
loadApp:function(app) {
var page = $('#app-content');
page.find('p.license').show();
Expand Down Expand Up @@ -112,23 +117,16 @@ OC.Settings.Apps = OC.Settings.Apps || {
page.find(".warning").hide();
}

page.find("div.multiselect").parent().remove();
if(OC.Settings.Apps.isType(app, 'filesystem') || OC.Settings.Apps.isType(app, 'prelogin') ||
OC.Settings.Apps.isType(app, 'authentication') || OC.Settings.Apps.isType(app, 'logging')) {
page.find("#groups_enable").hide();
page.find("label[for='groups_enable']").hide();
page.find("#groups_enable").attr('checked', null);
} else {
$('#group_select > option').each(function (i, el) {
if (app.groups.length === 0 || app.groups.indexOf(el.value) >= 0) {
$(el).attr('selected', 'selected');
} else {
$(el).attr('selected', null);
}
});
$('#group_select').val((app.groups || []).join(','));
if (app.active) {
if (app.groups.length) {
$('#group_select').multiSelect();
OC.Settings.Apps.setupGroupsSelect();
page.find("#groups_enable").attr('checked','checked');
} else {
page.find("#groups_enable").attr('checked', null);
Expand Down Expand Up @@ -383,6 +381,11 @@ $(document).ready(function(){
$('#group_select').change(function() {
var element = $('#app-content input.enable');
var groups = $(this).val();
if (groups && groups !== '') {
groups = groups.split(',');
} else {
groups = [];
}
var appid = element.data('appid');
if (appid) {
OC.Settings.Apps.enableApp(appid, false, element, groups);
Expand All @@ -404,14 +407,14 @@ $(document).ready(function(){
}

$("#groups_enable").change(function() {
var $select = $('#group_select');
$select.val('');
if (this.checked) {
$("div.multiselect").parent().remove();
$('#group_select').multiSelect();
} else {
$('#group_select').hide().val(null);
$("div.multiselect").parent().remove();
OC.Settings.Apps.setupGroupsSelect();
}

$('#group_select').change();
else {
$select.select2('destroy');
}
$select.change();
});
});
72 changes: 72 additions & 0 deletions settings/js/settings.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,72 @@
/**
* Copyright (c) 2014, Vincent Petry <[email protected]>
* This file is licensed under the Affero General Public License version 3 or later.
* See the COPYING-README file.
*/
OC.Settings = OC.Settings || {};
OC.Settings = _.extend(OC.Settings, {
/**
* Setup selection box for group selection.
* @param $elements jQuery element (hidden input) to setup select2 on
* @param [extraOptions] extra options hash to pass to select2
*/
setupGroupsSelect: function($elements, extraOptions) {
if ($elements.length > 0) {
// note: settings are saved through a "change" event registered
// on all input fields
$elements.select2(_.extend({
placeholder: t('core', 'Groups'),
allowClear: true,
multiple: true,
ajax: {
url: OC.generateUrl('/settings/ajax/grouplist'),
dataType: 'json',
quietMillis: 100,
data: function (term) {
return {
pattern: term, //search term
};
},
results: function (data) {
if (data.status === "success") {
var results = [];

// add groups
$.each(data.data.adminGroups, function(i, group) {
results.push({id:group.id, displayname:group.name});
});
$.each(data.data.groups, function(i, group) {
results.push({id:group.id, displayname:group.name});
});

return {results: results};
} else {
//FIXME add error handling
}
}
},
id: function(element) {
return element.id;
},
initSelection: function(element, callback) {
var selection =
_.map(($(element).val() || []).split(',').sort(),
function(groupName) {
return {
id: groupName,
displayname: groupName
};
});
callback(selection);
},
formatResult: function (element) {
return element.displayname;
},
formatSelection: function (element) {
return element.displayname;
}
}, extraOptions || {}));
}
}
});

6 changes: 1 addition & 5 deletions settings/templates/apps.php
Original file line number Diff line number Diff line change
Expand Up @@ -58,11 +58,7 @@ class="version"></span><small class="externalapp" style="visibility:hidden;"></s
<input class="hidden" type="checkbox" id="groups_enable"/>
<label class="hidden" for="groups_enable"><?php p($l->t('Enable only for specific groups')); ?></label>
<br />
<select class="hidden" id="group_select" multiple="multiple" title="<?php p($l->t('All')); ?>">
<?php foreach($_['groups'] as $group):?>
<option value="<?php p($group);?>"><?php p($group); ?></option>
<?php endforeach;?>
</select>
<input type="hidden" id="group_select" title="<?php p($l->t('All')); ?>" style="width: 200px">

<div class="warning hidden"></div>
</div>
Expand Down

0 comments on commit 0d28ba0

Please sign in to comment.