diff --git a/ChangeLog b/ChangeLog index a70e91cb0..7444b1f1a 100644 --- a/ChangeLog +++ b/ChangeLog @@ -23,6 +23,8 @@ * UI/WebServerResources/MailerUI.js (updateUnseenCount): used the method charAt to read a string character instead of accessing it as an array (was not supported by IE). + (saveAs): fixed a selection bug where only the first message would + be saved. * UI/WebServerResources/UIxAttendeesEditor.js (performSearchCallback): improved autocompletion when changing diff --git a/UI/WebServerResources/MailerUI.js b/UI/WebServerResources/MailerUI.js index a5f36dced..b91da1d76 100644 --- a/UI/WebServerResources/MailerUI.js +++ b/UI/WebServerResources/MailerUI.js @@ -509,7 +509,7 @@ function deleteMessageWithDelay(url, id, mailbox, messageId) { function onPrintCurrentMessage(event) { var messageList = $("messageListBody").down("TBODY"); - var rows = messageList.getSelectedNodes(); + var rows = messageList.getSelectedNodesId(); if (rows.length == 0) { showAlertDialog(_("Please select a message to print.")); } @@ -2486,19 +2486,13 @@ function onMarkMenuPrepareVisibility() { function saveAs(event) { var messageList = $("messageListBody").down("TBODY"); - var rows = messageList.getSelectedNodes(); - var uids = new Array(); // message IDs - var paths = new Array(); // row IDs - if (rows.length > 0) { - for (var i = 0; i < rows.length; i++) { - var uid = rows[i].readAttribute("id").substr(4); - var path = Mailer.currentMailbox + "/" + uid; - uids.push(uid); - paths.push(path); - } + var uids = messageList.getSelectedNodesId(); + if (uids.length > 0) { + for (var i = 0; i < uids.length; i++) + uids[i] = uids[i].substr(4); var url = ApplicationBaseURL + encodeURI(Mailer.currentMailbox) + "/saveMessages"; - window.location.href = (url+"?id="+uids+"&uid="+uids+"&mailbox="+Mailer.currentMailbox+"&path="+paths); + window.location.href = url + "?uid=" + uids.join(","); } else showAlertDialog(_("Please select a message."));