Opened 5 years ago

Closed 5 years ago

#1484976 closed Bugs (fixed)

Marking multiple messages for deletion doesn't work

Reported by: r@… Owned by:
Priority: 5 Milestone: 0.2-alpha
Component: Client Scripts Version: 0.1.1
Severity: normal Keywords:
Cc:

Description

After upgrading to RC-0.1.1, one can select multiple messages and move them to the Trash folder, however if RC is instead configured to mark messages as deleted, hitting the "delete" key or clicking the "delete" icon doesn't mark multiple messages as deleted (it appears to do nothing except remove focus from the highlighted messages). Firefox's error console complains "this.message_list.rows[id] has no properties, .../program/js/app.js, line 1507".

This functionality was working correctly in revision 1239 of app.js - the problem started with revision 1246.

Change History (6)

comment:1 Changed 5 years ago by alec

  • Milestone changed from later to 0.1.2

works for me, try svn-trunk

comment:2 Changed 5 years ago by r@…

I experience the same problem with [6f987669]. I'm running an "out of the box" configuration of RoundCube, the only changes I've made are:

$rcmail_config['db_dsnw'] = 'mysql://xxxx@xxxx/roundcubemail';
$rcmail_config['db_backend'] = 'db';
$rcmail_config['db_persistent'] = TRUE;
$rcmail_config['default_host'] = 'localhost';
$rcmail_config['mail_domain'] = 'xxxx.xxxx';
$rcmail_config['smtp_server'] = 'localhost';
$rcmail_config['trash_mbox'] = '';

...I've cleared the browser cache multiple times. Behavior is in the same in Firefox as well as IE. All I have to do to "fix" it is downgrade app.js to [fdccdb4a].

comment:3 Changed 5 years ago by r@…

Patch for app.js?

1502d1501
<       }
1514a1514
>      }

comment:4 Changed 5 years ago by r@…

Never mind that patch, it fixes marking multiple message for deletion but breaks marking them as read or unread.

comment:5 Changed 5 years ago by r@…

Please close this ticket - the problem was fixed in revision 1292 of app.js.

comment:6 Changed 5 years ago by thomasb

  • Resolution set to fixed
  • Status changed from new to closed
Note: See TracTickets for help on using tickets.