X-Git-Url: https://git.mxchange.org/?a=blobdiff_plain;f=src%2FCore%2FNotificationsManager.php;h=8a1c1569ee3fe9ccaebaf294291f3607f4626304;hb=b8186ba98bfdd8a366d7ee2a491df040bb5a70c2;hp=936c1f7d237ccf975a76c063da2929228d13172a;hpb=4ca68c7af05b7b41e1b04936b9080802ca2b7716;p=friendica.git diff --git a/src/Core/NotificationsManager.php b/src/Core/NotificationsManager.php index 936c1f7d23..8a1c1569ee 100644 --- a/src/Core/NotificationsManager.php +++ b/src/Core/NotificationsManager.php @@ -1,31 +1,32 @@ a = get_app(); - } - +class NotificationsManager extends BaseObject +{ /** * @brief set some extra note properties * @@ -38,65 +39,69 @@ class NotificationsManager { * - msg_html: message as html string * - msg_plain: message as plain text string */ - private function _set_extra($notes) { - $rets = array(); - foreach($notes as $n) { - $local_time = datetime_convert('UTC',date_default_timezone_get(),$n['date']); + private function _set_extra($notes) + { + $rets = []; + foreach ($notes as $n) { + $local_time = DateTimeFormat::local($n['date']); $n['timestamp'] = strtotime($local_time); - $n['date_rel'] = relative_date($n['date']); - $n['msg_html'] = bbcode($n['msg'], false, false, false, false); - $n['msg_plain'] = explode("\n",trim(html2plain($n['msg_html'], 0)))[0]; + $n['date_rel'] = Temporal::getRelativeDate($n['date']); + $n['msg_html'] = BBCode::convert($n['msg'], false); + $n['msg_plain'] = explode("\n", trim(HTML::toPlaintext($n['msg_html'], 0)))[0]; $rets[] = $n; } return $rets; } - /** * @brief Get all notifications for local_user() * - * @param array $filter optional Array "column name"=>value: filter query by columns values - * @param string $order optional Space separated list of column to sort by. prepend name with "+" to sort ASC, "-" to sort DESC. Default to "-date" - * @param string $limit optional Query limits + * @param array $filter optional Array "column name"=>value: filter query by columns values + * @param string $order optional Space separated list of column to sort by. + * Prepend name with "+" to sort ASC, "-" to sort DESC. Default to "-date" + * @param string $limit optional Query limits * * @return array of results or false on errors */ - public function getAll($filter = array(), $order="-date", $limit="") { - $filter_str = array(); + public function getAll($filter = [], $order = "-date", $limit = "") + { + $filter_str = []; $filter_sql = ""; - foreach($filter as $column => $value) { + foreach ($filter as $column => $value) { $filter_str[] = sprintf("`%s` = '%s'", $column, dbesc($value)); } - if (count($filter_str)>0) { - $filter_sql = "AND ".implode(" AND ", $filter_str); + if (count($filter_str) > 0) { + $filter_sql = "AND " . implode(" AND ", $filter_str); } $aOrder = explode(" ", $order); - $asOrder = array(); - foreach($aOrder as $o) { + $asOrder = []; + foreach ($aOrder as $o) { $dir = "asc"; - if ($o[0]==="-") { + if ($o[0] === "-") { $dir = "desc"; - $o = substr($o,1); + $o = substr($o, 1); } - if ($o[0]==="+") { + if ($o[0] === "+") { $dir = "asc"; - $o = substr($o,1); + $o = substr($o, 1); } $asOrder[] = "$o $dir"; } $order_sql = implode(", ", $asOrder); - if($limit!="") - $limit = " LIMIT ".$limit; - - $r = q("SELECT * FROM `notify` WHERE `uid` = %d $filter_sql ORDER BY $order_sql $limit", - intval(local_user()) - ); + if ($limit != "") { + $limit = " LIMIT " . $limit; + } + $r = q( + "SELECT * FROM `notify` WHERE `uid` = %d $filter_sql ORDER BY $order_sql $limit", + intval(local_user()) + ); - if (DBM::is_result($r)) + if (DBM::is_result($r)) { return $this->_set_extra($r); + } return false; } @@ -104,11 +109,13 @@ class NotificationsManager { /** * @brief Get one note for local_user() by $id value * - * @param int $id + * @param int $id identity * @return array note values or null if not found */ - public function getByID($id) { - $r = q("SELECT * FROM `notify` WHERE `id` = %d AND `uid` = %d LIMIT 1", + public function getByID($id) + { + $r = q( + "SELECT * FROM `notify` WHERE `id` = %d AND `uid` = %d LIMIT 1", intval($id), intval(local_user()) ); @@ -121,12 +128,14 @@ class NotificationsManager { /** * @brief set seen state of $note of local_user() * - * @param array $note - * @param bool $seen optional true or false, default true + * @param array $note note array + * @param bool $seen optional true or false, default true * @return bool true on success, false on errors */ - public function setSeen($note, $seen = true) { - return q("UPDATE `notify` SET `seen` = %d WHERE ( `link` = '%s' OR ( `parent` != 0 AND `parent` = %d AND `otype` = '%s' )) AND `uid` = %d", + public function setSeen($note, $seen = true) + { + return q( + "UPDATE `notify` SET `seen` = %d WHERE (`link` = '%s' OR (`parent` != 0 AND `parent` = %d AND `otype` = '%s')) AND `uid` = %d", intval($seen), dbesc($note['link']), intval($note['parent']), @@ -141,8 +150,10 @@ class NotificationsManager { * @param bool $seen optional true or false. default true * @return bool true on success, false on error */ - public function setAllSeen($seen = true) { - return q("UPDATE `notify` SET `seen` = %d WHERE `uid` = %d", + public function setAllSeen($seen = true) + { + return q( + "UPDATE `notify` SET `seen` = %d WHERE `uid` = %d", intval($seen), intval(local_user()) ); @@ -153,44 +164,45 @@ class NotificationsManager { * * @return array with with notifications TabBar data */ - public function getTabs() { - $tabs = array( - array( - 'label' => t('System'), - 'url'=>'notifications/system', - 'sel'=> (($this->a->argv[1] == 'system') ? 'active' : ''), - 'id' => 'system-tab', + public function getTabs() + { + $tabs = [ + [ + 'label' => L10n::t('System'), + 'url' => 'notifications/system', + 'sel' => ((self::getApp()->argv[1] == 'system') ? 'active' : ''), + 'id' => 'system-tab', 'accesskey' => 'y', - ), - array( - 'label' => t('Network'), - 'url'=>'notifications/network', - 'sel'=> (($this->a->argv[1] == 'network') ? 'active' : ''), - 'id' => 'network-tab', + ], + [ + 'label' => L10n::t('Network'), + 'url' => 'notifications/network', + 'sel' => ((self::getApp()->argv[1] == 'network') ? 'active' : ''), + 'id' => 'network-tab', 'accesskey' => 'w', - ), - array( - 'label' => t('Personal'), - 'url'=>'notifications/personal', - 'sel'=> (($this->a->argv[1] == 'personal') ? 'active' : ''), - 'id' => 'personal-tab', + ], + [ + 'label' => L10n::t('Personal'), + 'url' => 'notifications/personal', + 'sel' => ((self::getApp()->argv[1] == 'personal') ? 'active' : ''), + 'id' => 'personal-tab', 'accesskey' => 'r', - ), - array( - 'label' => t('Home'), - 'url' => 'notifications/home', - 'sel'=> (($this->a->argv[1] == 'home') ? 'active' : ''), - 'id' => 'home-tab', + ], + [ + 'label' => L10n::t('Home'), + 'url' => 'notifications/home', + 'sel' => ((self::getApp()->argv[1] == 'home') ? 'active' : ''), + 'id' => 'home-tab', 'accesskey' => 'h', - ), - array( - 'label' => t('Introductions'), - 'url' => 'notifications/intros', - 'sel'=> (($this->a->argv[1] == 'intros') ? 'active' : ''), - 'id' => 'intro-tab', + ], + [ + 'label' => L10n::t('Introductions'), + 'url' => 'notifications/intros', + 'sel' => ((self::getApp()->argv[1] == 'intros') ? 'active' : ''), + 'id' => 'intro-tab', 'accesskey' => 'i', - ), - ); + ], + ]; return $tabs; } @@ -198,25 +210,24 @@ class NotificationsManager { /** * @brief Format the notification query in an usable array * - * @param array $notifs The array from the db query - * @param string $ident The notifications identifier (e.g. network) + * @param array $notifs The array from the db query + * @param string $ident The notifications identifier (e.g. network) * @return array - * string 'label' => The type of the notification - * string 'link' => URL to the source - * string 'image' => The avatar image - * string 'url' => The profile url of the contact - * string 'text' => The notification text - * string 'when' => The date of the notification - * string 'ago' => T relative date of the notification - * bool 'seen' => Is the notification marked as "seen" + * string 'label' => The type of the notification + * string 'link' => URL to the source + * string 'image' => The avatar image + * string 'url' => The profile url of the contact + * string 'text' => The notification text + * string 'when' => The date of the notification + * string 'ago' => T relative date of the notification + * bool 'seen' => Is the notification marked as "seen" */ - private function formatNotifs($notifs, $ident = "") { - - $notif = array(); - $arr = array(); + private function formatNotifs($notifs, $ident = "") + { + $notif = []; + $arr = []; if (DBM::is_result($notifs)) { - foreach ($notifs as $it) { // Because we use different db tables for the notification query // we have sometimes $it['unseen'] and sometimes $it['seen]. @@ -229,123 +240,122 @@ class NotificationsManager { switch ($ident) { case 'system': $default_item_label = 'notify'; - $default_item_link = System::baseUrl(true).'/notify/view/'. $it['id']; + $default_item_link = System::baseUrl(true) . '/notify/view/' . $it['id']; $default_item_image = proxy_url($it['photo'], false, PROXY_SIZE_MICRO); $default_item_url = $it['url']; - $default_item_text = strip_tags(bbcode($it['msg'])); - $default_item_when = datetime_convert('UTC', date_default_timezone_get(), $it['date'], 'r'); - $default_item_ago = relative_date($it['date']); + $default_item_text = strip_tags(BBCode::convert($it['msg'])); + $default_item_when = DateTimeFormat::local($it['date'], 'r'); + $default_item_ago = Temporal::getRelativeDate($it['date']); break; case 'home': $default_item_label = 'comment'; - $default_item_link = System::baseUrl(true).'/display/'.$it['pguid']; + $default_item_link = System::baseUrl(true) . '/display/' . $it['pguid']; $default_item_image = proxy_url($it['author-avatar'], false, PROXY_SIZE_MICRO); $default_item_url = $it['author-link']; - $default_item_text = sprintf(t("%s commented on %s's post"), $it['author-name'], $it['pname']); - $default_item_when = datetime_convert('UTC', date_default_timezone_get(), $it['created'], 'r'); - $default_item_ago = relative_date($it['created']); + $default_item_text = L10n::t("%s commented on %s's post", $it['author-name'], $it['pname']); + $default_item_when = DateTimeFormat::local($it['created'], 'r'); + $default_item_ago = Temporal::getRelativeDate($it['created']); break; default: $default_item_label = (($it['id'] == $it['parent']) ? 'post' : 'comment'); - $default_item_link = System::baseUrl(true).'/display/'.$it['pguid']; + $default_item_link = System::baseUrl(true) . '/display/' . $it['pguid']; $default_item_image = proxy_url($it['author-avatar'], false, PROXY_SIZE_MICRO); $default_item_url = $it['author-link']; $default_item_text = (($it['id'] == $it['parent']) - ? sprintf(t("%s created a new post"), $it['author-name']) - : sprintf(t("%s commented on %s's post"), $it['author-name'], $it['pname'])); - $default_item_when = datetime_convert('UTC', date_default_timezone_get(), $it['created'], 'r'); - $default_item_ago = relative_date($it['created']); - + ? L10n::t("%s created a new post", $it['author-name']) + : L10n::t("%s commented on %s's post", $it['author-name'], $it['pname'])); + $default_item_when = DateTimeFormat::local($it['created'], 'r'); + $default_item_ago = Temporal::getRelativeDate($it['created']); } // Transform the different types of notification in an usable array - switch ($it['verb']){ + switch ($it['verb']) { case ACTIVITY_LIKE: - $notif = array( + $notif = [ 'label' => 'like', - 'link' => System::baseUrl(true).'/display/'.$it['pguid'], + 'link' => System::baseUrl(true) . '/display/' . $it['pguid'], 'image' => proxy_url($it['author-avatar'], false, PROXY_SIZE_MICRO), 'url' => $it['author-link'], - 'text' => sprintf(t("%s liked %s's post"), $it['author-name'], $it['pname']), + 'text' => L10n::t("%s liked %s's post", $it['author-name'], $it['pname']), 'when' => $default_item_when, 'ago' => $default_item_ago, 'seen' => $it['seen'] - ); + ]; break; case ACTIVITY_DISLIKE: - $notif = array( + $notif = [ 'label' => 'dislike', - 'link' => System::baseUrl(true).'/display/'.$it['pguid'], + 'link' => System::baseUrl(true) . '/display/' . $it['pguid'], 'image' => proxy_url($it['author-avatar'], false, PROXY_SIZE_MICRO), 'url' => $it['author-link'], - 'text' => sprintf(t("%s disliked %s's post"), $it['author-name'], $it['pname']), + 'text' => L10n::t("%s disliked %s's post", $it['author-name'], $it['pname']), 'when' => $default_item_when, 'ago' => $default_item_ago, 'seen' => $it['seen'] - ); + ]; break; case ACTIVITY_ATTEND: - $notif = array( + $notif = [ 'label' => 'attend', - 'link' => System::baseUrl(true).'/display/'.$it['pguid'], + 'link' => System::baseUrl(true) . '/display/' . $it['pguid'], 'image' => proxy_url($it['author-avatar'], false, PROXY_SIZE_MICRO), 'url' => $it['author-link'], - 'text' => sprintf(t("%s is attending %s's event"), $it['author-name'], $it['pname']), + 'text' => L10n::t("%s is attending %s's event", $it['author-name'], $it['pname']), 'when' => $default_item_when, 'ago' => $default_item_ago, 'seen' => $it['seen'] - ); + ]; break; case ACTIVITY_ATTENDNO: - $notif = array( + $notif = [ 'label' => 'attendno', - 'link' => System::baseUrl(true).'/display/'.$it['pguid'], + 'link' => System::baseUrl(true) . '/display/' . $it['pguid'], 'image' => proxy_url($it['author-avatar'], false, PROXY_SIZE_MICRO), 'url' => $it['author-link'], - 'text' => sprintf( t("%s is not attending %s's event"), $it['author-name'], $it['pname']), + 'text' => L10n::t("%s is not attending %s's event", $it['author-name'], $it['pname']), 'when' => $default_item_when, 'ago' => $default_item_ago, 'seen' => $it['seen'] - ); + ]; break; case ACTIVITY_ATTENDMAYBE: - $notif = array( + $notif = [ 'label' => 'attendmaybe', - 'link' => System::baseUrl(true).'/display/'.$it['pguid'], + 'link' => System::baseUrl(true) . '/display/' . $it['pguid'], 'image' => proxy_url($it['author-avatar'], false, PROXY_SIZE_MICRO), 'url' => $it['author-link'], - 'text' => sprintf(t("%s may attend %s's event"), $it['author-name'], $it['pname']), + 'text' => L10n::t("%s may attend %s's event", $it['author-name'], $it['pname']), 'when' => $default_item_when, 'ago' => $default_item_ago, 'seen' => $it['seen'] - ); + ]; break; case ACTIVITY_FRIEND: - $xmlhead="<"."?xml version='1.0' encoding='UTF-8' ?".">"; - $obj = parse_xml_string($xmlhead.$it['object']); + $xmlhead = "<" . "?xml version='1.0' encoding='UTF-8' ?" . ">"; + $obj = XML::parseString($xmlhead . $it['object']); $it['fname'] = $obj->title; - $notif = array( + $notif = [ 'label' => 'friend', - 'link' => System::baseUrl(true).'/display/'.$it['pguid'], + 'link' => System::baseUrl(true) . '/display/' . $it['pguid'], 'image' => proxy_url($it['author-avatar'], false, PROXY_SIZE_MICRO), 'url' => $it['author-link'], - 'text' => sprintf(t("%s is now friends with %s"), $it['author-name'], $it['fname']), + 'text' => L10n::t("%s is now friends with %s", $it['author-name'], $it['fname']), 'when' => $default_item_when, 'ago' => $default_item_ago, 'seen' => $it['seen'] - ); + ]; break; default: - $notif = array( + $notif = [ 'label' => $default_item_label, 'link' => $default_item_link, 'image' => $default_item_image, @@ -354,7 +364,7 @@ class NotificationsManager { 'when' => $default_item_when, 'ago' => $default_item_ago, 'seen' => $it['seen'] - ); + ]; } $arr[] = $notif; @@ -362,32 +372,36 @@ class NotificationsManager { } return $arr; - } /** * @brief Total number of network notifications - * @param int|string $seen - * If 0 only include notifications into the query - * which aren't marked as "seen" + * @param int|string $seen If 0 only include notifications into the query + * which aren't marked as "seen" + * * @return int Number of network notifications */ - private function networkTotal($seen = 0) { + private function networkTotal($seen = 0) + { $sql_seen = ""; + $index_hint = ""; - if($seen === 0) - $sql_seen = " AND `item`.`unseen` = 1 "; + if ($seen === 0) { + $sql_seen = " AND `item`.`unseen` "; + $index_hint = "USE INDEX (`uid_unseen_contactid`)"; + } - $r = q("SELECT COUNT(*) AS `total` - FROM `item` INNER JOIN `item` AS `pitem` ON `pitem`.`id`=`item`.`parent` - WHERE `item`.`visible` = 1 AND `pitem`.`parent` != 0 AND - `item`.`deleted` = 0 AND `item`.`uid` = %d AND `item`.`wall` = 0 + $r = q( + "SELECT COUNT(*) AS `total` + FROM `item` $index_hint STRAIGHT_JOIN `item` AS `pitem` ON `pitem`.`id`=`item`.`parent` + WHERE `item`.`visible` AND `pitem`.`parent` != 0 AND + NOT `item`.`deleted` AND `item`.`uid` = %d AND NOT `item`.`wall` $sql_seen", intval(local_user()) ); - - if (DBM::is_result($r)) + if (DBM::is_result($r)) { return $r[0]['total']; + } return 0; } @@ -395,71 +409,77 @@ class NotificationsManager { /** * @brief Get network notifications * - * @param int|string $seen - * If 0 only include notifications into the query - * which aren't marked as "seen" - * @param int $start Start the query at this point - * @param int $limit Maximum number of query results + * @param int|string $seen If 0 only include notifications into the query + * which aren't marked as "seen" + * @param int $start Start the query at this point + * @param int $limit Maximum number of query results * * @return array with - * string 'ident' => Notification identifier - * int 'total' => Total number of available network notifications - * array 'notifications' => Network notifications + * string 'ident' => Notification identifier + * int 'total' => Total number of available network notifications + * array 'notifications' => Network notifications */ - public function networkNotifs($seen = 0, $start = 0, $limit = 80) { + public function networkNotifs($seen = 0, $start = 0, $limit = 80) + { $ident = 'network'; $total = $this->networkTotal($seen); - $notifs = array(); + $notifs = []; $sql_seen = ""; + $index_hint = ""; - if($seen === 0) - $sql_seen = " AND `item`.`unseen` = 1 "; - + if ($seen === 0) { + $sql_seen = " AND `item`.`unseen` "; + $index_hint = "USE INDEX (`uid_unseen_contactid`)"; + } - $r = q("SELECT `item`.`id`,`item`.`parent`, `item`.`verb`, `item`.`author-name`, `item`.`unseen`, + $r = q( + "SELECT `item`.`id`,`item`.`parent`, `item`.`verb`, `item`.`author-name`, `item`.`unseen`, `item`.`author-link`, `item`.`author-avatar`, `item`.`created`, `item`.`object` AS `object`, `pitem`.`author-name` AS `pname`, `pitem`.`author-link` AS `plink`, `pitem`.`guid` AS `pguid` - FROM `item` INNER JOIN `item` AS `pitem` ON `pitem`.`id`=`item`.`parent` - WHERE `item`.`visible` = 1 AND `pitem`.`parent` != 0 AND - `item`.`deleted` = 0 AND `item`.`uid` = %d AND `item`.`wall` = 0 + FROM `item` $index_hint STRAIGHT_JOIN `item` AS `pitem` ON `pitem`.`id`=`item`.`parent` + WHERE `item`.`visible` AND `pitem`.`parent` != 0 AND + NOT `item`.`deleted` AND `item`.`uid` = %d AND NOT `item`.`wall` $sql_seen ORDER BY `item`.`created` DESC LIMIT %d, %d ", - intval(local_user()), - intval($start), - intval($limit) + intval(local_user()), + intval($start), + intval($limit) ); - - if (DBM::is_result($r)) + if (DBM::is_result($r)) { $notifs = $this->formatNotifs($r, $ident); + } - $arr = array ( + $arr = [ 'notifications' => $notifs, 'ident' => $ident, 'total' => $total, - ); + ]; return $arr; } /** * @brief Total number of system notifications - * @param int|string $seen - * If 0 only include notifications into the query - * which aren't marked as "seen" + * @param int|string $seen If 0 only include notifications into the query + * which aren't marked as "seen" + * * @return int Number of system notifications */ - private function systemTotal($seen = 0) { + private function systemTotal($seen = 0) + { $sql_seen = ""; - if($seen === 0) - $sql_seen = " AND `seen` = 0 "; + if ($seen === 0) { + $sql_seen = " AND NOT `seen` "; + } - $r = q("SELECT COUNT(*) AS `total` FROM `notify` WHERE `uid` = %d $sql_seen", + $r = q( + "SELECT COUNT(*) AS `total` FROM `notify` WHERE `uid` = %d $sql_seen", intval(local_user()) ); - - if (DBM::is_result($r)) + if (DBM::is_result($r)) { return $r[0]['total']; + } return 0; } @@ -467,56 +487,60 @@ class NotificationsManager { /** * @brief Get system notifications * - * @param int|string $seen - * If 0 only include notifications into the query - * which aren't marked as "seen" - * @param int $start Start the query at this point - * @param int $limit Maximum number of query results + * @param int|string $seen If 0 only include notifications into the query + * which aren't marked as "seen" + * @param int $start Start the query at this point + * @param int $limit Maximum number of query results * * @return array with - * string 'ident' => Notification identifier - * int 'total' => Total number of available system notifications - * array 'notifications' => System notifications + * string 'ident' => Notification identifier + * int 'total' => Total number of available system notifications + * array 'notifications' => System notifications */ - public function systemNotifs($seen = 0, $start = 0, $limit = 80) { + public function systemNotifs($seen = 0, $start = 0, $limit = 80) + { $ident = 'system'; $total = $this->systemTotal($seen); - $notifs = array(); + $notifs = []; $sql_seen = ""; - if($seen === 0) - $sql_seen = " AND `seen` = 0 "; + if ($seen === 0) { + $sql_seen = " AND NOT `seen` "; + } - $r = q("SELECT `id`, `url`, `photo`, `msg`, `date`, `seen` FROM `notify` + $r = q( + "SELECT `id`, `url`, `photo`, `msg`, `date`, `seen` FROM `notify` WHERE `uid` = %d $sql_seen ORDER BY `date` DESC LIMIT %d, %d ", intval(local_user()), intval($start), intval($limit) ); - - if (DBM::is_result($r)) + if (DBM::is_result($r)) { $notifs = $this->formatNotifs($r, $ident); + } - $arr = array ( + $arr = [ 'notifications' => $notifs, 'ident' => $ident, 'total' => $total, - ); + ]; return $arr; } /** - * @brief Addional SQL query string for the personal notifications + * @brief Additional SQL query string for the personal notifications * - * @return string The additional sql query + * @return string The additional SQL query */ - private function _personal_sql_extra() { - $myurl = System::baseUrl(true) . '/profile/'. $this->a->user['nickname']; - $myurl = substr($myurl,strpos($myurl,'://')+3); - $myurl = str_replace(array('www.','.'),array('','\\.'),$myurl); - $diasp_url = str_replace('/profile/','/u/',$myurl); - $sql_extra = sprintf(" AND ( `item`.`author-link` regexp '%s' OR `item`.`tag` regexp '%s' OR `item`.`tag` regexp '%s' ) ", + private function personalSqlExtra() + { + $myurl = System::baseUrl(true) . '/profile/' . self::getApp()->user['nickname']; + $myurl = substr($myurl, strpos($myurl, '://') + 3); + $myurl = str_replace(['www.', '.'], ['', '\\.'], $myurl); + $diasp_url = str_replace('/profile/', '/u/', $myurl); + $sql_extra = sprintf( + " AND (`item`.`author-link` REGEXP '%s' OR `item`.`tag` REGEXP '%s' OR `item`.`tag` REGEXP '%s') ", dbesc($myurl . '$'), dbesc($myurl . '\\]'), dbesc($diasp_url . '\\]') @@ -527,29 +551,34 @@ class NotificationsManager { /** * @brief Total number of personal notifications - * @param int|string $seen - * If 0 only include notifications into the query - * which aren't marked as "seen" + * @param int|string $seen If 0 only include notifications into the query + * which aren't marked as "seen" + * * @return int Number of personal notifications */ - private function personalTotal($seen = 0) { + private function personalTotal($seen = 0) + { $sql_seen = ""; - $sql_extra = $this->_personal_sql_extra(); + $index_hint = ""; + $sql_extra = $this->personalSqlExtra(); - if($seen === 0) - $sql_seen = " AND `item`.`unseen` = 1 "; + if ($seen === 0) { + $sql_seen = " AND `item`.`unseen` "; + $index_hint = "USE INDEX (`uid_unseen_contactid`)"; + } - $r = q("SELECT COUNT(*) AS `total` - FROM `item` INNER JOIN `item` AS `pitem` ON `pitem`.`id`=`item`.`parent` - WHERE `item`.`visible` = 1 + $r = q( + "SELECT COUNT(*) AS `total` + FROM `item` $index_hint + WHERE `item`.`visible` $sql_extra $sql_seen - AND `item`.`deleted` = 0 AND `item`.`uid` = %d AND `item`.`wall` = 0 " , + AND NOT `item`.`deleted` AND `item`.`uid` = %d AND NOT `item`.`wall`", intval(local_user()) ); - - if (DBM::is_result($r)) + if (DBM::is_result($r)) { return $r[0]['total']; + } return 0; } @@ -557,75 +586,84 @@ class NotificationsManager { /** * @brief Get personal notifications * - * @param int|string $seen - * If 0 only include notifications into the query - * which aren't marked as "seen" - * @param int $start Start the query at this point - * @param int $limit Maximum number of query results + * @param int|string $seen If 0 only include notifications into the query + * which aren't marked as "seen" + * @param int $start Start the query at this point + * @param int $limit Maximum number of query results * * @return array with - * string 'ident' => Notification identifier - * int 'total' => Total number of available personal notifications - * array 'notifications' => Personal notifications + * string 'ident' => Notification identifier + * int 'total' => Total number of available personal notifications + * array 'notifications' => Personal notifications */ - public function personalNotifs($seen = 0, $start = 0, $limit = 80) { + public function personalNotifs($seen = 0, $start = 0, $limit = 80) + { $ident = 'personal'; $total = $this->personalTotal($seen); - $sql_extra = $this->_personal_sql_extra(); - $notifs = array(); + $sql_extra = $this->personalSqlExtra(); + $notifs = []; $sql_seen = ""; + $index_hint = ""; - if($seen === 0) - $sql_seen = " AND `item`.`unseen` = 1 "; + if ($seen === 0) { + $sql_seen = " AND `item`.`unseen` "; + $index_hint = "USE INDEX (`uid_unseen_contactid`)"; + } - $r = q("SELECT `item`.`id`,`item`.`parent`, `item`.`verb`, `item`.`author-name`, `item`.`unseen`, + $r = q( + "SELECT `item`.`id`,`item`.`parent`, `item`.`verb`, `item`.`author-name`, `item`.`unseen`, `item`.`author-link`, `item`.`author-avatar`, `item`.`created`, `item`.`object` AS `object`, `pitem`.`author-name` AS `pname`, `pitem`.`author-link` AS `plink`, `pitem`.`guid` AS `pguid` - FROM `item` INNER JOIN `item` AS `pitem` ON `pitem`.`id`=`item`.`parent` - WHERE `item`.`visible` = 1 + FROM `item` $index_hint STRAIGHT_JOIN `item` AS `pitem` ON `pitem`.`id`=`item`.`parent` + WHERE `item`.`visible` $sql_extra $sql_seen - AND `item`.`deleted` = 0 AND `item`.`uid` = %d AND `item`.`wall` = 0 - ORDER BY `item`.`created` DESC LIMIT %d, %d " , - intval(local_user()), - intval($start), - intval($limit) + AND NOT `item`.`deleted` AND `item`.`uid` = %d AND NOT `item`.`wall` + ORDER BY `item`.`created` DESC LIMIT %d, %d ", + intval(local_user()), + intval($start), + intval($limit) ); - - if (DBM::is_result($r)) + if (DBM::is_result($r)) { $notifs = $this->formatNotifs($r, $ident); + } - $arr = array ( + $arr = [ 'notifications' => $notifs, 'ident' => $ident, 'total' => $total, - ); + ]; return $arr; } /** * @brief Total number of home notifications - * @param int|string $seen - * If 0 only include notifications into the query - * which aren't marked as "seen" + * @param int|string $seen If 0 only include notifications into the query + * which aren't marked as "seen" + * * @return int Number of home notifications */ - private function homeTotal($seen = 0) { + private function homeTotal($seen = 0) + { $sql_seen = ""; + $index_hint = ""; - if($seen === 0) - $sql_seen = " AND `item`.`unseen` = 1 "; + if ($seen === 0) { + $sql_seen = " AND `item`.`unseen` "; + $index_hint = "USE INDEX (`uid_unseen_contactid`)"; + } - $r = q("SELECT COUNT(*) AS `total` FROM `item` + $r = q( + "SELECT COUNT(*) AS `total` FROM `item` $index_hint WHERE `item`.`visible` = 1 AND `item`.`deleted` = 0 AND `item`.`uid` = %d AND `item`.`wall` = 1 $sql_seen", intval(local_user()) ); - - if (DBM::is_result($r)) + if (DBM::is_result($r)) { return $r[0]['total']; + } return 0; } @@ -633,71 +671,79 @@ class NotificationsManager { /** * @brief Get home notifications * - * @param int|string $seen - * If 0 only include notifications into the query - * which aren't marked as "seen" - * @param int $start Start the query at this point - * @param int $limit Maximum number of query results + * @param int|string $seen If 0 only include notifications into the query + * which aren't marked as "seen" + * @param int $start Start the query at this point + * @param int $limit Maximum number of query results * * @return array with - * string 'ident' => Notification identifier - * int 'total' => Total number of available home notifications - * array 'notifications' => Home notifications + * string 'ident' => Notification identifier + * int 'total' => Total number of available home notifications + * array 'notifications' => Home notifications */ - public function homeNotifs($seen = 0, $start = 0, $limit = 80) { + public function homeNotifs($seen = 0, $start = 0, $limit = 80) + { $ident = 'home'; $total = $this->homeTotal($seen); - $notifs = array(); + $notifs = []; $sql_seen = ""; + $index_hint = ""; - if($seen === 0) - $sql_seen = " AND `item`.`unseen` = 1 "; + if ($seen === 0) { + $sql_seen = " AND `item`.`unseen` "; + $index_hint = "USE INDEX (`uid_unseen_contactid`)"; + } - $r = q("SELECT `item`.`id`,`item`.`parent`, `item`.`verb`, `item`.`author-name`, `item`.`unseen`, + $r = q( + "SELECT `item`.`id`,`item`.`parent`, `item`.`verb`, `item`.`author-name`, `item`.`unseen`, `item`.`author-link`, `item`.`author-avatar`, `item`.`created`, `item`.`object` AS `object`, `pitem`.`author-name` AS `pname`, `pitem`.`author-link` AS `plink`, `pitem`.`guid` AS `pguid` - FROM `item` INNER JOIN `item` AS `pitem` ON `pitem`.`id`=`item`.`parent` - WHERE `item`.`visible` = 1 AND - `item`.`deleted` = 0 AND `item`.`uid` = %d AND `item`.`wall` = 1 + FROM `item` $index_hint STRAIGHT_JOIN `item` AS `pitem` ON `pitem`.`id`=`item`.`parent` + WHERE `item`.`visible` AND + NOT `item`.`deleted` AND `item`.`uid` = %d AND `item`.`wall` $sql_seen ORDER BY `item`.`created` DESC LIMIT %d, %d ", - intval(local_user()), - intval($start), - intval($limit) + intval(local_user()), + intval($start), + intval($limit) ); - - if (DBM::is_result($r)) + if (DBM::is_result($r)) { $notifs = $this->formatNotifs($r, $ident); + } - $arr = array ( + $arr = [ 'notifications' => $notifs, 'ident' => $ident, 'total' => $total, - ); + ]; return $arr; } /** * @brief Total number of introductions - * @param bool $all - * If false only include introductions into the query - * which aren't marked as ignored + * @param bool $all If false only include introductions into the query + * which aren't marked as ignored + * * @return int Number of introductions */ - private function introTotal($all = false) { + private function introTotal($all = false) + { $sql_extra = ""; - if(!$all) + if (!$all) { $sql_extra = " AND `ignore` = 0 "; + } - $r = q("SELECT COUNT(*) AS `total` FROM `intro` + $r = q( + "SELECT COUNT(*) AS `total` FROM `intro` WHERE `intro`.`uid` = %d $sql_extra AND `intro`.`blocked` = 0 ", - intval($_SESSION['uid']) + intval($_SESSION['uid']) ); - if (DBM::is_result($r)) + if (DBM::is_result($r)) { return $r[0]['total']; + } return 0; } @@ -705,29 +751,31 @@ class NotificationsManager { /** * @brief Get introductions * - * @param bool $all - * If false only include introductions into the query - * which aren't marked as ignored - * @param int $start Start the query at this point - * @param int $limit Maximum number of query results + * @param bool $all If false only include introductions into the query + * which aren't marked as ignored + * @param int $start Start the query at this point + * @param int $limit Maximum number of query results * * @return array with * string 'ident' => Notification identifier * int 'total' => Total number of available introductions * array 'notifications' => Introductions */ - public function introNotifs($all = false, $start = 0, $limit = 80) { + public function introNotifs($all = false, $start = 0, $limit = 80) + { $ident = 'introductions'; - $total = $this->introTotal($seen); - $notifs = array(); + $total = $this->introTotal($all); + $notifs = []; $sql_extra = ""; - if(!$all) + if (!$all) { $sql_extra = " AND `ignore` = 0 "; + } - /// @todo Fetch contact details by "get_contact_details_by_url" instead of queries to contact, fcontact and gcontact - $r = q("SELECT `intro`.`id` AS `intro_id`, `intro`.*, `contact`.*, - `fcontact`.`name` AS `fname`, `fcontact`.`url` AS `furl`, + /// @todo Fetch contact details by "Contact::getDetailsByUrl" instead of queries to contact, fcontact and gcontact + $r = q( + "SELECT `intro`.`id` AS `intro_id`, `intro`.*, `contact`.*, + `fcontact`.`name` AS `fname`, `fcontact`.`url` AS `furl`, `fcontact`.`addr` AS `faddr`, `fcontact`.`photo` AS `fphoto`, `fcontact`.`request` AS `frequest`, `gcontact`.`location` AS `glocation`, `gcontact`.`about` AS `gabout`, `gcontact`.`keywords` AS `gkeywords`, `gcontact`.`gender` AS `ggender`, @@ -738,19 +786,19 @@ class NotificationsManager { LEFT JOIN `fcontact` ON `intro`.`fid` = `fcontact`.`id` WHERE `intro`.`uid` = %d $sql_extra AND `intro`.`blocked` = 0 LIMIT %d, %d", - intval($_SESSION['uid']), - intval($start), - intval($limit) + intval($_SESSION['uid']), + intval($start), + intval($limit) ); - - if (DBM::is_result($r)) + if (DBM::is_result($r)) { $notifs = $this->formatIntros($r); + } - $arr = array ( + $arr = [ 'ident' => $ident, 'total' => $total, 'notifications' => $notifs, - ); + ]; return $arr; } @@ -761,69 +809,69 @@ class NotificationsManager { * @param array $intros The array from the db query * @return array with the introductions */ - private function formatIntros($intros) { + private function formatIntros($intros) + { $knowyou = ''; - foreach($intros as $it) { + foreach ($intros as $it) { // There are two kind of introduction. Contacts suggested by other contacts and normal connection requests. // We have to distinguish between these two because they use different data. - // Contact suggestions - if($it['fid']) { - - $return_addr = bin2hex($this->a->user['nickname'] . '@' . $this->a->get_hostname() . (($this->a->path) ? '/' . $this->a->path : '')); + if ($it['fid']) { + $return_addr = bin2hex(self::getApp()->user['nickname'] . '@' . self::getApp()->get_hostname() . ((self::getApp()->path) ? '/' . self::getApp()->path : '')); - $intro = array( + $intro = [ 'label' => 'friend_suggestion', - 'notify_type' => t('Friend Suggestion'), + 'notify_type' => L10n::t('Friend Suggestion'), 'intro_id' => $it['intro_id'], 'madeby' => $it['name'], + 'madeby_url' => $it['url'], + 'madeby_zrl' => Contact::magicLink($it['url']), + 'madeby_addr' => $it['addr'], 'contact_id' => $it['contact-id'], - 'photo' => ((x($it,'fphoto')) ? proxy_url($it['fphoto'], false, PROXY_SIZE_SMALL) : "images/person-175.jpg"), + 'photo' => ((x($it, 'fphoto')) ? proxy_url($it['fphoto'], false, PROXY_SIZE_SMALL) : "images/person-175.jpg"), 'name' => $it['fname'], - 'url' => zrl($it['furl']), + 'url' => $it['furl'], + 'zrl' => Contact::magicLink($it['furl']), 'hidden' => $it['hidden'] == 1, - 'post_newfriend' => (intval(PConfig::get(local_user(),'system','post_newfriend')) ? '1' : 0), - + 'post_newfriend' => (intval(PConfig::get(local_user(), 'system', 'post_newfriend')) ? '1' : 0), 'knowyou' => $knowyou, 'note' => $it['note'], 'request' => $it['frequest'] . '?addr=' . $return_addr, + ]; - ); - - // Normal connection requests + // Normal connection requests } else { - $it = $this->getMissingIntroData($it); // Don't show these data until you are connected. Diaspora is doing the same. - if($it['gnetwork'] === NETWORK_DIASPORA) { + if ($it['gnetwork'] === NETWORK_DIASPORA) { $it['glocation'] = ""; $it['gabout'] = ""; $it['ggender'] = ""; } - $intro = array( + $intro = [ 'label' => (($it['network'] !== NETWORK_OSTATUS) ? 'friend_request' : 'follower'), - 'notify_type' => (($it['network'] !== NETWORK_OSTATUS) ? t('Friend/Connect Request') : t('New Follower')), + 'notify_type' => (($it['network'] !== NETWORK_OSTATUS) ? L10n::t('Friend/Connect Request') : L10n::t('New Follower')), 'dfrn_id' => $it['issued-id'], 'uid' => $_SESSION['uid'], 'intro_id' => $it['intro_id'], 'contact_id' => $it['contact-id'], - 'photo' => ((x($it,'photo')) ? proxy_url($it['photo'], false, PROXY_SIZE_SMALL) : "images/person-175.jpg"), + 'photo' => ((x($it, 'photo')) ? proxy_url($it['photo'], false, PROXY_SIZE_SMALL) : "images/person-175.jpg"), 'name' => $it['name'], - 'location' => bbcode($it['glocation'], false, false), - 'about' => bbcode($it['gabout'], false, false), + 'location' => BBCode::convert($it['glocation'], false), + 'about' => BBCode::convert($it['gabout'], false), 'keywords' => $it['gkeywords'], 'gender' => $it['ggender'], 'hidden' => $it['hidden'] == 1, - 'post_newfriend' => (intval(PConfig::get(local_user(),'system','post_newfriend')) ? '1' : 0), + 'post_newfriend' => (intval(PConfig::get(local_user(), 'system', 'post_newfriend')) ? '1' : 0), 'url' => $it['url'], - 'zrl' => zrl($it['url']), + 'zrl' => Contact::magicLink($it['url']), 'addr' => $it['gaddr'], 'network' => $it['gnetwork'], 'knowyou' => $it['knowyou'], 'note' => $it['note'], - ); + ]; } $arr[] = $intro; @@ -840,7 +888,8 @@ class NotificationsManager { * * @return array The array with the intro data */ - private function getMissingIntroData($arr) { + private function getMissingIntroData($arr) + { // If the network and the addr isn't available from the gcontact // table entry, take the one of the contact table entry if ($arr['gnetwork'] == "") { @@ -853,7 +902,7 @@ class NotificationsManager { // If the network and addr is still not available // get the missing data data from other sources if ($arr['gnetwork'] == "" || $arr['gaddr'] == "") { - $ret = get_contact_details_by_url($arr['url']); + $ret = Contact::getDetailsByURL($arr['url']); if ($arr['gnetwork'] == "" && $ret['network'] != "") { $arr['gnetwork'] = $ret['network'];