[Vtigercrm-commits] [Vtiger development] #7868: MailManager IMAP Connector fixes

[Vtiger development]

#7868: MailManager IMAP Connector fixes
-----------------------+-------------------------
 Reporter:  dainius    |      Owner:  developer
     Type:  defect     |     Status:  new
 Priority:  minor      |  Milestone:  Unassigned
Component:  vtigercrm  |    Version:  6.0.0 RC
 Severity:  Low        |   Keywords:  mailmanager
-----------------------+-------------------------
 Unable to delete multiple selected messages in MailManager.
 _msgno in ValueMap is populated with values separated with `%2C` instead
 of comma and 'explode' does not split values properly in
 `\modules\MailManager\connectors\Connector.php`

 function deleteMail($msgno){
         $msgno = trim($msgno,',');
         '''$msgno = explode(',',$msgno);'''
         .....



 ----


 '''vtiger6''' is hardcoded, but it does not exist in installation. Should
 be without '''vtiger6''' or not used at all - loaded in next line by
 autoload.

 line 203: `import('~~vtiger6/modules/MailManager/models/Folder.php');`
 ----

 @ is missing (not important, but would be nice to keep IMAP calls the same
 way in a class)

 line 283: `imap_expunge($this->mBox);`
 should be `@imap_expunge($this->mBox);`

 ----

--
Ticket URL: <http://trac.vtiger.com/cgi-bin/trac.cgi/ticket/7868>
Vtiger development <http://trac.vtiger.com/>
Vtiger CRM