Issue 8321: Follow items now work again
This commit is contained in:
parent
44eebde9cb
commit
825a7f9f93
3 changed files with 14 additions and 142 deletions
|
|
@ -20,156 +20,24 @@
|
||||||
*/
|
*/
|
||||||
|
|
||||||
use Friendica\App;
|
use Friendica\App;
|
||||||
use Friendica\Core\Hook;
|
use Friendica\Network\HTTPException;
|
||||||
use Friendica\Core\Logger;
|
use Friendica\Core\Logger;
|
||||||
use Friendica\Core\Session;
|
use Friendica\Core\Session;
|
||||||
use Friendica\Core\System;
|
|
||||||
use Friendica\Database\DBA;
|
|
||||||
use Friendica\DI;
|
|
||||||
use Friendica\Model\Item;
|
use Friendica\Model\Item;
|
||||||
use Friendica\Protocol\Activity;
|
|
||||||
use Friendica\Util\Security;
|
|
||||||
use Friendica\Util\Strings;
|
use Friendica\Util\Strings;
|
||||||
use Friendica\Util\XML;
|
|
||||||
|
|
||||||
function subthread_content(App $a) {
|
|
||||||
|
|
||||||
|
function subthread_content(App $a)
|
||||||
|
{
|
||||||
if (!Session::isAuthenticated()) {
|
if (!Session::isAuthenticated()) {
|
||||||
return;
|
throw new HTTPException\ForbiddenException();
|
||||||
}
|
}
|
||||||
|
|
||||||
$activity = Activity::FOLLOW;
|
|
||||||
|
|
||||||
$item_id = (($a->argc > 1) ? Strings::escapeTags(trim($a->argv[1])) : 0);
|
$item_id = (($a->argc > 1) ? Strings::escapeTags(trim($a->argv[1])) : 0);
|
||||||
|
|
||||||
$condition = ["`parent` = ? OR `parent-uri` = ? AND `parent` = `id`", $item_id, $item_id];
|
if (!Item::performLike($item_id, 'follow')) {
|
||||||
$item = Item::selectFirst([], $condition);
|
Logger::info('Following item failed', ['item' => $item_id]);
|
||||||
|
throw new HTTPException\BadRequestException();
|
||||||
if (empty($item_id) || !DBA::isResult($item)) {
|
|
||||||
Logger::log('subthread: no item ' . $item_id);
|
|
||||||
return;
|
|
||||||
}
|
}
|
||||||
|
Logger::info('Followed item', ['item' => $item_id]);
|
||||||
$owner_uid = $item['uid'];
|
return;
|
||||||
|
|
||||||
if (!Security::canWriteToUserWall($owner_uid)) {
|
|
||||||
return;
|
|
||||||
}
|
|
||||||
|
|
||||||
$remote_owner = null;
|
|
||||||
|
|
||||||
if (!$item['wall']) {
|
|
||||||
// The top level post may have been written by somebody on another system
|
|
||||||
$contact = DBA::selectFirst('contact', [], ['id' => $item['contact-id'], 'uid' => $item['uid']]);
|
|
||||||
if (!DBA::isResult($contact)) {
|
|
||||||
return;
|
|
||||||
}
|
|
||||||
if (!$contact['self']) {
|
|
||||||
$remote_owner = $contact;
|
|
||||||
}
|
|
||||||
}
|
|
||||||
|
|
||||||
$owner = null;
|
|
||||||
// this represents the post owner on this system.
|
|
||||||
|
|
||||||
$r = q("SELECT `contact`.*, `user`.`nickname` FROM `contact` LEFT JOIN `user` ON `contact`.`uid` = `user`.`uid`
|
|
||||||
WHERE `contact`.`self` = 1 AND `contact`.`uid` = %d LIMIT 1",
|
|
||||||
intval($owner_uid)
|
|
||||||
);
|
|
||||||
|
|
||||||
if (DBA::isResult($r)) {
|
|
||||||
$owner = $r[0];
|
|
||||||
}
|
|
||||||
|
|
||||||
if (!$owner) {
|
|
||||||
Logger::log('like: no owner');
|
|
||||||
return;
|
|
||||||
}
|
|
||||||
|
|
||||||
if (!$remote_owner) {
|
|
||||||
$remote_owner = $owner;
|
|
||||||
}
|
|
||||||
|
|
||||||
$contact = null;
|
|
||||||
// This represents the person posting
|
|
||||||
|
|
||||||
if (local_user() && (local_user() == $owner_uid)) {
|
|
||||||
$contact = $owner;
|
|
||||||
} else {
|
|
||||||
$contact = DBA::selectFirst('contact', [], ['id' => $_SESSION['visitor_id'], 'uid' => $owner_uid]);
|
|
||||||
if (!DBA::isResult($contact)) {
|
|
||||||
return;
|
|
||||||
}
|
|
||||||
}
|
|
||||||
|
|
||||||
$uri = Item::newURI($owner_uid);
|
|
||||||
|
|
||||||
$post_type = (($item['resource-id']) ? DI::l10n()->t('photo') : DI::l10n()->t('status'));
|
|
||||||
$objtype = (($item['resource-id']) ? Activity\ObjectType::IMAGE : Activity\ObjectType::NOTE );
|
|
||||||
$link = XML::escape('<link rel="alternate" type="text/html" href="' . DI::baseUrl() . '/display/' . $item['guid'] . '" />' . "\n");
|
|
||||||
$body = $item['body'];
|
|
||||||
|
|
||||||
$obj = <<< EOT
|
|
||||||
|
|
||||||
<object>
|
|
||||||
<type>$objtype</type>
|
|
||||||
<local>1</local>
|
|
||||||
<id>{$item['uri']}</id>
|
|
||||||
<link>$link</link>
|
|
||||||
<title></title>
|
|
||||||
<content>$body</content>
|
|
||||||
</object>
|
|
||||||
EOT;
|
|
||||||
$bodyverb = DI::l10n()->t('%1$s is following %2$s\'s %3$s');
|
|
||||||
|
|
||||||
if (!isset($bodyverb)) {
|
|
||||||
return;
|
|
||||||
}
|
|
||||||
|
|
||||||
$arr = [];
|
|
||||||
|
|
||||||
$arr['guid'] = System::createUUID();
|
|
||||||
$arr['uri'] = $uri;
|
|
||||||
$arr['uid'] = $owner_uid;
|
|
||||||
$arr['contact-id'] = $contact['id'];
|
|
||||||
$arr['wall'] = $item['wall'];
|
|
||||||
$arr['origin'] = 1;
|
|
||||||
$arr['gravity'] = GRAVITY_ACTIVITY;
|
|
||||||
$arr['parent'] = $item['id'];
|
|
||||||
$arr['parent-uri'] = $item['uri'];
|
|
||||||
$arr['thr-parent'] = $item['uri'];
|
|
||||||
$arr['owner-name'] = $remote_owner['name'];
|
|
||||||
$arr['owner-link'] = $remote_owner['url'];
|
|
||||||
$arr['owner-avatar'] = $remote_owner['thumb'];
|
|
||||||
$arr['author-name'] = $contact['name'];
|
|
||||||
$arr['author-link'] = $contact['url'];
|
|
||||||
$arr['author-avatar'] = $contact['thumb'];
|
|
||||||
|
|
||||||
$ulink = '[url=' . $contact['url'] . ']' . $contact['name'] . '[/url]';
|
|
||||||
$alink = '[url=' . $item['author-link'] . ']' . $item['author-name'] . '[/url]';
|
|
||||||
$plink = '[url=' . DI::baseUrl() . '/display/' . $item['guid'] . ']' . $post_type . '[/url]';
|
|
||||||
$arr['body'] = sprintf( $bodyverb, $ulink, $alink, $plink );
|
|
||||||
|
|
||||||
$arr['verb'] = $activity;
|
|
||||||
$arr['object-type'] = $objtype;
|
|
||||||
$arr['object'] = $obj;
|
|
||||||
$arr['allow_cid'] = $item['allow_cid'];
|
|
||||||
$arr['allow_gid'] = $item['allow_gid'];
|
|
||||||
$arr['deny_cid'] = $item['deny_cid'];
|
|
||||||
$arr['deny_gid'] = $item['deny_gid'];
|
|
||||||
$arr['visible'] = 1;
|
|
||||||
$arr['unseen'] = 1;
|
|
||||||
|
|
||||||
$post_id = Item::insert($arr);
|
|
||||||
|
|
||||||
if (!$item['visible']) {
|
|
||||||
Item::update(['visible' => true], ['id' => $item['id']]);
|
|
||||||
}
|
|
||||||
|
|
||||||
$arr['id'] = $post_id;
|
|
||||||
|
|
||||||
Hook::callAll('post_local_end', $arr);
|
|
||||||
|
|
||||||
exit();
|
|
||||||
|
|
||||||
}
|
}
|
||||||
|
|
|
||||||
|
|
@ -3168,6 +3168,10 @@ class Item
|
||||||
case 'unattendmaybe':
|
case 'unattendmaybe':
|
||||||
$activity = Activity::ATTENDMAYBE;
|
$activity = Activity::ATTENDMAYBE;
|
||||||
break;
|
break;
|
||||||
|
case 'follow':
|
||||||
|
case 'unfollow':
|
||||||
|
$activity = Activity::FOLLOW;
|
||||||
|
break;
|
||||||
default:
|
default:
|
||||||
Logger::log('like: unknown verb ' . $verb . ' for item ' . $item_id);
|
Logger::log('like: unknown verb ' . $verb . ' for item ' . $item_id);
|
||||||
return false;
|
return false;
|
||||||
|
|
|
||||||
|
|
@ -352,7 +352,7 @@ function frio_display_item(App $a, &$arr)
|
||||||
$subthread = [
|
$subthread = [
|
||||||
'menu' => 'follow_thread',
|
'menu' => 'follow_thread',
|
||||||
'title' => DI::l10n()->t('Follow Thread'),
|
'title' => DI::l10n()->t('Follow Thread'),
|
||||||
'action' => 'dosubthread(' . $arr['item']['id'] . '); return false;',
|
'action' => 'dosubthread(' . $arr['item']['id'] . ');',
|
||||||
'href' => '#'
|
'href' => '#'
|
||||||
];
|
];
|
||||||
}
|
}
|
||||||
|
|
|
||||||
Loading…
Reference in a new issue