[Tickets #12589] Re: Kolab_Storage: Ensure status($folder) is not cached

[noreply at bugs.horde.org]

DO NOT REPLY TO THIS MESSAGE. THIS EMAIL ADDRESS IS NOT MONITORED.

Ticket URL: http://bugs.horde.org/ticket/12589
------------------------------------------------------------------------------
  Ticket             | 12589
  Updated By         | Thomas Jarosch <thomas.jarosch at intra2net.com>
  Summary            | Kolab_Storage: Ensure status($folder) is not cached
  Queue              | Horde Framework Packages
  Version            | Git master
  Type               | Bug
  State              | Feedback
  Priority           | 1. Low
  Milestone          |
  Patch              | 1
  Owners             | Jan Schneider
------------------------------------------------------------------------------


Thomas Jarosch <thomas.jarosch at intra2net.com> (2013-08-21 17:19) wrote:

> The correct way of fixing this (at least as of Horde_Imap_Client  
> 2.14.0) will be to add Horde_Imap_Client::STATUS_FORCE_REFRESH to  
> the mask passed in the $flags parameter to status().

while debugging an ActiveSync problem, I've noticed that  
Imap_Client::vanished()
internally calls Imap_Client::search(). The search() function internally uses
status() to optimize for empty mailboxes.

Is that a safe thing to do for a long running script with just one  
selected folder?
I think we need add the STATUS_FORCE_REFRESH flag in case
"current_selected_folder == $mailbox_to_check".

There are also other places we call status().