From 6e6941a5466f45328d03dcb2819125c83e2c3c48 Mon Sep 17 00:00:00 2001 From: Michael Date: Tue, 6 Jun 2017 21:56:25 +0000 Subject: [PATCH] We have an alternate link --- include/dfrn.php | 59 +++++++++++++++++++++++++++++++++ mod/display.php | 21 ++++++++++-- view/templates/display-head.tpl | 2 +- 3 files changed, 79 insertions(+), 3 deletions(-) diff --git a/include/dfrn.php b/include/dfrn.php index 7e4d7af2bc..5d0e6337a4 100644 --- a/include/dfrn.php +++ b/include/dfrn.php @@ -284,6 +284,65 @@ class dfrn { return $atom; } + /** + * @brief Generate an atom entry for a given item id + * + * @param int $item_id The item id + * + * @return string DFRN feed entry + */ + public static function itemFeed($item_id) { + $r = q("SELECT `item`.*, `item`.`id` AS `item_id`, + `contact`.`name`, `contact`.`network`, `contact`.`photo`, `contact`.`url`, + `contact`.`name-date`, `contact`.`uri-date`, `contact`.`avatar-date`, + `contact`.`thumb`, `contact`.`dfrn-id`, `contact`.`self`, + `sign`.`signed_text`, `sign`.`signature`, `sign`.`signer` + FROM `item` + STRAIGHT_JOIN `contact` ON `contact`.`id` = `item`.`contact-id` + AND (NOT `contact`.`blocked` OR `contact`.`pending`) + LEFT JOIN `sign` ON `sign`.`iid` = `item`.`id` + WHERE `item`.`id` = %d AND `item`.`visible` AND NOT `item`.`moderated` AND `item`.`parent` != 0 + AND `item`.`wall` AND NOT `item`.`private`", + intval($item_id) + ); + + if (!dbm::is_result($r)) { + killme(); + } + + $item = $r[0]; + + $r = q("SELECT `contact`.*, `user`.`nickname`, `user`.`timezone`, `user`.`page-flags`, `user`.`account-type` + FROM `contact` INNER JOIN `user` ON `user`.`uid` = `contact`.`uid` + WHERE `contact`.`self` AND `user`.`uid` = %d LIMIT 1", + intval($item['uid']) + ); + + if (!dbm::is_result($r)) { + killme(); + } + + $owner = $r[0]; + + $doc = new DOMDocument('1.0', 'utf-8'); + $doc->formatOutput = true; + + $alternatelink = $owner['url']; + + $author = "dfrn:owner"; + //$author = "author"; + + $root = self::add_header($doc, $owner, $author, $alternatelink, true); + + $type = 'html'; + + $entry = self::entry($doc, $type, $item, $owner, true); + $root->appendChild($entry); + + $atom = trim($doc->saveXML()); + return $atom; + } + /** * @brief Create XML text for DFRN mails * diff --git a/mod/display.php b/mod/display.php index bc0ba388bf..e31b89998c 100644 --- a/mod/display.php +++ b/mod/display.php @@ -11,6 +11,17 @@ function display_init(App $a) { $nick = (($a->argc > 1) ? $a->argv[1] : ''); $profiledata = array(); + if ($a->argc == 3) { + if (substr($a->argv[2], -5) == '.atom') { + require_once('include/dfrn.php'); + $item_id = substr($a->argv[2], 0, -5); + $xml = dfrn::itemFeed($item_id); + header("Content-type: application/atom+xml"); + echo $xml; + http_status_exit(($xml) ? 200 : 500); + } + } + // If there is only one parameter, then check if this parameter could be a guid if ($a->argc == 2) { $nick = ""; @@ -278,11 +289,17 @@ function display_content(App $a, $update = 0) { return; } - $alternate = App::get_baseurl().'/display/'.$nick.'/'.$item_id.'.atom'; + // We are displaying an "alternate" link if that post was public. See issue 2864 + $items = q("SELECT `id` FROM `item` WHERE `id` = %d AND NOT `private` AND `wall`", intval($item_id)); + if (dbm::is_result($items)) { + $alternate = App::get_baseurl().'/display/'.$nick.'/'.$item_id.'.atom'; + } else { + $alternate = ''; + } + $a->page['htmlhead'] .= replace_macros(get_markup_template('display-head.tpl'), array('$alternate' => $alternate)); - $groups = array(); $contact = null; diff --git a/view/templates/display-head.tpl b/view/templates/display-head.tpl index f4d050eec2..dda8162146 100644 --- a/view/templates/display-head.tpl +++ b/view/templates/display-head.tpl @@ -1,4 +1,4 @@ - +{{if $alternate}}{{/if}}