ZP-584 downloading email attachments within public folders fails on some

android devices.

Released under the Affero GNU General Public License (AGPL) version 3.

src/backend/zarafa/mapimapping.php

@@ -210,6 +210,7 @@ class MAPIMapping {
             "html"                  => PR_HTML,
             "rtfinsync"             => PR_RTF_IN_SYNC,
             "processed"             => PR_PROCESSED,
+            "storeentryid"          => PR_STORE_ENTRYID,
         );
     }
 

src/backend/zarafa/mapiprovider.php

@@ -677,6 +677,7 @@ class MAPIProvider {
         $attachtable = mapi_message_getattachmenttable($mapimessage);
         $rows = mapi_table_queryallrows($attachtable, array(PR_ATTACH_NUM));
         $entryid = bin2hex($messageprops[$emailproperties["entryid"]]);
+        $storeentryid = bin2hex($messageprops[$emailproperties["storeentryid"]]);
 
         foreach($rows as $row) {
             if(isset($row[PR_ATTACH_NUM])) {
@@ -708,7 +709,7 @@ class MAPIProvider {
 
                 // set AS version specific parameters
                 if (Request::GetProtocolVersion() >= 12.0) {
-                    $attach->filereference = $entryid.":".$row[PR_ATTACH_NUM];
+                    $attach->filereference = sprintf("%s:%s:%s", $storeentryid, $entryid, $row[PR_ATTACH_NUM]);
                     $attach->method = (isset($attachprops[PR_ATTACH_METHOD])) ? $attachprops[PR_ATTACH_METHOD] : ATTACH_BY_VALUE;
 
                     // if displayname does not have the eml extension for embedde messages, android and WP devices won't open it
@@ -741,7 +742,7 @@ class MAPIProvider {
                 }
                 else {
                     $attach->attsize = $attachprops[PR_ATTACH_SIZE];
-                    $attach->attname = $entryid.":".$row[PR_ATTACH_NUM];
+                    $attach->attname = sprintf("%s:%s:%s", $storeentryid, $entryid, $row[PR_ATTACH_NUM]);
                     if(!isset($message->attachments))
                         $message->attachments = array();
 

src/backend/zarafa/zarafa.php

@@ -82,6 +82,7 @@ class BackendZarafa implements IBackend, ISearchProvider {
     private $store;
     private $storeName;
     private $storeCache;
+    private $storeProps;
     private $importedFolders;
     private $notifications;
     private $changesSink;
@@ -108,6 +109,7 @@ class BackendZarafa implements IBackend, ISearchProvider {
         $this->store = false;
         $this->storeName = false;
         $this->storeCache = array();
+        $this->storeProps = array();
         $this->importedFolders = array();
         $this->notifications = false;
         $this->changesSink = false;
@@ -665,10 +667,40 @@ class BackendZarafa implements IBackend, ISearchProvider {
         if(!strpos($attname, ":"))
             throw new StatusException(sprintf("ZarafaBackend->GetAttachmentData('%s'): Error, attachment requested for non-existing item", $attname), SYNC_ITEMOPERATIONSSTATUS_INVALIDATT);
 
-        list($id, $attachnum) = explode(":", $attname);
+        // backwards compatibility if the storeentryid is not available in the $attname
+        if (substr_count($attname, ':') == 1) {
+            list($id, $attachnum) = explode(":", $attname);
+        }
+        else {
+            list($storeEntryid, $id, $attachnum) = explode(":", $attname);
+        }
 
+        // Assume that the attachment is in the user's store.
+        $store = $this->store;
         $entryid = hex2bin($id);
-        $message = mapi_msgstore_openentry($this->store, $entryid);
+        if (isset($storeEntryid)) {
+            $storeEntryid = hex2bin($storeEntryid);
+            $storeProps = $this->getStoreProps();
+            // If the attachment is not in the user's store, check if public store is available and it's there.
+            // Otherwise try to open the store.
+            if ($storeProps[PR_ENTRYID] != $storeEntryid) {
+                if (isset($this->storeCache['SYSTEM']) && $this->storeCache['SYSTEM'] == $storeEntryid) {
+                    ZLog::Write(LOGLEVEL_DEBUG, 'ZarafaBackend->GetAttachmentData the attachment is in a message in a public store.');
+                    $store = $this->storeCache['SYSTEM'];
+                }
+                else {
+                    ZLog::Write(LOGLEVEL_DEBUG, sprintf("Open store %s", bin2hex($storeEntryid)));
+                    $store = @mapi_openmsgstore($this->session, $storeEntryid);
+
+                    if (!$store) {
+                        ZLog::Write(LOGLEVEL_WARN, sprintf("ZarafaBackend->GetAttachmentData: Could not open store %s", bin2hex($storeEntryid)));
+                        return false;
+                    }
+                }
+            }
+        }
+
+        $message = mapi_msgstore_openentry($store, $entryid);
         if(!$message)
             throw new StatusException(sprintf("ZarafaBackend->GetAttachmentData('%s'): Error, unable to open item for attachment data for id '%s' with: 0x%X", $attname, $id, mapi_last_hresult()), SYNC_ITEMOPERATIONSSTATUS_INVALIDATT);
 
@@ -2091,6 +2123,20 @@ class BackendZarafa implements IBackend, ISearchProvider {
         }
         return true;
     }
+
+    /**
+     * Gets the required store properties.
+     *
+     * @access private
+     * @return array
+     */
+    private function getStoreProps() {
+        if (empty($this->storeProps)) {
+            ZLog::Write(LOGLEVEL_DEBUG, "BackendZarafa->getStoreProps(): Getting store properties.");
+            $this->storeProps = mapi_getprops($this->store, array(PR_ENTRYID));
+        }
+        return $this->storeProps;
+    }
 }
 
 /**