Opened 3 years ago

Closed 3 years ago

#1486708 closed Feature Patches (fixed)

Trigger event listupdate for email search result listing

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 a search (quicksearch). Beforehand "listupdate" was only called within non filtered "list" actions.

This may be necessary in task address book too.

*** app.js.orig 2010-05-01 15:31:50.000000000 +0200
--- app.js.src  2010-05-01 15:32:55.000000000 +0200
***************
*** 4243,4248 ****
--- 4243,4249 ----

        case 'check-recent':
        case 'getunread':
+       case 'search':
        case 'list':
          if (this.task == 'mail') {
            if (this.message_list && response.action == 'list')
***************
*** 4250,4256 ****
            this.enable_command('show', 'expunge', 'select-all', 'select-none', 'sort', (this.env.messagecount > 0));
            this.enable_command('purge', this.purge_mailbox_test());

!           if (response.action == 'list')
              this.triggerEvent('listupdate', { folder:this.env.mailbox, rowcount:this.message_list.rowcount });
          }
          else if (this.task == 'addressbook') {
--- 4251,4257 ----
            this.enable_command('show', 'expunge', 'select-all', 'select-none', 'sort', (this.env.messagecount > 0));
            this.enable_command('purge', this.purge_mailbox_test());

!           if (response.action == 'list' || response.action == 'search')
              this.triggerEvent('listupdate', { folder:this.env.mailbox, rowcount:this.message_list.rowcount });
          }
          else if (this.task == 'addressbook') {

Change History (1)

comment:1 Changed 3 years ago by alec

  • Resolution set to fixed
  • Status changed from new to closed

Fixed in [c833ed40].

Note: See TracTickets for help on using tickets.