keith/friendica
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

Merge pull request #12314 from annando/issue-12302

Browse source 
Issue 12302: Remote self is working again
This commit is contained in:
Hypolite Petovan 2022-12-03 02:55:34 -05:00 committed by GitHub
commit dbbbef368f
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
1 changed files with 39 additions and 44 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

57
src/Model/Item.php
View file

@ -2291,24 +2291,24 @@ class Item
public static function isRemoteSelf(array $contact, array &$datarray): bool public static function isRemoteSelf(array $contact, array &$datarray): bool
{ {
if (!$contact['remote_self']) { if ($contact['remote_self'] != Contact::MIRROR_OWN_POST) {
return false; return false;
} }
// Prevent the forwarding of posts that are forwarded // Prevent the forwarding of posts that are forwarded
if (!empty($datarray["extid"]) && ($datarray["extid"] == Protocol::DFRN)) { if (!empty($datarray['extid']) && ($datarray['extid'] == Protocol::DFRN)) {
Logger::info('Already forwarded'); Logger::info('Already forwarded');
return false; return false;
} }
// Prevent to forward already forwarded posts // Prevent to forward already forwarded posts
if ($datarray["app"] == DI::baseUrl()->getHostname()) { if ($datarray['app'] == DI::baseUrl()->getHostname()) {
Logger::info('Already forwarded (second test)'); Logger::info('Already forwarded (second test)');
return false; return false;
} }
// Only forward posts // Only forward posts
if ($datarray["verb"] != Activity::POST) { if ($datarray['verb'] != Activity::POST) {
Logger::info('No post'); Logger::info('No post');
return false; return false;
} }
@ -2320,54 +2320,49 @@ class Item
$datarray2 = $datarray; $datarray2 = $datarray;
Logger::info('remote-self start', ['contact' => $contact['url'], 'remote_self'=> $contact['remote_self'], 'item' => $datarray]); Logger::info('remote-self start', ['contact' => $contact['url'], 'remote_self'=> $contact['remote_self'], 'item' => $datarray]);
if ($contact['remote_self'] == Contact::MIRROR_OWN_POST) {
$self = DBA::selectFirst('contact', ['id', 'name', 'url', 'thumb'], $self = DBA::selectFirst('contact', ['id', 'name', 'url', 'thumb'],
['uid' => $contact['uid'], 'self' => true]); ['uid' => $contact['uid'], 'self' => true]);
if (DBA::isResult($self)) { if (!DBA::isResult($self)) {
$datarray['contact-id'] = $self["id"]; Logger::error('Self contact not found', ['uid' => $contact['uid']]);
return false;
}
$datarray['owner-name'] = $self["name"]; $datarray['contact-id'] = $self['id'];
$datarray['owner-link'] = $self["url"];
$datarray['owner-avatar'] = $self["thumb"];
$datarray['author-name'] = $datarray['owner-name']; $datarray['author-name'] = $datarray['owner-name'] = $self['name'];
$datarray['author-link'] = $datarray['owner-link']; $datarray['author-link'] = $datarray['owner-link'] = $self['url'];
$datarray['author-avatar'] = $datarray['owner-avatar']; $datarray['author-avatar'] = $datarray['owner-avatar'] = $self['thumb'];
unset($datarray['edited']); unset($datarray['edited']);
unset($datarray['network']); unset($datarray['network']);
unset($datarray['owner-id']); unset($datarray['owner-id']);
unset($datarray['author-id']); unset($datarray['author-id']);
}
if ($contact['network'] != Protocol::FEED) { if ($contact['network'] != Protocol::FEED) {
$old_uri_id = $datarray["uri-id"] ?? 0; $old_uri_id = $datarray['uri-id'] ?? 0;
$datarray["guid"] = System::createUUID(); $datarray['guid'] = System::createUUID();
unset($datarray["plink"]); unset($datarray['plink']);
$datarray["uri"] = self::newURI($datarray["guid"]); $datarray['uri'] = self::newURI($datarray['guid']);
$datarray["uri-id"] = ItemURI::getIdByURI($datarray["uri"]); $datarray['uri-id'] = ItemURI::getIdByURI($datarray['uri']);
$datarray["extid"] = Protocol::DFRN; $datarray['extid'] = Protocol::DFRN;
$urlpart = parse_url($datarray2['author-link']); $urlpart = parse_url($datarray2['author-link']);
$datarray["app"] = $urlpart["host"]; $datarray['app'] = $urlpart['host'];
if (!empty($old_uri_id)) { if (!empty($old_uri_id)) {
Post\Media::copy($old_uri_id, $datarray["uri-id"]); Post\Media::copy($old_uri_id, $datarray['uri-id']);
} }
unset($datarray["parent-uri"]); unset($datarray['parent-uri']);
unset($datarray["thr-parent"]); unset($datarray['thr-parent']);
} else {
$datarray['private'] = self::PUBLIC;
}
}
if ($contact['network'] != Protocol::FEED) {
// Store the original post // Store the original post
$result = self::insert($datarray2); $result = self::insert($datarray2);
Logger::info('remote-self post original item', ['contact' => $contact['url'], 'result'=> $result, 'item' => $datarray2]); Logger::info('remote-self post original item', ['contact' => $contact['url'], 'result'=> $result, 'item' => $datarray2]);
} else { } else {
Logger::info('No valid mirroring option', ['uid' => $contact['uid'], 'id' => $contact['id'], 'network' => $contact['network'], 'remote_self' => $contact['remote_self']]); $datarray['private'] = self::PUBLIC;
return false; $datarray['app'] = 'Feed';
$result = true;
} }
return (bool)$result; return (bool)$result;