holm/egroupware
1
1
Fork 0
forked from extern/egroupware
Code Issues Pull Requests Packages Projects Releases Wiki Activity

WIP mail inline image

Browse Source 
- Define static methods for resolving inline images from CID, based on types
- Fix not showing inline images on reply
This commit is contained in:
Hadi Nategh 2015-05-15 14:00:37 +00:00
parent a4da02db69
commit 73499db1c7
2 changed files with 123 additions and 139 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
mail/inc/class.mail_compose.inc.php
View File

@ -2081,6 +2081,7 @@ class mail_compose
} }
$this->sessionData['body'] .= '</blockquote><br>'; $this->sessionData['body'] .= '</blockquote><br>';
$this->sessionData['body'] = mail_ui::resolve_inline_images($this->sessionData['body'], $_folder, $_uid, $_partID, 'html');
} else { } else {
//$this->sessionData['body'] = @htmlspecialchars(lang("on")." ".$headers['DATE']." ".$mail_bo->decode_header($fromAddress), ENT_QUOTES) . " ".lang("wrote").":\r\n"; //$this->sessionData['body'] = @htmlspecialchars(lang("on")." ".$headers['DATE']." ".$mail_bo->decode_header($fromAddress), ENT_QUOTES) . " ".lang("wrote").":\r\n";
// take care the way the ReplyHeader is created here, is used later on in uicompose::compose, in case you force replys to be HTML (prefs) // take care the way the ReplyHeader is created here, is used later on in uicompose::compose, in case you force replys to be HTML (prefs)
@ -2100,7 +2101,7 @@ class mail_compose
if ($bodyParts[$i]['charSet']===false) $bodyParts[$i]['charSet'] = mail_bo::detect_encoding($bodyParts[$i]['body']); if ($bodyParts[$i]['charSet']===false) $bodyParts[$i]['charSet'] = mail_bo::detect_encoding($bodyParts[$i]['body']);
$newBody = translation::convert($bodyParts[$i]['body'], $bodyParts[$i]['charSet']); $newBody = translation::convert($bodyParts[$i]['body'], $bodyParts[$i]['charSet']);
#error_log( "GetReplyData (Plain) CharSet:".mb_detect_encoding($bodyParts[$i]['body'] . 'a' , strtoupper($bodyParts[$i]['charSet']).','.strtoupper($this->displayCharset).',UTF-8, ISO-8859-1')); #error_log( "GetReplyData (Plain) CharSet:".mb_detect_encoding($bodyParts[$i]['body'] . 'a' , strtoupper($bodyParts[$i]['charSet']).','.strtoupper($this->displayCharset).',UTF-8, ISO-8859-1'));
$newBody = mail_ui::resolve_inline_images($newBody, $_folder, $_uid, $_partID, 'plain');
$this->sessionData['body'] .= "\r\n"; $this->sessionData['body'] .= "\r\n";
// create body new, with good line breaks and indention // create body new, with good line breaks and indention
foreach(explode("\n",$newBody) as $value) { foreach(explode("\n",$newBody) as $value) {

259
mail/inc/class.mail_ui.inc.php
View File

@ -3013,7 +3013,7 @@ class mail_ui
// create links for inline images // create links for inline images
if ($modifyURI) if ($modifyURI)
{ {
$newBody = preg_replace_callback("/\[cid:(.*)\]/iU",array($this,'image_callback_plain'),$newBody); $newBody = self::resolve_inline_images($newBody, $this->mailbox, $this->uid, $this->partID, 'plain');
} }
//TODO:$newBody = $this->highlightQuotes($newBody); //TODO:$newBody = $this->highlightQuotes($newBody);
@ -3064,9 +3064,7 @@ class mail_ui
// create links for inline images // create links for inline images
if ($modifyURI) if ($modifyURI)
{ {
$newBody = preg_replace_callback("/src=(\"|\')cid:(.*)(\"|\')/iU",array($this,'image_callback'),$newBody); $newBody = self::resolve_inline_images ($newBody, $this->mailbox, $this->uid, $this->partID);
$newBody = preg_replace_callback("/url\(cid:(.*)\);/iU",array($this,'image_callback_url'),$newBody);
$newBody = preg_replace_callback("/background=(\"|\')cid:(.*)(\"|\')/iU",array($this,'image_callback_background'),$newBody);
} }
// email addresses / mailto links get now activated on client-side // email addresses / mailto links get now activated on client-side
} }
@ -3082,161 +3080,146 @@ class mail_ui
return $body; return $body;
} }
/** /**
* preg_replace callback to replace image cid url's * Resolve inline images from CID to proper url
* *
* @param array $matches matches from preg_replace("/src=(\"|\')cid:(.*)(\"|\')/iU",...) * @param string $_body message content
* @return string src attribute to replace * @param string $_mailbox mail folder
* @param string $_uid uid
* @param string $_partID part id
* @param string $_messageType = 'html', message type is either html or plain
* @return string message body including all CID images replaced
*/ */
function image_callback($matches) public static function resolve_inline_images ($_body,$_mailbox, $_uid, $_partID, $_messageType = 'html')
{ {
static $cache = array(); // some caching, if mails containing the same image multiple times if ($_messageType === 'plain')
$linkData = array (
'menuaction' => 'mail.mail_ui.displayImage',
'uid' => $this->uid,
'mailbox' => base64_encode($this->mailbox),
'cid' => base64_encode($matches[2]),
'partID' => $this->partID,
);
$imageURL = egw::link('/index.php', $linkData);
// to test without data uris, comment the if close incl. it's body
if (html::$user_agent != 'msie' || html::$ua_version >= 8)
{ {
if (!isset($cache[$imageURL])) return self::resolve_inline_image_byType($_body, $_mailbox, $_uid, $_partID, 'plain');
{ }
$attachment = $this->mail_bo->getAttachmentByCID($this->uid, $matches[2], $this->partID); else
{
// only use data uri for "smaller" images, as otherwise the first display of the mail takes to long foreach(array('src','url','background') as $type)
if (($attachment instanceof Horde_Mime_Part) && $attachment->getBytes() < 8192) // msie=8 allows max 32k data uris {
{ $_body = self::resolve_inline_image_byType($_body, $_mailbox, $_uid, $_partID, $type);
$this->mail_bo->fetchPartContents($this->uid, $attachment); }
$cache[$imageURL] = 'data:'.$attachment->getType().';base64,'.base64_encode($attachment->getContents()); return $_body;
}
else
{
$cache[$imageURL] = $imageURL;
}
}
$imageURL = $cache[$imageURL];
} }
return 'src="'.$imageURL.'"';
} }
/** /**
* preg_replace callback to replace image cid url's * Replace CID with proper type of content understandable by browser
* *
* @param array $matches matches from preg_replace("/src=(\"|\')cid:(.*)(\"|\')/iU",...) * @param type $_body content of message
* @return string src attribute to replace * @param type $_mailbox mail box
* @param type $_uid uid
* @param type $_partID part id
* @param type $_type = 'src' type of inline image that needs to be resolved and replaced
* - types: {plain|src|url|background}
* @return string returns body content including all CID replacements
*/ */
function image_callback_plain($matches) public static function resolve_inline_image_byType ($_body,$_mailbox, $_uid, $_partID, $_type ='src')
{ {
static $cache = array(); // some caching, if mails containing the same image multiple times /**
//error_log(__METHOD__.__LINE__.array2string($matches)); * Callback for preg_replace_callback function
$linkData = array ( * returns matched CID replacement string based on given type
'menuaction' => 'mail.mail_ui.displayImage', * @param array $matches
'uid' => $this->uid, * @param string $_mailbox
'mailbox' => base64_encode($this->mailbox), * @param string $_uid
'cid' => base64_encode($matches[1]), * @param string $_partID
'partID' => $this->partID, * @param string $_type
); * @return string|boolean returns the replace
$imageURL = egw::link('/index.php', $linkData); */
$replace_callback = function ($matches) use ($_mailbox,$_uid, $_partID, $_type)
// to test without data uris, comment the if close incl. it's body
if (html::$user_agent != 'msie' || html::$ua_version >= 8)
{ {
if (!isset($cache[$imageURL])) if (!$_type) return false;
$CID = '';
// Build up matches according to selected type
switch ($_type)
{ {
$attachment = $this->mail_bo->getAttachmentByCID($this->uid, $matches[1], $this->partID); case "plain":
$CID = $matches[1];
break;
case "src":
$CID = $matches[2];
break;
case "url":
$CID = $matches[1];
break;
case "background":
$CID = $matches[2];
break;
}
// only use data uri for "smaller" images, as otherwise the first display of the mail takes to long static $cache = array(); // some caching, if mails containing the same image multiple times
if (($attachment instanceof Horde_Mime_Part) && bytes($attachment->getBytes()) < 8192) // msie=8 allows max 32k data uris
if (is_array($matches) && $CID)
{
$linkData = array (
'menuaction' => 'mail.mail_ui.displayImage',
'uid' => $_uid,
'mailbox' => base64_encode($_mailbox),
'cid' => base64_encode($CID),
'partID' => $_partID,
);
$imageURL = egw::link('/index.php', $linkData);
// to test without data uris, comment the if close incl. it's body
if (html::$user_agent != 'msie' || html::$ua_version >= 8)
{ {
$this->mail_bo->fetchPartContents($this->uid, $attachment); if (!isset($cache[$imageURL]))
$cache[$imageURL] = 'data:'.$attachment->getType().';base64,'.base64_encode($attachment->getContents()); {
if ($_type !="background")
{
$bo = emailadmin_imapbase::getInstance(false, self::$icServerID);
$attachment = $bo->getAttachmentByCID($_uid, $CID, $_partID);
// only use data uri for "smaller" images, as otherwise the first display of the mail takes to long
if (($attachment instanceof Horde_Mime_Part) && $attachment->getBytes() < 8192) // msie=8 allows max 32k data uris
{
$bo->fetchPartContents($_uid, $attachment);
$cache[$imageURL] = 'data:'.$attachment->getType().';base64,'.base64_encode($attachment->getContents());
}
else
{
$cache[$imageURL] = $imageURL;
}
}
else
{
$cache[$imageURL] = $imageURL;
}
}
$imageURL = $cache[$imageURL];
} }
else
// Decides the final result of replacement according to the type
switch ($_type)
{ {
$cache[$imageURL] = $imageURL; case "plain":
return '<img src="'.$imageURL.'" />';
case "src":
return 'src="'.$imageURL.'"';
case "url":
return 'url('.$imageURL.');';
case "background":
return 'background="'.$imageURL.'"';
} }
} }
$imageURL = $cache[$imageURL]; return false;
} };
return '<img src="'.$imageURL.'" />';
} // return new body content base on chosen type
switch($_type)
/**
* preg_replace callback to replace image cid url's
*
* @param array $matches matches from preg_replace("/src=(\"|\')cid:(.*)(\"|\')/iU",...)
* @return string src attribute to replace
*/
function image_callback_url($matches)
{
static $cache = array(); // some caching, if mails containing the same image multiple times
//error_log(__METHOD__.__LINE__.array2string($matches));
$linkData = array (
'menuaction' => 'mail.mail_ui.displayImage',
'uid' => $this->uid,
'mailbox' => base64_encode($this->mailbox),
'cid' => base64_encode($matches[1]),
'partID' => $this->partID,
);
$imageURL = egw::link('/index.php', $linkData);
// to test without data uris, comment the if close incl. it's body
if (html::$user_agent != 'msie' || html::$ua_version >= 8)
{ {
if (!isset($cache[$imageURL])) case"plain":
{ return preg_replace_callback("/\[cid:(.*)\]/iU",$replace_callback,$_body);
$attachment = $this->mail_bo->getAttachmentByCID($this->uid, $matches[1], $this->partID); case "src":
return preg_replace_callback("/src=(\"|\')cid:(.*)(\"|\')/iU",$replace_callback,$_body);
// only use data uri for "smaller" images, as otherwise the first display of the mail takes to long case "url":
if (($attachment instanceof Horde_Mime_Part) && $attachment->getBytes() < 8192) // msie=8 allows max 32k data uris return preg_replace_callback("/url\(cid:(.*)\);/iU",$replace_callback,$_body);
{ case "background":
$this->mail_bo->fetchPartContents($this->uid, $attachment); return preg_replace_callback("/background=(\"|\')cid:(.*)(\"|\')/iU",$replace_callback,$_body);
$cache[$imageURL] = 'data:'.$attachment->getType().';base64,'.base64_encode($attachment->getContents());
}
else
{
$cache[$imageURL] = $imageURL;
}
}
$imageURL = $cache[$imageURL];
} }
return 'url('.$imageURL.');';
}
/**
* preg_replace callback to replace image cid url's
*
* @param array $matches matches from preg_replace("/src=(\"|\')cid:(.*)(\"|\')/iU",...)
* @return string src attribute to replace
*/
function image_callback_background($matches)
{
static $cache = array(); // some caching, if mails containing the same image multiple times
$linkData = array (
'menuaction' => 'mail.mail_ui.displayImage',
'uid' => $this->uid,
'mailbox' => base64_encode($this->mailbox),
'cid' => base64_encode($matches[2]),
'partID' => $this->partID,
);
$imageURL = egw::link('/index.php', $linkData);
// to test without data uris, comment the if close incl. it's body
if (html::$user_agent != 'msie' || html::$ua_version >= 8)
{
if (!isset($cache[$imageURL]))
{
$cache[$imageURL] = $imageURL;
}
$imageURL = $cache[$imageURL];
}
return 'background="'.$imageURL.'"';
} }
/** /**