#1488182 closed Bugs (fixed)

new_messages hook is called incorrectly when 'Check for new mail' button is pressed immediately after user login (m/b problem in mailbox status routines)

Reported by: ural Owned by:
Priority: 5 Milestone: 0.7-beta
Component: Plugin API Version: git-master
Severity: normal Keywords: new_messages newmail_notifier newmail check
Cc:

Description

I tried newmail_notifier plugin and found a problem.
If I click on 'Check for new mail' button first time after login into roundcube, newmail notification is always pops out (I have no new mail in mailboxes at this moment).

---
Here is additional info:
my current folderlist is attached in png.
'check mail in all folders' is not set (no difference anyway).

1.Log how new_messages hook is called after first time when 'check for new mail button is pressed'(no new mail exist in this folders):

[11-Nov-2011 20:22:32 +0600]: notify hook: Sent,
[11-Nov-2011 20:22:32 +0600]: notify hook: Spam,
[11-Nov-2011 20:22:32 +0600]: notify hook: Bacula,
[11-Nov-2011 20:22:32 +0600]: notify hook: #konturinfo,
[11-Nov-2011 20:22:32 +0600]: notify hook: Nagios,
[11-Nov-2011 20:22:32 +0600]: notify hook: Public/&BB0EPgQyBD4EQQRCBDg-,
[11-Nov-2011 20:22:32 +0600]: notify hook: Public/&BB8EPgQ0BDQENQRABDYEOgQw-,
[11-Nov-2011 20:22:32 +0600]: notify hook: Uralsib,
[11-Nov-2011 20:22:32 +0600]: notify hook: &BBEEOwQwBDEEOwQw-,

2.Log how new_messages hook is called on standard check recent procedure (each 5 min, no problem here, new mail is in inbox):

[11-Nov-2011 20:41:39 +0600]: notify hook: INBOX,


Attachments (1)

folderlist.png (23.2 KB) - added by ural 19 months ago.

Download all attachments as: .zip

Change History (2)

Changed 19 months ago by ural

comment:1 Changed 19 months ago by alec

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

Hitting the button shouldn't show notifications at all. I assume you have not up-to-date code from svn. In [1cb0d640] I've added additional check which should also help to prevent similiar situations.

Note: See TracTickets for help on using tickets.