Merge pull request #10467 from annando/fetch-owner-data

Use "getOwnerDataById" and "getOwnerDataByNick" to fetch owner data
This commit is contained in:
Hypolite Petovan 2021-07-04 13:41:56 -04:00 committed by GitHub
commit 54844d21a3
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
6 changed files with 36 additions and 79 deletions

View file

@ -34,7 +34,7 @@ use Friendica\DI;
use Friendica\Model\Contact; use Friendica\Model\Contact;
use Friendica\Model\Event; use Friendica\Model\Event;
use Friendica\Model\Item; use Friendica\Model\Item;
use Friendica\Model\Profile; use Friendica\Model\User;
use Friendica\Module\BaseProfile; use Friendica\Module\BaseProfile;
use Friendica\Network\HTTPException; use Friendica\Network\HTTPException;
use Friendica\Util\DateTimeFormat; use Friendica\Util\DateTimeFormat;
@ -67,7 +67,7 @@ function cal_init(App $a)
return; return;
} }
$a->profile = Profile::getByNickname($nick); $a->profile = User::getOwnerDataByNick($nick);
if (empty($a->profile)) { if (empty($a->profile)) {
throw new HTTPException\NotFoundException(DI::l10n()->t('User not found.')); throw new HTTPException\NotFoundException(DI::l10n()->t('User not found.'));
} }

View file

@ -37,7 +37,6 @@ use Friendica\Model\Contact;
use Friendica\Model\Item; use Friendica\Model\Item;
use Friendica\Model\Photo; use Friendica\Model\Photo;
use Friendica\Model\Post; use Friendica\Model\Post;
use Friendica\Model\Profile;
use Friendica\Model\Tag; use Friendica\Model\Tag;
use Friendica\Model\User; use Friendica\Model\User;
use Friendica\Module\BaseProfile; use Friendica\Module\BaseProfile;
@ -73,7 +72,7 @@ function photos_init(App $a) {
$a->profile_uid = $user['uid']; $a->profile_uid = $user['uid'];
$is_owner = (local_user() && (local_user() == $a->profile_uid)); $is_owner = (local_user() && (local_user() == $a->profile_uid));
$profile = Profile::getByNickname($nick); $profile = User::getOwnerDataByNick($nick);
$account_type = Contact::getAccountType($profile); $account_type = Contact::getAccountType($profile);

View file

@ -30,7 +30,6 @@ use Friendica\DI;
use Friendica\Model\Attach; use Friendica\Model\Attach;
use Friendica\Model\Contact; use Friendica\Model\Contact;
use Friendica\Model\Item; use Friendica\Model\Item;
use Friendica\Model\Profile;
use Friendica\Model\User; use Friendica\Model\User;
use Friendica\Module\BaseProfile; use Friendica\Module\BaseProfile;
use Friendica\Security\Security; use Friendica\Security\Security;
@ -56,7 +55,7 @@ function videos_init(App $a)
$a->data['user'] = $user[0]; $a->data['user'] = $user[0];
$a->profile_uid = $user[0]['uid']; $a->profile_uid = $user[0]['uid'];
$profile = Profile::getByNickname($nick); $profile = User::getOwnerDataByNick($nick);
$account_type = Contact::getAccountType($profile); $account_type = Contact::getAccountType($profile);

View file

@ -283,21 +283,6 @@ class Profile
return; return;
} }
/**
* Get the profile for the given nick name
*
* Includes all available profile data
*
* @param string $nickname nick
* @return array
* @throws \Exception
*/
public static function getByNickname($nickname)
{
$profile = DBA::selectFirst('owner-view', [], ['nickname' => $nickname]);
return $profile;
}
/** /**
* Formats a profile for display in the sidebar. * Formats a profile for display in the sidebar.
* *

View file

@ -27,7 +27,6 @@ use Friendica\Core\System;
use Friendica\Database\DBA; use Friendica\Database\DBA;
use Friendica\DI; use Friendica\DI;
use Friendica\Model\Contact; use Friendica\Model\Contact;
use Friendica\Model\Profile;
use Friendica\Model\User; use Friendica\Model\User;
/** /**
@ -51,7 +50,7 @@ class NoScrape extends BaseModule
System::jsonError(403, 'Authentication required'); System::jsonError(403, 'Authentication required');
} }
$profile = Profile::getByNickname($which); $profile = User::getOwnerDataByNick($which);
if (empty($profile['uid'])) { if (empty($profile['uid'])) {
System::jsonError(404, 'Profile not found'); System::jsonError(404, 'Profile not found');

View file

@ -316,73 +316,48 @@ class Transmitter
*/ */
public static function getProfile($uid) public static function getProfile($uid)
{ {
if ($uid != 0) { $owner = User::getOwnerDataById($uid);
$condition = ['uid' => $uid, 'blocked' => false, 'account_expired' => false,
'account_removed' => false, 'verified' => true];
$fields = ['guid', 'nickname', 'pubkey', 'account-type', 'page-flags'];
$user = DBA::selectFirst('user', $fields, $condition);
if (!DBA::isResult($user)) {
return [];
}
$fields = ['locality', 'region', 'country-name', 'net-publish'];
$profile = DBA::selectFirst('profile', $fields, ['uid' => $uid]);
if (!DBA::isResult($profile)) {
return [];
}
$fields = ['id', 'name', 'url', 'location', 'about', 'avatar', 'photo', 'updated'];
$contact = DBA::selectFirst('contact', $fields, ['uid' => $uid, 'self' => true]);
if (!DBA::isResult($contact)) {
return [];
}
} else {
$contact = User::getSystemAccount();
$user = ['guid' => '', 'nickname' => $contact['nick'], 'pubkey' => $contact['pubkey'],
'account-type' => $contact['contact-type'], 'page-flags' => User::PAGE_FLAGS_NORMAL];
$profile = ['locality' => '', 'region' => '', 'country-name' => '', 'net-publish' => false];
}
$data = ['@context' => ActivityPub::CONTEXT]; $data = ['@context' => ActivityPub::CONTEXT];
$data['id'] = $contact['url']; $data['id'] = $owner['url'];
if (!empty($user['guid'])) { if (!empty($owner['guid'])) {
$data['diaspora:guid'] = $user['guid']; $data['diaspora:guid'] = $owner['guid'];
} }
$data['type'] = ActivityPub::ACCOUNT_TYPES[$user['account-type']]; $data['type'] = ActivityPub::ACCOUNT_TYPES[$owner['account-type']];
if ($uid != 0) { if ($uid != 0) {
$data['following'] = DI::baseUrl() . '/following/' . $user['nickname']; $data['following'] = DI::baseUrl() . '/following/' . $owner['nick'];
$data['followers'] = DI::baseUrl() . '/followers/' . $user['nickname']; $data['followers'] = DI::baseUrl() . '/followers/' . $owner['nick'];
$data['inbox'] = DI::baseUrl() . '/inbox/' . $user['nickname']; $data['inbox'] = DI::baseUrl() . '/inbox/' . $owner['nick'];
$data['outbox'] = DI::baseUrl() . '/outbox/' . $user['nickname']; $data['outbox'] = DI::baseUrl() . '/outbox/' . $owner['nick'];
} else { } else {
$data['inbox'] = DI::baseUrl() . '/friendica/inbox'; $data['inbox'] = DI::baseUrl() . '/friendica/inbox';
} }
$data['preferredUsername'] = $user['nickname']; $data['preferredUsername'] = $owner['nick'];
$data['name'] = $contact['name']; $data['name'] = $owner['name'];
if (!empty($profile['country-name'] . $profile['region'] . $profile['locality'])) { if (!empty($owner['country-name'] . $owner['region'] . $owner['locality'])) {
$data['vcard:hasAddress'] = ['@type' => 'vcard:Home', 'vcard:country-name' => $profile['country-name'], $data['vcard:hasAddress'] = ['@type' => 'vcard:Home', 'vcard:country-name' => $owner['country-name'],
'vcard:region' => $profile['region'], 'vcard:locality' => $profile['locality']]; 'vcard:region' => $owner['region'], 'vcard:locality' => $owner['locality']];
} }
if (!empty($contact['about'])) { if (!empty($owner['about'])) {
$data['summary'] = BBCode::convert($contact['about'], false); $data['summary'] = BBCode::convert($owner['about'], false);
} }
$data['url'] = $contact['url']; $data['url'] = $owner['url'];
$data['manuallyApprovesFollowers'] = in_array($user['page-flags'], [User::PAGE_FLAGS_NORMAL, User::PAGE_FLAGS_PRVGROUP]); $data['manuallyApprovesFollowers'] = in_array($owner['page-flags'], [User::PAGE_FLAGS_NORMAL, User::PAGE_FLAGS_PRVGROUP]);
$data['discoverable'] = $profile['net-publish']; $data['discoverable'] = $owner['net-publish'];
$data['publicKey'] = ['id' => $contact['url'] . '#main-key', $data['publicKey'] = ['id' => $owner['url'] . '#main-key',
'owner' => $contact['url'], 'owner' => $owner['url'],
'publicKeyPem' => $user['pubkey']]; 'publicKeyPem' => $owner['pubkey']];
$data['endpoints'] = ['sharedInbox' => DI::baseUrl() . '/inbox']; $data['endpoints'] = ['sharedInbox' => DI::baseUrl() . '/inbox'];
$data['icon'] = ['type' => 'Image', 'url' => Contact::getAvatarUrlForId($contact['id'], '', $contact['updated'])]; $data['icon'] = ['type' => 'Image', 'url' => Contact::getAvatarUrlForId($owner['id'], '', $owner['updated'])];
$resourceid = Photo::ridFromURI($contact['photo']); $resourceid = Photo::ridFromURI($owner['photo']);
if (!empty($resourceid)) { if (!empty($resourceid)) {
$photo = Photo::selectFirst(['type'], ["resource-id" => $resourceid]); $photo = Photo::selectFirst(['type'], ["resource-id" => $resourceid]);
if (!empty($photo['type'])) { if (!empty($photo['type'])) {
@ -390,10 +365,10 @@ class Transmitter
} }
} }
if (!empty($contact['header'])) { if (!empty($owner['header'])) {
$data['image'] = ['type' => 'Image', 'url' => Contact::getHeaderUrlForId($contact['id'], '', $contact['updated'])]; $data['image'] = ['type' => 'Image', 'url' => Contact::getHeaderUrlForId($owner['id'], '', $owner['updated'])];
$resourceid = Photo::ridFromURI($contact['header']); $resourceid = Photo::ridFromURI($owner['header']);
if (!empty($resourceid)) { if (!empty($resourceid)) {
$photo = Photo::selectFirst(['type'], ["resource-id" => $resourceid]); $photo = Photo::selectFirst(['type'], ["resource-id" => $resourceid]);
if (!empty($photo['type'])) { if (!empty($photo['type'])) {
@ -1302,7 +1277,7 @@ class Transmitter
'mediaType' => $attachment['mimetype'], 'mediaType' => $attachment['mimetype'],
'url' => $attachment['url'], 'url' => $attachment['url'],
'name' => $attachment['description']]; 'name' => $attachment['description']];
if (!empty($attachment['height'])) { if (!empty($attachment['height'])) {
$attach['height'] = $attachment['height']; $attach['height'] = $attachment['height'];
} }
@ -1310,7 +1285,7 @@ class Transmitter
if (!empty($attachment['width'])) { if (!empty($attachment['width'])) {
$attach['width'] = $attachment['width']; $attach['width'] = $attachment['width'];
} }
if (!empty($attachment['preview'])) { if (!empty($attachment['preview'])) {
$attach['image'] = $attachment['preview']; $attach['image'] = $attachment['preview'];
} }
@ -1334,7 +1309,7 @@ class Transmitter
'mediaType' => $attachment['mimetype'], 'mediaType' => $attachment['mimetype'],
'url' => $attachment['url'], 'url' => $attachment['url'],
'name' => $attachment['description']]; 'name' => $attachment['description']];
if (!empty($attachment['height'])) { if (!empty($attachment['height'])) {
$attach['height'] = $attachment['height']; $attach['height'] = $attachment['height'];
} }
@ -1347,7 +1322,7 @@ class Transmitter
$attach['image'] = $attachment['preview']; $attach['image'] = $attachment['preview'];
} }
$attachments[] = $attach; $attachments[] = $attach;
} }
// Currently deactivated, since it creates side effects on Mastodon and Pleroma. // Currently deactivated, since it creates side effects on Mastodon and Pleroma.
// It will be activated, once this cleared. // It will be activated, once this cleared.