<?php
/**
- * @copyright Copyright (C) 2010-2021, the Friendica project
+ * @copyright Copyright (C) 2010-2022, the Friendica project
*
* @license GNU AGPL version 3 or any later version
*
use Friendica\Core\Logger;
use Friendica\Core\Protocol;
+use Friendica\Core\Worker;
use Friendica\Database\DBA;
+use Friendica\DI;
+use Friendica\Model\Item;
use Friendica\Network\Probe;
use Friendica\Util\DateTimeFormat;
use Friendica\Util\Strings;
* @throws \Friendica\Network\HTTPException\InternalServerErrorException
* @throws \ImagickException
*/
- public static function getByURL($handle, $update = null)
+ public static function getByURL(string $handle, $update = null): array
{
+ Logger::debug('Fetch fcontact', ['handle' => $handle, 'update' => $update]);
$person = DBA::selectFirst('fcontact', [], ['network' => Protocol::DIASPORA, 'addr' => $handle]);
if (!DBA::isResult($person)) {
$urls = [$handle, str_replace('http://', 'https://', $handle), Strings::normaliseLink($handle)];
}
if (DBA::isResult($person)) {
- Logger::debug('In cache', ['person' => $person]);
+ Logger::debug('In cache', ['handle' => $handle]);
if (is_null($update)) {
- // update record occasionally so it doesn't get stale
- $d = strtotime($person["updated"]." +00:00");
- if ($d < strtotime("now - 14 days")) {
- $update = true;
- }
-
- if (empty($person['guid']) || empty($person['uri-id'])) {
- $update = true;
+ $update = empty($person['guid']) || empty($person['uri-id']) || ($person['created'] <= DBA::NULL_DATETIME);
+ if (GServer::getNextUpdateDate(true, $person['created'], $person['updated'], false) < DateTimeFormat::utcNow()) {
+ Logger::debug('Start background update', ['handle' => $handle]);
+ Worker::add(['priority' => Worker::PRIORITY_LOW, 'dont_fork' => true], 'UpdateFContact', $handle);
}
}
} elseif (is_null($update)) {
- $update = !DBA::isResult($person);
+ $update = true;
} else {
$person = [];
}
// Note that Friendica contacts will return a "Diaspora person"
// if Diaspora connectivity is enabled on their server
if ($data['network'] ?? '' === Protocol::DIASPORA) {
- self::updateFContact($data);
+ self::updateFromProbeArray($data);
$person = self::getByURL($handle, false);
}
* @param array $arr The fcontact data
* @throws \Exception
*/
- private static function updateFContact($arr)
+ public static function updateFromProbeArray(array $arr)
{
$uriid = ItemURI::insert(['uri' => $arr['url'], 'guid' => $arr['guid']]);
- $contact = Contact::getByUriId($uriid, ['id']);
- if (!empty($contact['id'])) {
+ $fcontact = DBA::selectFirst('fcontact', ['created'], ['url' => $arr['url'], 'network' => $arr['network']]);
+ $contact = Contact::getByUriId($uriid, ['id', 'created']);
+ $apcontact = APContact::getByURL($arr['url'], false);
+ if (!empty($apcontact)) {
+ $interacted = $apcontact['following_count'];
+ $interacting = $apcontact['followers_count'];
+ $posts = $apcontact['statuses_count'];
+ } elseif (!empty($contact['id'])) {
$last_interaction = DateTimeFormat::utc('now - 180 days');
$interacted = DBA::count('contact-relation', ["`cid` = ? AND NOT `follows` AND `last-interaction` > ?", $contact['id'], $last_interaction]);
$interacting = DBA::count('contact-relation', ["`relation-cid` = ? AND NOT `follows` AND `last-interaction` > ?", $contact['id'], $last_interaction]);
- $posts = Post::countPosts(['author-id' => $contact['id'], 'gravity' => [GRAVITY_PARENT, GRAVITY_COMMENT]]);
+ $posts = DBA::count('post', ['author-id' => $contact['id'], 'gravity' => [Item::GRAVITY_PARENT, Item::GRAVITY_COMMENT]]);
}
- $fields = ['name' => $arr["name"], 'photo' => $arr["photo"],
- 'request' => $arr["request"], 'nick' => $arr["nick"],
- 'addr' => strtolower($arr["addr"]), 'guid' => $arr["guid"],
- 'batch' => $arr["batch"], 'notify' => $arr["notify"],
- 'poll' => $arr["poll"], 'confirm' => $arr["confirm"],
- 'alias' => $arr["alias"], 'pubkey' => $arr["pubkey"],
- 'uri-id' => $uriid, 'interacting_count' => $interacting ?? 0,
- 'interacted_count' => $interacted ?? 0, 'post_count' => $posts ?? 0,
- 'updated' => DateTimeFormat::utcNow()];
-
- $condition = ['url' => $arr["url"], 'network' => $arr["network"]];
+ $fields = [
+ 'name' => $arr['name'],
+ 'photo' => $arr['photo'],
+ 'request' => $arr['request'],
+ 'nick' => $arr['nick'],
+ 'addr' => strtolower($arr['addr']),
+ 'guid' => $arr['guid'],
+ 'batch' => $arr['batch'],
+ 'notify' => $arr['notify'],
+ 'poll' => $arr['poll'],
+ 'confirm' => $arr['confirm'],
+ 'alias' => $arr['alias'],
+ 'pubkey' => $arr['pubkey'],
+ 'uri-id' => $uriid,
+ 'interacting_count' => $interacting ?? 0,
+ 'interacted_count' => $interacted ?? 0,
+ 'post_count' => $posts ?? 0,
+ 'updated' => DateTimeFormat::utcNow(),
+ ];
+
+ if (empty($fcontact['created'])) {
+ $fields['created'] = $fields['updated'];
+ } elseif (!empty($contact['created']) && ($fcontact['created'] <= DBA::NULL_DATETIME)) {
+ $fields['created'] = $contact['created'];
+ }
- DBA::update('fcontact', $fields, $condition, true);
+ $fields = DI::dbaDefinition()->truncateFieldsForTable('fcontact', $fields);
+ DBA::update('fcontact', $fields, ['url' => $arr['url'], 'network' => $arr['network']], true);
}
/**
* get a url (scheme://domain.tld/u/user) from a given Diaspora*
* fcontact guid
*
- * @param mixed $fcontact_guid Hexadecimal string guid
- *
- * @return string the contact url or null
+ * @param string $fcontact_guid Hexadecimal string guid
+ * @return string|null the contact url or null
* @throws \Exception
*/
- public static function getUrlByGuid($fcontact_guid)
+ public static function getUrlByGuid(string $fcontact_guid)
{
Logger::info('fcontact', ['guid' => $fcontact_guid]);