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

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.

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.