From 7eecf873da8d2f28e20dc8fd0e949e6abc5762b4 Mon Sep 17 00:00:00 2001 From: Aleksander Machniak <alec@alec.pl> Date: Mon, 09 Dec 2013 13:16:14 -0500 Subject: [PATCH] Fix issue where children of selected and collapsed thread were skipped on various actions (#1489457) --- program/js/app.js | 17 ++--------------- 1 files changed, 2 insertions(+), 15 deletions(-) diff --git a/program/js/app.js b/program/js/app.js index 398417a..bad8a2f 100644 --- a/program/js/app.js +++ b/program/js/app.js @@ -2662,20 +2662,7 @@ // delete selected messages from the current mailbox this.delete_messages = function(event) { - var uid, i, len, trash = this.env.trash_mailbox, - list = this.message_list, - selection = list ? list.get_selection() : []; - - // exit if no mailbox specified or if selection is empty - if (!this.env.uid && !selection.length) - return; - - // also select childs of collapsed rows - for (i=0, len=selection.length; i<len; i++) { - uid = selection[i]; - if (list.rows[uid].has_children && !list.rows[uid].expanded) - list.select_children(uid); - } + var list = this.message_list, trash = this.env.trash_mailbox; // if config is set to flag for deletion if (this.env.flag_for_deletion) { @@ -2715,7 +2702,7 @@ this._with_selected_messages('delete', post_data); }; - // Send a specifc move/delete request with UIDs of all selected messages + // Send a specific move/delete request with UIDs of all selected messages // @private this._with_selected_messages = function(action, post_data, lock) { -- Gitblit v1.9.1