- fixed a null value handled over to Friendica\Model\APContact::getByURL()
- added missing type-hints
This commit is contained in:
Roland Häder 2022-06-20 19:04:01 +02:00
parent e96a548286
commit 4fb03cf163
No known key found for this signature in database
GPG Key ID: C82EDE5DDFA0BA77
2 changed files with 9 additions and 10 deletions

View File

@ -1841,13 +1841,13 @@ class Item
$parsed = parse_url($uri);
// Remove the scheme to make sure that "https" and "http" doesn't make a difference
unset($parsed["scheme"]);
unset($parsed['scheme']);
// Glue it together to be able to make a hash from it
$host_id = implode("/", $parsed);
$host_id = implode('/', $parsed);
// Use a mixture of several hashes to provide some GUID like experience
return hash("crc32", $host) . '-'. hash('joaat', $host_id) . '-'. hash('fnv164', $host_id);
return hash('crc32', $host) . '-'. hash('joaat', $host_id) . '-'. hash('fnv164', $host_id);
}
/**
@ -1859,9 +1859,9 @@ class Item
* @return string
* @throws \Friendica\Network\HTTPException\InternalServerErrorException
*/
public static function newURI($uid, $guid = "")
public static function newURI(int $uid, string $guid = ''): string
{
if ($guid == "") {
if ($guid == '') {
$guid = System::createUUID();
}

View File

@ -80,13 +80,12 @@ class Receiver
/**
* Checks incoming message from the inbox
*
* @param $body
* @param $header
* @param string $body Body string
* @param array $header Header lines
* @param integer $uid User ID
* @throws \Exception
* @todo Find type for $body/$header
*/
public static function processInbox($body, $header, int $uid)
public static function processInbox(string $body, array $header, int $uid)
{
$activity = json_decode($body, true);
if (empty($activity)) {
@ -98,7 +97,7 @@ class Receiver
$actor = JsonLD::fetchElement($ldactivity, 'as:actor', '@id');
$apcontact = APContact::getByURL($actor);
$apcontact = APContact::getByURL($actor ?? '');
if (empty($apcontact)) {
Logger::notice('Unable to retrieve AP contact for actor - message is discarded', ['actor' => $actor]);
return;