tweak(Felamimail/AttachmentCache): improve 404 handling

... convert Felamimail_Exception_IMAPMessageNotFound to Tinebase_Exception_NotFound
tine20 · Feb 13, 2023 · c044bf7 · c044bf7
1 parent 20c9adb
commit c044bf7
Showing 1 changed file with 14 additions and 6 deletions.
diff --git a/tine20/Felamimail/Controller/AttachmentCache.php b/tine20/Felamimail/Controller/AttachmentCache.php
@@ -123,10 +123,14 @@ public function get($_id, $_containerId = null, $_getRelatedData = true, $_getDe
     }
 
     /**
-     * inspect creation of one record (before create)
-     *
-     * @param   Felamimail_Model_AttachmentCache $_record
-     * @return  void
+     * @param Felamimail_Model_AttachmentCache $_record
+     * @return void
+     * @throws Tinebase_Exception_InvalidArgument
+     * @throws Tinebase_Exception_NotFound
+     * @throws Tinebase_Exception_Record_DefinitionFailure
+     * @throws Tinebase_Exception_Record_Validation
+     * @throws Tinebase_Exception_SystemGeneric
+     * @throws Zend_Mime_Exception
      */
     protected function _inspectBeforeCreate(Tinebase_Record_Interface $_record)
     {
@@ -153,8 +157,12 @@ protected function _inspectBeforeCreate(Tinebase_Record_Interface $_record)
             $fileName = $attachment['filename'];
 
         } else {
-            $msgPart = $ctrl->getMessagePart($_record->{Felamimail_Model_AttachmentCache::FLD_SOURCE_ID},
-                $_record->{Felamimail_Model_AttachmentCache::FLD_PART_ID});
+            try {
+                $msgPart = $ctrl->getMessagePart($_record->{Felamimail_Model_AttachmentCache::FLD_SOURCE_ID},
+                    $_record->{Felamimail_Model_AttachmentCache::FLD_PART_ID});
+            } catch (Felamimail_Exception_IMAPMessageNotFound $feiamnf) {
+                throw new Tinebase_Exception_NotFound($feiamnf->getMessage());
+            }
             $stream = $msgPart->getDecodedStream();
             $fileName = $msgPart->filename;
         }