Opened 3 years ago
Closed 3 years ago
#1486734 closed Feature Patches (fixed)
Trigger event listupdate after message removal from list
| Reported by: | sager | Owned by: | |
|---|---|---|---|
| Priority: | 5 | Milestone: | 0.4-stable |
| Component: | Plugin API | Version: | 0.3.1 |
| Severity: | normal | Keywords: | listupdate |
| Cc: |
Description
I had to patch app.js (see below) to be able to trigger the event "listupdate" after removing a message from a folder.
*** app.js.orig 2010-05-01 15:31:50.000000000 +0200
--- app.js.src 2010-05-13 21:51:26.000000000 +0200
***************
*** 4226,4231 ****
--- 4226,4234 ----
this.enable_command('reply', 'reply-all', 'forward', 'delete', 'mark', 'print', 'open', 'edit', 'viewsource', 'download', true);
} else if (this.message_list)
this.message_list.init();
+
+ this.triggerEvent("listupdate",{folder:this.env.mailbox,rowcount:this.message_list.rowcount});
+
break;
case 'purge':
Change History (5)
comment:1 Changed 3 years ago by sager
- Type changed from Bugs to Patches
comment:2 Changed 3 years ago by alec
comment:3 Changed 3 years ago by alec
For what purpose are you using this event?
comment:4 Changed 3 years ago by thomasb
It sounds reasonable to also trigger "listupdate" after removing a message from it. It actually is an update of the list.
comment:5 Changed 3 years ago by alec
- Resolution set to fixed
- Status changed from new to closed
Fixed in [13e1552d].
Note: See
TracTickets for help on using
tickets.

This code
else if (this.message_list) this.message_list.init();has been removed in trunk, so I'm not sure we should trigger listupdate in this place.