Merge pull request #9 from iyzoer/fixes-undefined-index

check date is null

src/Fetch/Attachment.php

@@ -100,12 +100,14 @@ class Attachment
                     try {
                         $hObject = imap_rfc822_parse_headers($header);
 
-                        $subject = MIME::decode($hObject->subject, Message::$charset);
-                        $subject = preg_replace('#\s+#', ' ', $subject);
-                        $subject = preg_replace('#^(.{0,50})#u', '$1', $subject);
+                        if (property_exists($hObject, 'subject')) {
+                            $subject = MIME::decode($hObject->subject, Message::$charset);
+                            $subject = preg_replace('#\s+#', ' ', $subject);
+                            $subject = preg_replace('#^(.{0,50})#u', '$1', $subject);
 
-                        if ($subject) {
-                            $this->filename = $subject . '.eml';
+                            if ($subject) {
+                                $this->filename = $subject . '.eml';
+                            }
                         }
                     } catch (\Throwable $e) {
 
@@ -132,7 +134,9 @@ class Attachment
             $this->setFileName($parameters['name']);
         }
 
-        $this->size = $structure->bytes;
+        if (property_exists($structure, 'bytes')) {
+            $this->size = $structure->bytes;
+        }
 
         $this->mimeType = Message::typeIdToString($structure->type);
 

src/Fetch/Message.php

@@ -249,9 +249,15 @@ class Message
 
             return false;
 
-        $this->subject = MIME::decode($messageOverview->subject, self::$charset);
-        $this->date    = strtotime($messageOverview->date);
-        $this->size    = $messageOverview->size;
+        if (property_exists($messageOverview, 'subject')) {
+            $this->subject = MIME::decode($messageOverview->subject, self::$charset);
+        }
+
+        if (property_exists($messageOverview, 'date') && null !== $messageOverview->date) {
+            $this->date = strtotime($messageOverview->date);
+        }
+
+        $this->size = $messageOverview->size;
 
         foreach (self::$flagTypes as $flag)
             $this->status[$flag] = ($messageOverview->$flag == 1);
@@ -381,7 +387,7 @@ class Message
     public function getStructure($forceReload = false)
     {
         if ($forceReload || !isset($this->structure)) {
-            $this->structure = imap_fetchstructure($this->imapStream, $this->uid, FT_UID);
+            $this->structure = @imap_fetchstructure($this->imapStream, $this->uid, FT_UID);
         }
 
         return $this->structure;
@@ -532,6 +538,10 @@ class Message
      */
     protected function processStructure($structure, $partIdentifier = null)
     {
+        if (!$structure) {
+            return;
+        }
+
         $parameters = self::getParametersFromStructure($structure);
 
         // quick fix for Content-Disposition extended notation
@@ -754,7 +764,13 @@ class Message
             foreach ($addresses as $address) {
                 if (property_exists($address, 'mailbox') && $address->mailbox != 'undisclosed-recipients') {
                     $currentAddress = array();
-                    $currentAddress['address'] = $address->mailbox . '@' . $address->host;
+
+                    $host = '';
+                    if (property_exists($address, 'host')) {
+                        $host = $address->host;
+                    }
+
+                    $currentAddress['address'] = $address->mailbox . '@' . $host;
                     if (isset($address->personal)) {
                         $currentAddress['name'] = MIME::decode($address->personal, self::$charset);
                     }