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

Fix for account variable name

Browse source
This commit is contained in:
Michael 2022-06-10 20:32:21 +00:00
parent 9a1abda314
commit 4157f816f5
2 changed files with 28 additions and 28 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

14
src/Factory/Api/Mastodon/Account.php
View file

@ -76,22 +76,22 @@ class Account extends BaseFactory
*/ */
public function createFromUriId(int $contactUriId, $uid = 0): \Friendica\Object\Api\Mastodon\Account public function createFromUriId(int $contactUriId, $uid = 0): \Friendica\Object\Api\Mastodon\Account
{ {
$avatar = DBA::selectFirst('account-user-view', [], ['uri-id' => $contactUriId, 'uid' => [0, $uid]], ['order' => ['id' => true]]); $account = DBA::selectFirst('account-user-view', [], ['uri-id' => $contactUriId, 'uid' => [0, $uid]], ['order' => ['id' => true]]);
if (empty($avatar)) { if (empty($account)) {
throw new HTTPException\NotFoundException('Contact ' . $contactUriId . ' not found'); throw new HTTPException\NotFoundException('Contact ' . $contactUriId . ' not found');
} }
$fields = new Fields(); $fields = new Fields();
if (Contact::isLocal($avatar['url'])) { if (Contact::isLocal($account['url'])) {
$self_contact = Contact::selectFirst(['uid'], ['nurl' => $avatar['nurl'], 'self' => true]); $self_contact = Contact::selectFirst(['uid'], ['nurl' => $account['nurl'], 'self' => true]);
if (!empty($self_contact['uid'])) { if (!empty($self_contact['uid'])) {
$profileFields = $this->profileFieldRepo->selectPublicFieldsByUserId($self_contact['uid']); $profileFields = $this->profileFieldRepo->selectPublicFieldsByUserId($self_contact['uid']);
$fields = $this->mstdnFieldFactory->createFromProfileFields($profileFields); $fields = $this->mstdnFieldFactory->createFromProfileFields($profileFields);
} }
} }
return new \Friendica\Object\Api\Mastodon\Account($this->baseUrl, $avatar, $fields); return new \Friendica\Object\Api\Mastodon\Account($this->baseUrl, $account, $fields);
} }
/** /**
@ -101,10 +101,10 @@ class Account extends BaseFactory
*/ */
public function createFromUserId(int $userId): \Friendica\Object\Api\Mastodon\Account public function createFromUserId(int $userId): \Friendica\Object\Api\Mastodon\Account
{ {
$avatar = DBA::selectFirst('account-user-view', [], ['uid' => $userId, 'self' => true]); $account = DBA::selectFirst('account-user-view', [], ['uid' => $userId, 'self' => true]);
$profileFields = $this->profileFieldRepo->selectPublicFieldsByUserId($userId); $profileFields = $this->profileFieldRepo->selectPublicFieldsByUserId($userId);
$fields = $this->mstdnFieldFactory->createFromProfileFields($profileFields); $fields = $this->mstdnFieldFactory->createFromProfileFields($profileFields);
return new \Friendica\Object\Api\Mastodon\Account($this->baseUrl, $avatar, $fields); return new \Friendica\Object\Api\Mastodon\Account($this->baseUrl, $account, $fields);
} }
} }

42
src/Object/Api/Mastodon/Account.php
View file

@ -86,37 +86,37 @@ class Account extends BaseDataTransferObject
* Creates an account record from a public contact record. Expects all contact table fields to be set. * Creates an account record from a public contact record. Expects all contact table fields to be set.
* *
* @param BaseURL $baseUrl * @param BaseURL $baseUrl
* @param array $avatar entry of "account-user-view" * @param array $account entry of "account-user-view"
* @param Fields $fields Profile fields * @param Fields $fields Profile fields
* @throws \Friendica\Network\HTTPException\InternalServerErrorException * @throws \Friendica\Network\HTTPException\InternalServerErrorException
*/ */
public function __construct(BaseURL $baseUrl, array $avatar, Fields $fields) public function __construct(BaseURL $baseUrl, array $account, Fields $fields)
{ {
$this->id = (string)$avatar['pid']; $this->id = (string)$account['pid'];
$this->username = $avatar['nick']; $this->username = $account['nick'];
$this->acct = $this->acct =
strpos($avatar['url'], $baseUrl->get() . '/') === 0 ? strpos($account['url'], $baseUrl->get() . '/') === 0 ?
$avatar['nick'] : $account['nick'] :
$avatar['addr']; $account['addr'];
$this->display_name = $avatar['name']; $this->display_name = $account['name'];
$this->locked = (bool)$avatar['manually-approve']; $this->locked = (bool)$account['manually-approve'];
$this->bot = ($avatar['contact-type'] == Contact::TYPE_NEWS); $this->bot = ($account['contact-type'] == Contact::TYPE_NEWS);
$this->discoverable = !$avatar['unsearchable']; $this->discoverable = !$account['unsearchable'];
$this->group = ($avatar['contact-type'] == Contact::TYPE_COMMUNITY); $this->group = ($account['contact-type'] == Contact::TYPE_COMMUNITY);
$this->created_at = DateTimeFormat::utc($avatar['created'] ?: DBA::NULL_DATETIME, DateTimeFormat::JSON); $this->created_at = DateTimeFormat::utc($account['created'] ?: DBA::NULL_DATETIME, DateTimeFormat::JSON);
$this->note = BBCode::convertForUriId($avatar['uri-id'], $avatar['about'], BBCode::EXTERNAL); $this->note = BBCode::convertForUriId($account['uri-id'], $account['about'], BBCode::EXTERNAL);
$this->url = $avatar['url']; $this->url = $account['url'];
$this->avatar = Contact::getAvatarUrlForId($avatar['id'] ?? 0 ?: $avatar['pid'], Proxy::SIZE_SMALL, $avatar['updated'], $avatar['guid'] ?? ''); $this->avatar = Contact::getAvatarUrlForId($account['id'] ?? 0 ?: $account['pid'], Proxy::SIZE_SMALL, $account['updated'], $account['guid'] ?? '');
$this->avatar_static = $this->avatar; $this->avatar_static = $this->avatar;
$this->header = Contact::getHeaderUrlForId($avatar['id'] ?? 0 ?: $avatar['pid'], '', $avatar['updated'], $avatar['guid'] ?? ''); $this->header = Contact::getHeaderUrlForId($account['id'] ?? 0 ?: $account['pid'], '', $account['updated'], $account['guid'] ?? '');
$this->header_static = $this->header; $this->header_static = $this->header;
$this->followers_count = $avatar['ap-followers_count'] ?? $avatar['diaspora-interacted_count'] ?? 0; $this->followers_count = $account['ap-followers_count'] ?? $account['diaspora-interacted_count'] ?? 0;
$this->following_count = $avatar['ap-following_count'] ?? $avatar['diaspora-interacting_count'] ?? 0; $this->following_count = $account['ap-following_count'] ?? $account['diaspora-interacting_count'] ?? 0;
$this->statuses_count = $avatar['ap-statuses_count'] ?? $avatar['diaspora-post_count'] ?? 0; $this->statuses_count = $account['ap-statuses_count'] ?? $account['diaspora-post_count'] ?? 0;
$lastItem = $avatar['last-item'] ?: DBA::NULL_DATETIME; $lastItem = $account['last-item'] ?: DBA::NULL_DATETIME;
$this->last_status_at = $lastItem != DBA::NULL_DATETIME ? DateTimeFormat::utc($lastItem, 'Y-m-d') : null; $this->last_status_at = $lastItem != DBA::NULL_DATETIME ? DateTimeFormat::utc($lastItem, 'Y-m-d') : null;
// No custom emojis per account in Friendica // No custom emojis per account in Friendica