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

* Your Email Address
* Spam protection	Enter the letters below: __ .__..___. .. . / `\| \| _/ \|\| \| \__.\|__\./__.\__\|\|__\|
Comment	>>> 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.) > >
Attachment
Watch this ticket