mirror of
https://github.com/nextcloud/server.git
synced 2024-07-28 20:16:02 +02:00
Merge pull request #12401 from owncloud/add-limit-parameter-to-getSharewith
Remove hard coded limit of 15 from getShareWith
This commit is contained in:
commit
54a6ba47c0
@ -281,8 +281,10 @@ if (isset($_POST['action']) && isset($_POST['itemType']) && isset($_POST['itemSo
|
||||
$users = array();
|
||||
$limit = 0;
|
||||
$offset = 0;
|
||||
while ($count < 15 && count($users) == $limit) {
|
||||
$limit = 15 - $count;
|
||||
// limit defaults to 15 if not specified via request parameter and can be no larger than 500
|
||||
$request_limit = min((int)$_GET['limit'] ?: 15, 500);
|
||||
while ($count < $request_limit && count($users) == $limit) {
|
||||
$limit = $request_limit - $count;
|
||||
if ($shareWithinGroupOnly) {
|
||||
$users = OC_Group::displayNamesInGroups($usergroups, (string)$_GET['search'], $limit, $offset);
|
||||
} else {
|
||||
@ -319,7 +321,7 @@ if (isset($_POST['action']) && isset($_POST['itemType']) && isset($_POST['itemSo
|
||||
continue;
|
||||
}
|
||||
|
||||
if ($count < 15) {
|
||||
if ($count < $request_limit) {
|
||||
if (!isset($_GET['itemShares'])
|
||||
|| !isset($_GET['itemShares'][OCP\Share::SHARE_TYPE_GROUP])
|
||||
|| !is_array((string)$_GET['itemShares'][OCP\Share::SHARE_TYPE_GROUP])
|
||||
|
@ -491,7 +491,7 @@ OC.Share={
|
||||
$('#shareWith').autocomplete({minLength: 2, delay: 750, source: function(search, response) {
|
||||
var $loading = $('#dropdown .shareWithLoading');
|
||||
$loading.removeClass('hidden');
|
||||
$.get(OC.filePath('core', 'ajax', 'share.php'), { fetch: 'getShareWith', search: search.term.trim(), itemShares: OC.Share.itemShares, itemType: itemType }, function(result) {
|
||||
$.get(OC.filePath('core', 'ajax', 'share.php'), { fetch: 'getShareWith', search: search.term.trim(), limit: 200, itemShares: OC.Share.itemShares, itemType: itemType }, function(result) {
|
||||
$loading.addClass('hidden');
|
||||
if (result.status == 'success' && result.data.length > 0) {
|
||||
$( "#shareWith" ).autocomplete( "option", "autoFocus", true );
|
||||
|
Loading…
Reference in New Issue
Block a user