Skip to content

Commit

Permalink
Used POST for the batch delete call instead of parameters
Browse files Browse the repository at this point in the history
This might have bust the 1024 limit. With post, no problem.
  • Loading branch information
lcharland committed Oct 10, 2012
1 parent 58bb454 commit d597aeb
Showing 1 changed file with 4 additions and 3 deletions.
7 changes: 4 additions & 3 deletions UI/WebServerResources/ContactsUI.js
Expand Up @@ -492,12 +492,13 @@ function onToolbarDeleteSelectedContactsConfirm(dialogId) {
disposeDialog();
var contactsList = $('contactsList');
var rowIds = contactsList.getSelectedRowsId();
var urlstr = (URLForFolderID(Contact.currentAddressBook)
+ "/batchDelete?ids=" + rowIds.join("/"));
var urlstr = (URLForFolderID(Contact.currentAddressBook) + "/batchDelete");
for (var i = 0; i < rowIds.length; i++) {
delete cachedContacts[Contact.currentAddressBook + "/" + rowIds[i]];
}
triggerAjaxRequest(urlstr, onContactDeleteEventCallback, rowIds);
triggerAjaxRequest(urlstr, onContactDeleteEventCallback, rowIds,
('ids=' + rowIds.join("/")),
{ "Content-type": "application/x-www-form-urlencoded" });
}

function onContactDeleteEventCallback(http) {
Expand Down

0 comments on commit d597aeb

Please sign in to comment.