office365 giving error

[deven-sitapara]

[Webklex\IMAP\Exceptions\GetMessagesFailedException]
Object of class Closure could not be converted to string

	$EmailClient = array (
		'host' => 'outlook.office365.com',
		'port' => 993,
		'encryption' => 'ssl',
		'validate_cert' => true,
		'username' => 'xxx',
		'password' => 'xxxxxxx',
		'protocol'      => 'imap'
	);

	 
	$oClient = new Client($EmailClient);
	$oClient->connect();

	/** @var \Webklex\IMAP\Folder $oFolder */
	if($oFolder = $oClient->getFolder("INBOX")){

		$LastEmailReadDateTime = '2018-10-05 13:49:56';
		$aMessage = $oFolder->messages(null)->since($LastEmailReadDateTime)->all()->get();


		echo "\n" . $oFolder->fullName;
		echo "\n aMessage " . count($aMessage);

	}

[Webklex]

Hi @devatsrs ,
please provide some additional information such as the used version and the complete error call stack.

Best reagrds

[deven-sitapara]

Hi @Webklex

I have tried 2 versions and found fix in both versions.

Version: "1.1.2"

here I dont get any error but $availableMessages doesnt return any emails.

On file
webklex/laravel-imap/src/IMAP/Folder.php

issue at following line

$availableMessages = imap_search($this->getClient()->getConnection(), $query, SE_UID, $charset);
Fix

//		"webklex/laravel-imap": "1.1.2" @version  1.1.2 fix
  	   if($this->getClient()->host == 'outlook.office365.com') {
                $availableMessages = imap_search($this->getClient()->getConnection(), $query, SE_UID);
            } else {
                $availableMessages = imap_search($this->getClient()->getConnection(), $query, SE_UID, $charset);
            }

And Fix for version 1.2.9

I get following error , I don't have error trace at the moment.

[Webklex\IMAP\Exceptions\GetMessagesFailedException]
Object of class Closure could not be converted to string

Replace following code on src/IMAP/Query/Query.php

  ```

  $available_messages->forPage($this->page, $this->limit)->each(function($msgno, $msglist) use(&$messages, $options) {
                $oMessage = new Message($msgno, $msglist, $this->getClient(), $this->getFetchOptions(), $this->getFetchBody(), $this->getFetchAttachment(), $this->getFetchFlags());
                switch ($options['message_key']){
                    case 'number':
                        $message_key = $oMessage->getMessageNo();
                        break;
                    case 'list':
                        $message_key = $msglist;
                        break;
                    default:
                        $message_key = $oMessage->getMessageId();
                        break;
                }
                $messages->put($message_key, $oMessage);
            });

 
With following code

            foreach($available_messages  as $msgno){

                $oMessage = new Message($msgno, $msglist, $this->getClient(), $this->getFetchOptions(), $this->getFetchBody(), $this->getFetchAttachment(), $this->getFetchFlags());
                switch ($options['message_key']){
                    case 'number':
                        $message_key = $oMessage->getMessageNo();
                        break;
                    case 'list':
                        $message_key = $msglist;
                        break;
                    default:
                        $message_key = $oMessage->getMessageId();
                        break;
                }
                $messages->put($message_key, $oMessage);
            }

[Webklex]

Hi @devatsrs,
please refer to #100 regarding your first problem.

if($this->getCharset() === null){
    $available_messages = imap_search($this->getClient()->getConnection(), $this->getRawQuery(), SE_UID);
}else{
    $available_messages = imap_search($this->getClient()->getConnection(), $this->getRawQuery(), SE_UID, $this->getCharset());
}

The second one is hard to tackle without any stack trace or a bit more information :)

Best regards

[deven-sitapara]

Thanks @Webklex 👍