[Tickets #12097] Re: Export of mailfolders fails with cyrus imap

[noreply at bugs.horde.org]

BITTE NICHT AUF DIESE NACHRICHT ANTWORTEN. NACHRICHTEN AN DIESE  
E-MAIL-ADRESSE WERDEN NICHT GELESEN.

Ticket-URL: http://bugs.horde.org/ticket/12097
------------------------------------------------------------------------------
  Ticket           | 12097
  Aktualisiert Von | roderick.braun at ph-freiburg.de
  Zusammenfassung  | Export of mailfolders fails with cyrus imap
  Warteschlange    | Horde Groupware Webmail Edition
  Version          | 5.0.4
  Typ              | Bug
  Status           | Feedback
  Priorität        | 1. Low
  Milestone        |
  Patch            | 1
  Zuständige       |
+Neue Anlage      | imap-update-mapping-fix2.patch
------------------------------------------------------------------------------


roderick.braun at ph-freiburg.de (2013-03-07 16:20) hat geschrieben:

>> The problem is in the method "update" in .../Imap/Client/Ids/Map.php
>> which is not checking whether the mapping array contains only valid
>> values (I guess they should always be int?).
>
> Yes.  So the question is... WHY are they not all integers for you?
I doubt this is only  a problem of our setup because looking in my  
IMAP logs I can't find any non rfc answer on the server side which  
could explain the problem.

> Your patch is not useful because it masks the problem but doesn't fix it.

You are right it doesn't fix the problem - I traced the bug and found  
a problem in the method "_updateCache" in class  
"Horde_Imap_Client_Base".  The mapping array is created unchecked  
using the getUid() which is either an integer or "null". So I would  
suggest to check the retrieved uid before assigning it to the mapping  
array.

Again I attached a patch.

You may say: This is also not solving the origin problem but  I think  
as long as you cannot be sure the method returns integer values in all  
cases you have to check the result in calling processes.
(Btw: It is not nescessary to update the cached mapping anyway to  
export mails successfully.)





roderick.braun at ph-freiburg.de (2013-03-07 16:20) hat hochgeladen:  
imap-update-mapping-fix2.patch

http://bugs.horde.org/h/services/download/?module=whups&actionID=download_file&file=imap-update-mapping-fix2.patch&ticket=12097&fn=%2Fimap-update-mapping-fix2.patch