6.0.0-beta1
7/17/25

[#11674] Kolab_Storage: Fix modify of Kolab data object: Notify framework about deleted backend ID
Summary Kolab_Storage: Fix modify of Kolab data object: Notify framework about deleted backend ID
Queue Horde Framework Packages
Queue Version Git master
Type Bug
State Resolved
Priority 2. Medium
Owners wrobel (at) horde (dot) org
Requester thomas.jarosch (at) intra2net (dot) com
Created 11/08/2012 (4634 days ago)
Due
Updated 11/22/2012 (4620 days ago)
Assigned 11/08/2012 (4634 days ago)
Resolved 11/22/2012 (4620 days ago)
Github Issue Link
Github Pull Request
Milestone
Patch Yes

History
11/22/2012 08:13:32 PM Gunnar Wrobel Comment #3
State ⇒ Resolved
Reply to this comment
Thanks for the patch. Applied and should be fixed.
11/22/2012 07:24:00 PM Git Commit Comment #2 Reply to this comment
Changes have been made in Git (master):

commit be17770018bc13c32b542f932fc28130d5db7b8e
Author: Thomas Jarosch <thomas.jarosch@intra2net.com>
Date:   Thu Nov 8 18:49:44 2012 +0100

     Import fix for Bug #11674.

     Fix modify of Kolab data object: Notify framework about deleted backend ID

     Otherwise the IMAP UID ("backend ID") -> object UID cache (B2O) pollutes.

     Signed-off-by: Thomas Jarosch <thomas.jarosch@intra2net.com>
     Signed-off-by: Gunnar Wrobel <p@rdus.de>

  .../lib/Horde/Kolab/Storage/Data/Base.php          |    3 ++-
  1 files changed, 2 insertions(+), 1 deletions(-)

http://git.horde.org/horde-git/-/commit/be17770018bc13c32b542f932fc28130d5db7b8e
11/08/2012 05:58:53 PM Thomas Jarosch State ⇒ Assigned
Assigned to Gunnar Wrobel
 
11/08/2012 05:57:52 PM Thomas Jarosch Comment #1
Priority ⇒ 2. Medium
New Attachment: 0001-Fix-modify-of-Kolab-data-object-Notify-framework-abo.patch Download
Patch ⇒ Yes
Milestone ⇒
Summary ⇒ Kolab_Storage: Fix modify of Kolab data object: Notify framework about deleted backend ID
Type ⇒ Bug
State ⇒ Unconfirmed
Queue ⇒ Horde Framework Packages
Reply to this comment
Hi,

when we are modifying Kolab objects, we didn't tell the framework 
about the deleted backend ID.
As a result, the backend ID (IMAP UID) -> object UID mapping cache 
becomes more and more polluted.

Attached patch fixes the issue.

@Gunnar: I carefully checked the code path. Only "drawback" about the 
delete notify is that the store() operation of the data cache will 
remove all attachments corresponding to the object UID. Later on the 
attachments are stored again, so still should be correct IMHO. Please 
double check.

Thomas

Saved Queries