]> git.mxchange.org Git - friendica.git/blobdiff - mod/message.php
Update functions and calls
[friendica.git] / mod / message.php
index d3bd1558cc8d5f6116755677be3e0a54250e7180..04073139f3cabe88f40232d8776ed8d48e8b032e 100644 (file)
 <?php
-
-require_once('include/acl_selectors.php');
-require_once('include/message.php');
-require_once('include/Smilies.php');
-
-function message_init(App $a) {
-
+/**
+ * @file mod/message.php
+ */
+use Friendica\App;
+use Friendica\Content\Smilies;
+use Friendica\Core\System;
+use Friendica\Database\DBM;
+use Friendica\Model\Contact;
+use Friendica\Model\Mail;
+
+require_once 'include/acl_selectors.php';
+require_once 'include/conversation.php';
+
+function message_init(App $a)
+{
        $tabs = '';
 
-       if ($a->argc >1 && is_numeric($a->argv[1])) {
-               $tabs = render_messages(get_messages(local_user(),0,5), 'mail_list.tpl');
+       if ($a->argc > 1 && is_numeric($a->argv[1])) {
+               $tabs = render_messages(get_messages(local_user(), 0, 5), 'mail_list.tpl');
        }
 
        $new = array(
                'label' => t('New Message'),
                'url' => 'message/new',
-               'sel'=> ($a->argv[1] == 'new'),
+               'sel' => $a->argc > 1 && $a->argv[1] == 'new',
                'accesskey' => 'm',
        );
 
        $tpl = get_markup_template('message_side.tpl');
        $a->page['aside'] = replace_macros($tpl, array(
-               '$tabs'=>$tabs,
-               '$new'=>$new,
+               '$tabs' => $tabs,
+               '$new' => $new,
        ));
-       $base = App::get_baseurl();
+       $base = System::baseUrl();
 
        $head_tpl = get_markup_template('message-head.tpl');
-       $a->page['htmlhead'] .= replace_macros($head_tpl,array(
-               '$baseurl' => App::get_baseurl(true),
+       $a->page['htmlhead'] .= replace_macros($head_tpl, array(
+               '$baseurl' => System::baseUrl(true),
                '$base' => $base
        ));
 
        $end_tpl = get_markup_template('message-end.tpl');
-       $a->page['end'] .= replace_macros($end_tpl,array(
-               '$baseurl' => App::get_baseurl(true),
+       $a->page['end'] .= replace_macros($end_tpl, array(
+               '$baseurl' => System::baseUrl(true),
                '$base' => $base
        ));
-
 }
 
-function message_post(App $a) {
-
-       if (! local_user()) {
-               notice( t('Permission denied.') . EOL);
+function message_post(App $a)
+{
+       if (!local_user()) {
+               notice(t('Permission denied.') . EOL);
                return;
        }
 
-       $replyto   = ((x($_REQUEST,'replyto'))   ? notags(trim($_REQUEST['replyto']))   : '');
-       $subject   = ((x($_REQUEST,'subject'))   ? notags(trim($_REQUEST['subject']))   : '');
-       $body      = ((x($_REQUEST,'body'))      ? escape_tags(trim($_REQUEST['body'])) : '');
-       $recipient = ((x($_REQUEST,'messageto')) ? intval($_REQUEST['messageto'])       : 0 );
+       $replyto   = x($_REQUEST, 'replyto')   ? notags(trim($_REQUEST['replyto']))   : '';
+       $subject   = x($_REQUEST, 'subject')   ? notags(trim($_REQUEST['subject']))   : '';
+       $body      = x($_REQUEST, 'body')      ? escape_tags(trim($_REQUEST['body'])) : '';
+       $recipient = x($_REQUEST, 'messageto') ? intval($_REQUEST['messageto'])       : 0;
 
-       $ret = send_message($recipient, $body, $subject, $replyto);
+       $ret = Mail::send($recipient, $body, $subject, $replyto);
        $norecip = false;
 
-       switch($ret){
+       switch ($ret) {
                case -1:
-                       notice( t('No recipient selected.') . EOL );
+                       notice(t('No recipient selected.') . EOL);
                        $norecip = true;
                        break;
                case -2:
-                       notice( t('Unable to locate contact information.') . EOL );
+                       notice(t('Unable to locate contact information.') . EOL);
                        break;
                case -3:
-                       notice( t('Message could not be sent.') . EOL );
+                       notice(t('Message could not be sent.') . EOL);
                        break;
                case -4:
-                       notice( t('Message collection failure.') . EOL );
+                       notice(t('Message collection failure.') . EOL);
                        break;
                default:
-                       info( t('Message sent.') . EOL );
+                       info(t('Message sent.') . EOL);
        }
 
        // fake it to go back to the input form if no recipient listed
-
-       if($norecip) {
+       if ($norecip) {
                $a->argc = 2;
                $a->argv[1] = 'new';
-       }
-       else
+       } else {
                goaway($_SESSION['return_url']);
-
-}
-
-// Note: the code in 'item_extract_images' and 'item_redir_and_replace_images'
-// is identical to the code in include/conversation.php
-if(! function_exists('item_extract_images')) {
-function item_extract_images($body) {
-
-       $saved_image = array();
-       $orig_body = $body;
-       $new_body = '';
-
-       $cnt = 0;
-       $img_start = strpos($orig_body, '[img');
-       $img_st_close = ($img_start !== false ? strpos(substr($orig_body, $img_start), ']') : false);
-       $img_end = ($img_start !== false ? strpos(substr($orig_body, $img_start), '[/img]') : false);
-       while(($img_st_close !== false) && ($img_end !== false)) {
-
-               $img_st_close++; // make it point to AFTER the closing bracket
-               $img_end += $img_start;
-
-               if(! strcmp(substr($orig_body, $img_start + $img_st_close, 5), 'data:')) {
-                       // This is an embedded image
-
-                       $saved_image[$cnt] = substr($orig_body, $img_start + $img_st_close, $img_end - ($img_start + $img_st_close));
-                       $new_body = $new_body . substr($orig_body, 0, $img_start) . '[!#saved_image' . $cnt . '#!]';
-
-                       $cnt++;
-               }
-               else
-                       $new_body = $new_body . substr($orig_body, 0, $img_end + strlen('[/img]'));
-
-               $orig_body = substr($orig_body, $img_end + strlen('[/img]'));
-
-               if($orig_body === false) // in case the body ends on a closing image tag
-                       $orig_body = '';
-
-               $img_start = strpos($orig_body, '[img');
-               $img_st_close = ($img_start !== false ? strpos(substr($orig_body, $img_start), ']') : false);
-               $img_end = ($img_start !== false ? strpos(substr($orig_body, $img_start), '[/img]') : false);
-       }
-
-       $new_body = $new_body . $orig_body;
-
-       return array('body' => $new_body, 'images' => $saved_image);
-}}
-
-if(! function_exists('item_redir_and_replace_images')) {
-function item_redir_and_replace_images($body, $images, $cid) {
-
-       $origbody = $body;
-       $newbody = '';
-
-       for($i = 0; $i < count($images); $i++) {
-               $search = '/\[url\=(.*?)\]\[!#saved_image' . $i . '#!\]\[\/url\]' . '/is';
-               $replace = '[url=' . z_path() . '/redir/' . $cid
-                          . '?f=1&url=' . '$1' . '][!#saved_image' . $i . '#!][/url]' ;
-
-               $img_end = strpos($origbody, '[!#saved_image' . $i . '#!][/url]') + strlen('[!#saved_image' . $i . '#!][/url]');
-               $process_part = substr($origbody, 0, $img_end);
-               $origbody = substr($origbody, $img_end);
-
-               $process_part = preg_replace($search, $replace, $process_part);
-               $newbody = $newbody . $process_part;
        }
-       $newbody = $newbody . $origbody;
-
-       $cnt = 0;
-       foreach ($images as $image) {
-               // We're depending on the property of 'foreach' (specified on the PHP website) that
-               // it loops over the array starting from the first element and going sequentially
-               // to the last element
-               $newbody = str_replace('[!#saved_image' . $cnt . '#!]', '[img]' . $image . '[/img]', $newbody);
-               $cnt++;
-       }
-
-       return $newbody;
-}}
-
-
-
-function message_content(App $a) {
+}
 
+function message_content(App $a)
+{
        $o = '';
        nav_set_selected('messages');
 
-       if (! local_user()) {
-               notice( t('Permission denied.') . EOL);
+       if (!local_user()) {
+               notice(t('Permission denied.') . EOL);
                return;
        }
 
-       $myprofile = App::get_baseurl().'/profile/' . $a->user['nickname'];
+       $myprofile = System::baseUrl() . '/profile/' . $a->user['nickname'];
 
        $tpl = get_markup_template('mail_head.tpl');
        $header = replace_macros($tpl, array(
                '$messages' => t('Messages'),
-               '$tab_content' => $tab_content
        ));
 
-
-       if(($a->argc == 3) && ($a->argv[1] === 'drop' || $a->argv[1] === 'dropconv')) {
-               if(! intval($a->argv[2]))
+       if (($a->argc == 3) && ($a->argv[1] === 'drop' || $a->argv[1] === 'dropconv')) {
+               if (!intval($a->argv[2])) {
                        return;
+               }
 
                // Check if we should do HTML-based delete confirmation
-               if($_REQUEST['confirm']) {
+               if ($_REQUEST['confirm']) {
                        // <form> can't take arguments in its "action" parameter
                        // so add any arguments as hidden inputs
                        $query = explode_querystring($a->query_string);
                        $inputs = array();
-                       foreach($query['args'] as $arg) {
-                               if(strpos($arg, 'confirm=') === false) {
+                       foreach ($query['args'] as $arg) {
+                               if (strpos($arg, 'confirm=') === false) {
                                        $arg_parts = explode('=', $arg);
                                        $inputs[] = array('name' => $arg_parts[0], 'value' => $arg_parts[1]);
                                }
@@ -210,28 +136,27 @@ function message_content(App $a) {
                        ));
                }
                // Now check how the user responded to the confirmation query
-               if($_REQUEST['canceled']) {
+               if ($_REQUEST['canceled']) {
                        goaway($_SESSION['return_url']);
                }
 
                $cmd = $a->argv[1];
-               if($cmd === 'drop') {
+               if ($cmd === 'drop') {
                        $r = q("DELETE FROM `mail` WHERE `id` = %d AND `uid` = %d LIMIT 1",
                                intval($a->argv[2]),
                                intval(local_user())
                        );
                        if ($r) {
-                               info( t('Message deleted.') . EOL );
+                               info(t('Message deleted.') . EOL);
                        }
-                       //goaway(App::get_baseurl(true) . '/message' );
+                       //goaway(System::baseUrl(true) . '/message' );
                        goaway($_SESSION['return_url']);
-               }
-               else {
+               } else {
                        $r = q("SELECT `parent-uri`,`convid` FROM `mail` WHERE `id` = %d AND `uid` = %d LIMIT 1",
                                intval($a->argv[2]),
                                intval(local_user())
                        );
-                       if (dbm::is_result($r)) {
+                       if (DBM::is_result($r)) {
                                $parent = $r[0]['parent-uri'];
                                $convid = $r[0]['convid'];
 
@@ -244,81 +169,78 @@ function message_content(App $a) {
                                // Actually if we do this, we can never receive another reply to that conversation,
                                // as we will never again have the info we need to re-create it.
                                // We'll just have to orphan it.
-
-                               //if($convid) {
+                               //if ($convid) {
                                //      q("delete from conv where id = %d limit 1",
                                //              intval($convid)
                                //      );
                                //}
 
-                               if($r)
-                                       info( t('Conversation removed.') . EOL );
+                               if ($r) {
+                                       info(t('Conversation removed.') . EOL);
+                               }
                        }
-                       //goaway(App::get_baseurl(true) . '/message' );
+                       //goaway(System::baseUrl(true) . '/message' );
                        goaway($_SESSION['return_url']);
                }
-
        }
 
-       if(($a->argc > 1) && ($a->argv[1] === 'new')) {
-
+       if (($a->argc > 1) && ($a->argv[1] === 'new')) {
                $o .= $header;
 
                $tpl = get_markup_template('msg-header.tpl');
                $a->page['htmlhead'] .= replace_macros($tpl, array(
-                       '$baseurl' => App::get_baseurl(true),
+                       '$baseurl' => System::baseUrl(true),
                        '$nickname' => $a->user['nickname'],
                        '$linkurl' => t('Please enter a link URL:')
                ));
 
                $tpl = get_markup_template('msg-end.tpl');
                $a->page['end'] .= replace_macros($tpl, array(
-                       '$baseurl' => App::get_baseurl(true),
+                       '$baseurl' => System::baseUrl(true),
                        '$nickname' => $a->user['nickname'],
                        '$linkurl' => t('Please enter a link URL:')
                ));
 
-               $preselect = (isset($a->argv[2])?array($a->argv[2]):false);
-
+               $preselect = isset($a->argv[2]) ? array($a->argv[2]) : false;
 
                $prename = $preurl = $preid = '';
 
-               if($preselect) {
+               if ($preselect) {
                        $r = q("SELECT `name`, `url`, `id` FROM `contact` WHERE `uid` = %d AND `id` = %d LIMIT 1",
                                intval(local_user()),
                                intval($a->argv[2])
                        );
-                       if (!dbm::is_result($r)) {
+                       if (!DBM::is_result($r)) {
                                $r = q("SELECT `name`, `url`, `id` FROM `contact` WHERE `uid` = %d AND `nurl` = '%s' LIMIT 1",
                                        intval(local_user()),
                                        dbesc(normalise_link(base64_decode($a->argv[2])))
                                );
                        }
 
-                       if (!dbm::is_result($r)) {
+                       if (!DBM::is_result($r)) {
                                $r = q("SELECT `name`, `url`, `id` FROM `contact` WHERE `uid` = %d AND `addr` = '%s' LIMIT 1",
                                        intval(local_user()),
                                        dbesc(base64_decode($a->argv[2]))
                                );
                        }
 
-                       if (dbm::is_result($r)) {
+                       if (DBM::is_result($r)) {
                                $prename = $r[0]['name'];
                                $preurl = $r[0]['url'];
                                $preid = $r[0]['id'];
                                $preselect = array($preid);
-                       } else
+                       } else {
                                $preselect = false;
+                       }
                }
 
-               $prefill = (($preselect) ? $prename  : '');
+               $prefill = $preselect ? $prename : '';
 
                // the ugly select box
-
-               $select = contact_select('messageto','message-to-select', $preselect, 4, true, false, false, 10);
+               $select = contact_select('messageto', 'message-to-select', $preselect, 4, true, false, false, 10);
 
                $tpl = get_markup_template('prv_message.tpl');
-               $o .= replace_macros($tpl,array(
+               $o .= replace_macros($tpl, array(
                        '$header' => t('Send Private Message'),
                        '$to' => t('To:'),
                        '$showinputs' => 'true',
@@ -326,8 +248,8 @@ function message_content(App $a) {
                        '$autocomp' => $autocomp,
                        '$preid' => $preid,
                        '$subject' => t('Subject:'),
-                       '$subjtxt' => ((x($_REQUEST,'subject')) ? strip_tags($_REQUEST['subject']) : ''),
-                       '$text' => ((x($_REQUEST,'body')) ? escape_tags(htmlspecialchars($_REQUEST['body'])) : ''),
+                       '$subjtxt' => x($_REQUEST, 'subject') ? strip_tags($_REQUEST['subject']) : '',
+                       '$text' => x($_REQUEST, 'body') ? escape_tags(htmlspecialchars($_REQUEST['body'])) : '',
                        '$readonly' => '',
                        '$yourmessage' => t('Your message:'),
                        '$select' => $select,
@@ -349,19 +271,19 @@ function message_content(App $a) {
 
                $o .= $header;
 
-               $r = q("SELECT count(*) AS `total` FROM `mail`, ANY_VALUE(`created`) AS `created`
+               $r = q("SELECT count(*) AS `total`, ANY_VALUE(`created`) AS `created` FROM `mail`
                        WHERE `mail`.`uid` = %d GROUP BY `parent-uri` ORDER BY `created` DESC",
                        intval(local_user())
                );
 
-               if (dbm::is_result($r)) {
+               if (DBM::is_result($r)) {
                        $a->set_pager_total($r[0]['total']);
                }
 
                $r = get_messages(local_user(), $a->pager['start'], $a->pager['itemspage']);
 
-               if (! dbm::is_result($r)) {
-                       info( t('No messages.') . EOL);
+               if (!DBM::is_result($r)) {
+                       info(t('No messages.') . EOL);
                        return $o;
                }
 
@@ -372,7 +294,7 @@ function message_content(App $a) {
                return $o;
        }
 
-       if(($a->argc > 1) && (intval($a->argv[1]))) {
+       if (($a->argc > 1) && (intval($a->argv[1]))) {
 
                $o .= $header;
 
@@ -382,12 +304,12 @@ function message_content(App $a) {
                        intval(local_user()),
                        intval($a->argv[1])
                );
-               if (dbm::is_result($r)) {
+               if (DBM::is_result($r)) {
                        $contact_id = $r[0]['contact-id'];
                        $convid = $r[0]['convid'];
 
                        $sql_extra = sprintf(" and `mail`.`parent-uri` = '%s' ", dbesc($r[0]['parent-uri']));
-                       if($convid)
+                       if ($convid)
                                $sql_extra = sprintf(" and ( `mail`.`parent-uri` = '%s' OR `mail`.`convid` = '%d' ) ",
                                        dbesc($r[0]['parent-uri']),
                                        intval($convid)
@@ -399,8 +321,8 @@ function message_content(App $a) {
                                intval(local_user())
                        );
                }
-               if(! count($messages)) {
-                       notice( t('Message not available.') . EOL );
+               if (!count($messages)) {
+                       notice(t('Message not available.') . EOL);
                        return $o;
                }
 
@@ -413,14 +335,14 @@ function message_content(App $a) {
 
                $tpl = get_markup_template('msg-header.tpl');
                $a->page['htmlhead'] .= replace_macros($tpl, array(
-                       '$baseurl' => App::get_baseurl(true),
+                       '$baseurl' => System::baseUrl(true),
                        '$nickname' => $a->user['nickname'],
                        '$linkurl' => t('Please enter a link URL:')
                ));
 
                $tpl = get_markup_template('msg-end.tpl');
                $a->page['end'] .= replace_macros($tpl, array(
-                       '$baseurl' => App::get_baseurl(true),
+                       '$baseurl' => System::baseUrl(true),
                        '$nickname' => $a->user['nickname'],
                        '$linkurl' => t('Please enter a link URL:')
                ));
@@ -429,78 +351,64 @@ function message_content(App $a) {
                $seen = 0;
                $unknown = false;
 
-               foreach($messages as $message) {
-                       if($message['unknown'])
+               foreach ($messages as $message) {
+                       if ($message['unknown'])
                                $unknown = true;
-                       if($message['from-url'] == $myprofile) {
+                       if ($message['from-url'] == $myprofile) {
                                $from_url = $myprofile;
                                $sparkle = '';
                        } elseif ($message['contact-id'] != 0) {
-                               $from_url = 'redir/'.$message['contact-id'];
+                               $from_url = 'redir/' . $message['contact-id'];
                                $sparkle = ' sparkle';
                        } else {
-                               $from_url = $message['from-url']."?zrl=".urlencode($myprofile);
+                               $from_url = $message['from-url'] . "?zrl=" . urlencode($myprofile);
                                $sparkle = ' sparkle';
                        }
 
-
                        $extracted = item_extract_images($message['body']);
-                       if($extracted['images'])
+                       if ($extracted['images']) {
                                $message['body'] = item_redir_and_replace_images($extracted['body'], $extracted['images'], $message['contact-id']);
-
-                       if($a->theme['template_engine'] === 'internal') {
-                               $from_name_e = template_escape($message['from-name']);
-                               $subject_e = template_escape($message['title']);
-                               $body_e = template_escape(Smilies::replace(bbcode($message['body'])));
-                               $to_name_e = template_escape($message['name']);
-                       } else {
-                               $from_name_e = $message['from-name'];
-                               $subject_e = $message['title'];
-                               $body_e = Smilies::replace(bbcode($message['body']));
-                               $to_name_e = $message['name'];
                        }
 
-                       $contact = get_contact_details_by_url($message['from-url']);
-                       if (isset($contact["thumb"]))
+                       $from_name_e = $message['from-name'];
+                       $subject_e = $message['title'];
+                       $body_e = Smilies::replace(bbcode($message['body']));
+                       $to_name_e = $message['name'];
+
+                       $contact = Contact::getDetailsByURL($message['from-url']);
+                       if (isset($contact["thumb"])) {
                                $from_photo = $contact["thumb"];
-                       else
+                       } else {
                                $from_photo = $message['from-photo'];
+                       }
 
                        $mails[] = array(
                                'id' => $message['id'],
                                'from_name' => $from_name_e,
                                'from_url' => $from_url,
+                               'from_addr' => $contact['addr'],
                                'sparkle' => $sparkle,
                                'from_photo' => proxy_url($from_photo, false, PROXY_SIZE_THUMB),
                                'subject' => $subject_e,
                                'body' => $body_e,
                                'delete' => t('Delete message'),
                                'to_name' => $to_name_e,
-                               'date' => datetime_convert('UTC',date_default_timezone_get(),$message['created'],'D, d M Y - g:i A'),
-                                'ago' => relative_date($message['created']),
+                               'date' => datetime_convert('UTC', date_default_timezone_get(), $message['created'], 'D, d M Y - g:i A'),
+                               'ago' => relative_date($message['created']),
                        );
 
                        $seen = $message['seen'];
                }
 
-
                $select = $message['name'] . '<input type="hidden" name="messageto" value="' . $contact_id . '" />';
                $parent = '<input type="hidden" name="replyto" value="' . $message['parent-uri'] . '" />';
 
                $tpl = get_markup_template('mail_display.tpl');
-
-               if($a->theme['template_engine'] === 'internal') {
-                       $subjtxt_e = template_escape($message['title']);
-               }
-               else {
-                       $subjtxt_e = $message['title'];
-               }
-
                $o = replace_macros($tpl, array(
                        '$thread_id' => $a->argv[1],
                        '$thread_subject' => $message['title'],
                        '$thread_seen' => $seen,
-                       '$delete' =>  t('Delete conversation'),
+                       '$delete' => t('Delete conversation'),
                        '$canreply' => (($unknown) ? false : '1'),
                        '$unknown_text' => t("No secure communications available. You <strong>may</strong> be able to respond from the sender's profile page."),
                        '$mails' => $mails,
@@ -510,7 +418,7 @@ function message_content(App $a) {
                        '$to' => t('To:'),
                        '$showinputs' => '',
                        '$subject' => t('Subject:'),
-                       '$subjtxt' => $subjtxt_e,
+                       '$subjtxt' => $message['title'],
                        '$readonly' => ' readonly="readonly" style="background: #BBBBBB;" ',
                        '$yourmessage' => t('Your message:'),
                        '$text' => '',
@@ -520,78 +428,76 @@ function message_content(App $a) {
                        '$insert' => t('Insert web link'),
                        '$submit' => t('Submit'),
                        '$wait' => t('Please wait')
-
                ));
 
                return $o;
        }
 }
 
-function get_messages($user, $lstart, $lend) {
+function get_messages($user, $lstart, $lend)
+{
        //TODO: rewritte with a sub-query to get the first message of each private thread with certainty
        return q("SELECT max(`mail`.`created`) AS `mailcreated`, min(`mail`.`seen`) AS `mailseen`,
-               ANY_VALUE(`mail`.`id`), ANY_VALUE(`mail`.`uid`), ANY_VALUE(`mail`.`guid`), ANY_VALUE(`mail`.`from-name`),
-               ANY_VALUE(`mail`.`from-photo`), ANY_VALUE(`mail`.`from-url`), ANY_VALUE(`mail`.`contact-id`),
-               ANY_VALUE(`mail`.`convid`), ANY_VALUE(`mail`.`title`), ANY_VALUE(`mail`.`body`), ANY_VALUE(`mail`.`seen`),
-               ANY_VALUE(`mail`.`reply`), ANY_VALUE(`mail`.`replied`), ANY_VALUE(`mail`.`unknown`),
-               ANY_VALUE(`mail`.`uri`), ANY_VALUE(`mail`.`parent-uri`), ANY_VALUE(`mail`.`created`),
-               ANY_VALUE(`contact`.`name`), ANY_VALUE(`contact`.`url`), ANY_VALUE(`contact`.`thumb`), ANY_VALUE(`contact`.`network`),
-               count( * ) as count
+               ANY_VALUE(`mail`.`id`) AS `id`, ANY_VALUE(`mail`.`uid`) AS `uid`, ANY_VALUE(`mail`.`guid`) AS `guid`,
+               ANY_VALUE(`mail`.`from-name`) AS `from-name`, ANY_VALUE(`mail`.`from-photo`) AS `from-photo`,
+               ANY_VALUE(`mail`.`from-url`) AS `from-url`, ANY_VALUE(`mail`.`contact-id`) AS `contact-id`,
+               ANY_VALUE(`mail`.`convid`) AS `convid`, ANY_VALUE(`mail`.`title`) AS `title`, ANY_VALUE(`mail`.`body`) AS `body`,
+               ANY_VALUE(`mail`.`seen`) AS `seen`, ANY_VALUE(`mail`.`reply`) AS `reply`, ANY_VALUE(`mail`.`replied`) AS `replied`,
+               ANY_VALUE(`mail`.`unknown`) AS `unknown`, ANY_VALUE(`mail`.`uri`) AS `uri`,
+               `mail`.`parent-uri`,
+               ANY_VALUE(`mail`.`created`) AS `created`, ANY_VALUE(`contact`.`name`) AS `name`, ANY_VALUE(`contact`.`url`) AS `url`,
+               ANY_VALUE(`contact`.`thumb`) AS `thumb`, ANY_VALUE(`contact`.`network`) AS `network`,
+               count( * ) as `count`
                FROM `mail` LEFT JOIN `contact` ON `mail`.`contact-id` = `contact`.`id`
-               WHERE `mail`.`uid` = %d GROUP BY `parent-uri` ORDER BY `mailcreated` DESC  LIMIT %d , %d ",
+               WHERE `mail`.`uid` = %d GROUP BY `parent-uri` ORDER BY `mailcreated` DESC LIMIT %d , %d ",
                intval($user), intval($lstart), intval($lend)
        );
 }
 
-function render_messages(array $msg, $t) {
-
+function render_messages(array $msg, $t)
+{
        $a = get_app();
 
        $tpl = get_markup_template($t);
        $rslt = '';
 
-       $myprofile = App::get_baseurl().'/profile/' . $a->user['nickname'];
-
-       foreach($msg as $rr) {
+       $myprofile = System::baseUrl() . '/profile/' . $a->user['nickname'];
 
-               if($rr['unknown'])
-                       $participants = sprintf( t("Unknown sender - %s"),$rr['from-name']);
-               elseif (link_compare($rr['from-url'], $myprofile))
-                       $participants = sprintf( t("You and %s"), $rr['name']);
-               else
-                       $participants = sprintf(t("%s and You"), $rr['from-name']);
-
-               if($a->theme['template_engine'] === 'internal') {
-                       $subject_e = template_escape((($rr['mailseen']) ? $rr['title'] : '<strong>' . $rr['title'] . '</strong>'));
-                       $body_e = template_escape($rr['body']);
-                       $to_name_e = template_escape($rr['name']);
-               }
-               else {
-                       $subject_e = (($rr['mailseen']) ? $rr['title'] : '<strong>' . $rr['title'] . '</strong>');
-                       $body_e = $rr['body'];
-                       $to_name_e = $rr['name'];
+       foreach ($msg as $rr) {
+               if ($rr['unknown']) {
+                       $participants = t("Unknown sender - %s", $rr['from-name']);
+               } elseif (link_compare($rr['from-url'], $myprofile)) {
+                       $participants = t("You and %s", $rr['name']);
+               } else {
+                       $participants = t("%s and You", $rr['from-name']);
                }
 
-               $contact = get_contact_details_by_url($rr['url']);
-               if (isset($contact["thumb"]))
+               $subject_e = (($rr['mailseen']) ? $rr['title'] : '<strong>' . $rr['title'] . '</strong>');
+               $body_e = $rr['body'];
+               $to_name_e = $rr['name'];
+
+               $contact = Contact::getDetailsByURL($rr['url']);
+               if (isset($contact["thumb"])) {
                        $from_photo = $contact["thumb"];
-               else
+               } else {
                        $from_photo = (($rr['thumb']) ? $rr['thumb'] : $rr['from-photo']);
+               }
 
                $rslt .= replace_macros($tpl, array(
                        '$id' => $rr['id'],
                        '$from_name' => $participants,
                        '$from_url' => (($rr['network'] === NETWORK_DFRN) ? 'redir/' . $rr['contact-id'] : $rr['url']),
+                       '$from_addr' => $contact['addr'],
                        '$sparkle' => ' sparkle',
                        '$from_photo' => proxy_url($from_photo, false, PROXY_SIZE_THUMB),
                        '$subject' => $subject_e,
                        '$delete' => t('Delete conversation'),
                        '$body' => $body_e,
                        '$to_name' => $to_name_e,
-                       '$date' => datetime_convert('UTC',date_default_timezone_get(),$rr['mailcreated'], t('D, d M Y - g:i A')),
-                                                                                                                       '$ago' => relative_date($rr['mailcreated']),
+                       '$date' => datetime_convert('UTC', date_default_timezone_get(), $rr['mailcreated'], t('D, d M Y - g:i A')),
+                       '$ago' => relative_date($rr['mailcreated']),
                        '$seen' => $rr['mailseen'],
-                       '$count' => sprintf( tt('%d message', '%d messages', $rr['count']), $rr['count']),
+                       '$count' => tt('%d message', '%d messages', $rr['count']),
                ));
        }