Fix formatting and PHP notices in include/tags

- Change remaining double quotes into single
This commit is contained in:
Hypolite Petovan 2018-01-01 15:57:29 -05:00
parent 9f04017e27
commit d0d4e3813b
1 changed files with 83 additions and 71 deletions

View File

@ -1,4 +1,5 @@
<?php <?php
/** /**
* @file include/tags.php * @file include/tags.php
*/ */
@ -8,11 +9,13 @@ use Friendica\Core\System;
use Friendica\Database\DBM; use Friendica\Database\DBM;
use Friendica\Model\Contact; use Friendica\Model\Contact;
function create_tags_from_item($itemid) { function create_tags_from_item($itemid)
{
$profile_base = System::baseUrl(); $profile_base = System::baseUrl();
$profile_data = parse_url($profile_base); $profile_data = parse_url($profile_base);
$profile_base_friendica = $profile_data['host'].$profile_data['path']."/profile/"; $profile_path = defaults($profile_data, 'path', '');
$profile_base_diaspora = $profile_data['host'].$profile_data['path']."/u/"; $profile_base_friendica = $profile_data['host'] . $profile_path . '/profile/';
$profile_base_diaspora = $profile_data['host'] . $profile_path . '/u/';
$messages = q("SELECT `guid`, `uid`, `id`, `edited`, `deleted`, `created`, `received`, `title`, `body`, `tag`, `parent` FROM `item` WHERE `id` = %d LIMIT 1", intval($itemid)); $messages = q("SELECT `guid`, `uid`, `id`, `edited`, `deleted`, `created`, `received`, `title`, `body`, `tag`, `parent` FROM `item` WHERE `id` = %d LIMIT 1", intval($itemid));
@ -28,48 +31,53 @@ function create_tags_from_item($itemid) {
intval(TERM_HASHTAG), intval(TERM_HASHTAG),
intval(TERM_MENTION)); intval(TERM_MENTION));
if ($message["deleted"]) if ($message['deleted']) {
return; return;
}
$taglist = explode(",", $message["tag"]); $taglist = explode(',', $message['tag']);
$tags = ""; $tags = '';
foreach ($taglist as $tag) foreach ($taglist as $tag) {
if ((substr(trim($tag), 0, 1) == "#") || (substr(trim($tag), 0, 1) == "@")) if ((substr(trim($tag), 0, 1) == '#') || (substr(trim($tag), 0, 1) == '@')) {
$tags .= " ".trim($tag); $tags .= ' ' . trim($tag);
else } else {
$tags .= " #".trim($tag); $tags .= ' #' . trim($tag);
}
}
$data = " ".$message["title"]." ".$message["body"]." ".$tags." "; $data = ' ' . $message['title'] . ' ' . $message['body'] . ' ' . $tags . ' ';
// ignore anything in a code block // ignore anything in a code block
$data = preg_replace('/\[code\](.*?)\[\/code\]/sm','',$data); $data = preg_replace('/\[code\](.*?)\[\/code\]/sm', '', $data);
$tags = array(); $tags = array();
$pattern = "/\W\#([^\[].*?)[\s'\".,:;\?!\[\]\/]/ism"; $pattern = '/\W\#([^\[].*?)[\s\'".,:;\?!\[\]\/]/ism';
if (preg_match_all($pattern, $data, $matches)) if (preg_match_all($pattern, $data, $matches)) {
foreach ($matches[1] as $match) foreach ($matches[1] as $match) {
$tags["#".strtolower($match)] = ""; $tags['#' . strtolower($match)] = '';
}
$pattern = "/\W([\#@])\[url\=(.*?)\](.*?)\[\/url\]/ism";
if (preg_match_all($pattern, $data, $matches, PREG_SET_ORDER)) {
foreach ($matches as $match)
$tags[$match[1].strtolower(trim($match[3], ',.:;[]/\"?!'))] = $match[2];
} }
foreach ($tags as $tag=>$link) { $pattern = '/\W([\#@])\[url\=(.*?)\](.*?)\[\/url\]/ism';
if (preg_match_all($pattern, $data, $matches, PREG_SET_ORDER)) {
foreach ($matches as $match) {
$tags[$match[1] . strtolower(trim($match[3], ',.:;[]/\"?!'))] = $match[2];
}
}
if (substr(trim($tag), 0, 1) == "#") { foreach ($tags as $tag => $link) {
if (substr(trim($tag), 0, 1) == '#') {
// try to ignore #039 or #1 or anything like that // try to ignore #039 or #1 or anything like that
if (ctype_digit(substr(trim($tag),1))) if (ctype_digit(substr(trim($tag), 1)))
continue; continue;
// try to ignore html hex escapes, e.g. #x2317 // try to ignore html hex escapes, e.g. #x2317
if ((substr(trim($tag),1,1) == 'x' || substr(trim($tag),1,1) == 'X') && ctype_digit(substr(trim($tag),2))) if ((substr(trim($tag), 1, 1) == 'x' || substr(trim($tag), 1, 1) == 'X') && ctype_digit(substr(trim($tag), 2)))
continue; continue;
$type = TERM_HASHTAG; $type = TERM_HASHTAG;
$term = substr($tag, 1); $term = substr($tag, 1);
} elseif (substr(trim($tag), 0, 1) == "@") { } elseif (substr(trim($tag), 0, 1) == '@') {
$type = TERM_MENTION; $type = TERM_MENTION;
$term = substr($tag, 1); $term = substr($tag, 1);
} else { // This shouldn't happen } else { // This shouldn't happen
@ -77,78 +85,78 @@ function create_tags_from_item($itemid) {
$term = $tag; $term = $tag;
} }
if ($message["uid"] == 0) { if ($message['uid'] == 0) {
$global = true; $global = true;
q("UPDATE `term` SET `global` = 1 WHERE `otype` = %d AND `guid` = '%s'", q("UPDATE `term` SET `global` = 1 WHERE `otype` = %d AND `guid` = '%s'",
intval(TERM_OBJ_POST), dbesc($message["guid"])); intval(TERM_OBJ_POST), dbesc($message['guid']));
} else { } else {
$isglobal = q("SELECT `global` FROM `term` WHERE `uid` = 0 AND `otype` = %d AND `guid` = '%s'", $isglobal = q("SELECT `global` FROM `term` WHERE `uid` = 0 AND `otype` = %d AND `guid` = '%s'",
intval(TERM_OBJ_POST), dbesc($message["guid"])); intval(TERM_OBJ_POST), dbesc($message['guid']));
$global = (count($isglobal) > 0); $global = (count($isglobal) > 0);
} }
$r = q("INSERT INTO `term` (`uid`, `oid`, `otype`, `type`, `term`, `url`, `guid`, `created`, `received`, `global`) $r = q("INSERT INTO `term` (`uid`, `oid`, `otype`, `type`, `term`, `url`, `guid`, `created`, `received`, `global`)
VALUES (%d, %d, %d, %d, '%s', '%s', '%s', '%s', '%s', %d)", VALUES (%d, %d, %d, %d, '%s', '%s', '%s', '%s', '%s', %d)",
intval($message["uid"]), intval($itemid), intval(TERM_OBJ_POST), intval($type), dbesc($term), intval($message['uid']), intval($itemid), intval(TERM_OBJ_POST), intval($type), dbesc($term),
dbesc($link), dbesc($message["guid"]), dbesc($message["created"]), dbesc($message["received"]), intval($global)); dbesc($link), dbesc($message['guid']), dbesc($message['created']), dbesc($message['received']), intval($global));
// Search for mentions // Search for mentions
if ((substr($tag, 0, 1) == '@') && (strpos($link, $profile_base_friendica) || strpos($link, $profile_base_diaspora))) { if ((substr($tag, 0, 1) == '@') && (strpos($link, $profile_base_friendica) || strpos($link, $profile_base_diaspora))) {
$users = q("SELECT `uid` FROM `contact` WHERE self AND (`url` = '%s' OR `nurl` = '%s')", $link, $link); $users = q("SELECT `uid` FROM `contact` WHERE self AND (`url` = '%s' OR `nurl` = '%s')", $link, $link);
foreach ($users AS $user) { foreach ($users AS $user) {
if ($user["uid"] == $message["uid"]) { if ($user['uid'] == $message['uid']) {
q("UPDATE `item` SET `mention` = 1 WHERE `id` = %d", intval($itemid)); q("UPDATE `item` SET `mention` = 1 WHERE `id` = %d", intval($itemid));
q("UPDATE `thread` SET `mention` = 1 WHERE `iid` = %d", intval($message["parent"])); q("UPDATE `thread` SET `mention` = 1 WHERE `iid` = %d", intval($message['parent']));
} }
} }
} }
} }
} }
function create_tags_from_itemuri($itemuri, $uid) { function create_tags_from_itemuri($itemuri, $uid)
{
$messages = q("SELECT `id` FROM `item` WHERE uri ='%s' AND uid=%d", dbesc($itemuri), intval($uid)); $messages = q("SELECT `id` FROM `item` WHERE uri ='%s' AND uid=%d", dbesc($itemuri), intval($uid));
if (count($messages)) { if (count($messages)) {
foreach ($messages as $message) { foreach ($messages as $message) {
create_tags_from_item($message["id"]); create_tags_from_item($message['id']);
} }
} }
} }
function update_items() { function update_items()
{
$messages = dba::p("SELECT `oid`,`item`.`guid`, `item`.`created`, `item`.`received` FROM `term` INNER JOIN `item` ON `item`.`id`=`term`.`oid` WHERE `term`.`otype` = 1 AND `term`.`guid` = ''"); $messages = dba::p("SELECT `oid`,`item`.`guid`, `item`.`created`, `item`.`received` FROM `term` INNER JOIN `item` ON `item`.`id`=`term`.`oid` WHERE `term`.`otype` = 1 AND `term`.`guid` = ''");
logger("fetched messages: ".dba::num_rows($messages)); logger('fetched messages: ' . dba::num_rows($messages));
while ($message = dba::fetch($messages)) { while ($message = dba::fetch($messages)) {
if ($message['uid'] == 0) {
if ($message["uid"] == 0) {
$global = true; $global = true;
q("UPDATE `term` SET `global` = 1 WHERE `otype` = %d AND `guid` = '%s'", q("UPDATE `term` SET `global` = 1 WHERE `otype` = %d AND `guid` = '%s'",
intval(TERM_OBJ_POST), dbesc($message["guid"])); intval(TERM_OBJ_POST), dbesc($message['guid']));
} else { } else {
$isglobal = q("SELECT `global` FROM `term` WHERE `uid` = 0 AND `otype` = %d AND `guid` = '%s'", $isglobal = q("SELECT `global` FROM `term` WHERE `uid` = 0 AND `otype` = %d AND `guid` = '%s'",
intval(TERM_OBJ_POST), dbesc($message["guid"])); intval(TERM_OBJ_POST), dbesc($message['guid']));
$global = (count($isglobal) > 0); $global = (count($isglobal) > 0);
} }
q("UPDATE `term` SET `guid` = '%s', `created` = '%s', `received` = '%s', `global` = %d WHERE `otype` = %d AND `oid` = %d", q("UPDATE `term` SET `guid` = '%s', `created` = '%s', `received` = '%s', `global` = %d WHERE `otype` = %d AND `oid` = %d",
dbesc($message["guid"]), dbesc($message["created"]), dbesc($message["received"]), dbesc($message['guid']), dbesc($message['created']), dbesc($message['received']),
intval($global), intval(TERM_OBJ_POST), intval($message["oid"])); intval($global), intval(TERM_OBJ_POST), intval($message['oid']));
} }
dba::close($messages); dba::close($messages);
$messages = dba::p("SELECT `guid` FROM `item` WHERE `uid` = 0"); $messages = dba::p("SELECT `guid` FROM `item` WHERE `uid` = 0");
logger("fetched messages: ".dba::num_rows($messages)); logger('fetched messages: ' . dba::num_rows($messages));
while ($message = dba::fetch(messages)) { while ($message = dba::fetch(messages)) {
q("UPDATE `item` SET `global` = 1 WHERE `guid` = '%s'", dbesc($message["guid"])); q("UPDATE `item` SET `global` = 1 WHERE `guid` = '%s'", dbesc($message['guid']));
} }
dba::close($messages); dba::close($messages);
@ -166,21 +174,22 @@ function update_items() {
* *
* @return arr Alphabetical sorted array of used tags of an user. * @return arr Alphabetical sorted array of used tags of an user.
*/ */
function tagadelic($uid, $count = 0, $owner_id = 0, $flags = '', $type = TERM_HASHTAG) { function tagadelic($uid, $count = 0, $owner_id = 0, $flags = '', $type = TERM_HASHTAG)
require_once('include/security.php'); {
require_once 'include/security.php';
$item_condition = item_condition(); $item_condition = item_condition();
$sql_options = item_permissions_sql($uid); $sql_options = item_permissions_sql($uid);
$limit = $count ? sprintf("LIMIT %d", intval($count)) : ""; $limit = $count ? sprintf('LIMIT %d', intval($count)) : '';
if ($flags) { if ($flags) {
if ($flags === 'wall') { if ($flags === 'wall') {
$sql_options .= " AND `item`.`wall` "; $sql_options .= ' AND `item`.`wall` ';
} }
} }
if ($owner_id) { if ($owner_id) {
$sql_options .= " AND `item`.`owner-id` = ".intval($owner_id)." "; $sql_options .= ' AND `item`.`owner-id` = ' . intval($owner_id) . ' ';
} }
// Fetch tags // Fetch tags
@ -194,7 +203,7 @@ function tagadelic($uid, $count = 0, $owner_id = 0, $flags = '', $type = TERM_HA
$type, $type,
TERM_OBJ_POST TERM_OBJ_POST
); );
if(!DBM::is_result($r)) { if (!DBM::is_result($r)) {
return array(); return array();
} }
@ -212,32 +221,32 @@ function tagadelic($uid, $count = 0, $owner_id = 0, $flags = '', $type = TERM_HA
* *
* @return string HTML formatted output. * @return string HTML formatted output.
*/ */
function wtagblock($uid, $count = 0,$owner_id = 0, $flags = '', $type = TERM_HASHTAG) { function wtagblock($uid, $count = 0, $owner_id = 0, $flags = '', $type = TERM_HASHTAG)
{
$o = ''; $o = '';
$r = tagadelic($uid, $count, $owner_id, $flags, $type); $r = tagadelic($uid, $count, $owner_id, $flags, $type);
if (count($r)) { if (count($r)) {
$contact = dba::select( $contact = dba::select(
"contact", 'contact',
array("url"), array('url'),
array("id" => $uid), array('id' => $uid),
array("limit" => 1) array('limit' => 1)
); );
$url = System::removedBaseUrl($contact['url']); $url = System::removedBaseUrl($contact['url']);
foreach ($r as $rr) { foreach ($r as $rr) {
$tag['level'] = $rr[2]; $tag['level'] = $rr[2];
$tag['url'] = $url."?tag=".urlencode($rr[0]); $tag['url'] = $url . '?tag=' . urlencode($rr[0]);
$tag['name'] = $rr[0]; $tag['name'] = $rr[0];
$tags[] = $tag; $tags[] = $tag;
} }
$tpl = get_markup_template("tagblock_widget.tpl"); $tpl = get_markup_template('tagblock_widget.tpl');
$o = replace_macros($tpl, array( $o = replace_macros($tpl, array(
'$title' => t('Tags'), '$title' => t('Tags'),
'$tags' => $tags '$tags' => $tags
)); ));
} }
return $o; return $o;
} }
@ -248,7 +257,8 @@ function wtagblock($uid, $count = 0,$owner_id = 0, $flags = '', $type = TERM_HAS
* @param array $arr Array of tags/terms with tag/term name and total count of use. * @param array $arr Array of tags/terms with tag/term name and total count of use.
* @return array Alphabetical sorted array of used tags/terms of an user. * @return array Alphabetical sorted array of used tags/terms of an user.
*/ */
function tag_calc($arr) { function tag_calc($arr)
{
$tags = array(); $tags = array();
$min = 1e9; $min = 1e9;
$max = -1e9; $max = -1e9;
@ -285,7 +295,8 @@ function tag_calc($arr) {
* *
* @return int * @return int
*/ */
function tags_sort($a, $b) { function tags_sort($a, $b)
{
if (strtolower($a[0]) == strtolower($b[0])) { if (strtolower($a[0]) == strtolower($b[0])) {
return 0; return 0;
} }
@ -298,21 +309,22 @@ function tags_sort($a, $b) {
* @param int $limit Max number of displayed tags. * @param int $limit Max number of displayed tags.
* @return string HTML formattat output. * @return string HTML formattat output.
*/ */
function tagcloud_wall_widget($limit = 50) { function tagcloud_wall_widget($limit = 50)
{
$a = get_app(); $a = get_app();
if(!$a->profile['profile_uid'] || !$a->profile['url']) { if (!$a->profile['profile_uid'] || !$a->profile['url']) {
return ""; return '';
} }
if(Feature::isEnabled($a->profile['profile_uid'], 'tagadelic')) { if (Feature::isEnabled($a->profile['profile_uid'], 'tagadelic')) {
$owner_id = Contact::getIdForURL($a->profile['url']); $owner_id = Contact::getIdForURL($a->profile['url']);
if(!$owner_id) { if (!$owner_id) {
return ""; return '';
} }
return wtagblock($a->profile['profile_uid'], $limit, $owner_id, 'wall'); return wtagblock($a->profile['profile_uid'], $limit, $owner_id, 'wall');
} }
return ""; return '';
} }