Fix for account variable name

diff --git a/src/Factory/Api/Mastodon/Account.php b/src/Factory/Api/Mastodon/Account.php
index cffd5d8924ff75a0cedde15b6485087a47254a4c..d0c654fc21767640d09e35c33f91145b73aea440 100644 (file)
--- a/src/Factory/Api/Mastodon/Account.php
+++ b/src/Factory/Api/Mastodon/Account.php
@@ -76,22 +76,22 @@ class Account extends BaseFactory
         */
        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]]);
-               if (empty($avatar)) {
+               $account = DBA::selectFirst('account-user-view', [], ['uri-id' => $contactUriId, 'uid' => [0, $uid]], ['order' => ['id' => true]]);
+               if (empty($account)) {
                        throw new HTTPException\NotFoundException('Contact ' . $contactUriId . ' not found');
                }
 
                $fields = new Fields();
 
-               if (Contact::isLocal($avatar['url'])) {
-                       $self_contact = Contact::selectFirst(['uid'], ['nurl' => $avatar['nurl'], 'self' => true]);
+               if (Contact::isLocal($account['url'])) {
+                       $self_contact = Contact::selectFirst(['uid'], ['nurl' => $account['nurl'], 'self' => true]);
                        if (!empty($self_contact['uid'])) {
                                $profileFields = $this->profileFieldRepo->selectPublicFieldsByUserId($self_contact['uid']);
                                $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
        {
-               $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);
                $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);
        }
 }

diff --git a/src/Object/Api/Mastodon/Account.php b/src/Object/Api/Mastodon/Account.php
index 89e3975d61c18e487ee5ded5836c63359199485d..3369a18c00a6855106d2efc5cd794fb3d10db8a4 100644 (file)
--- a/src/Object/Api/Mastodon/Account.php
+++ b/src/Object/Api/Mastodon/Account.php
@@ -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.
         *
         * @param BaseURL $baseUrl
-        * @param array   $avatar entry of "account-user-view"
+        * @param array   $account entry of "account-user-view"
         * @param Fields  $fields  Profile fields
         * @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->username        = $avatar['nick'];
+               $this->id              = (string)$account['pid'];
+               $this->username        = $account['nick'];
                $this->acct            =
-                       strpos($avatar['url'], $baseUrl->get() . '/') === 0 ?
-                               $avatar['nick'] :
-                               $avatar['addr'];
-               $this->display_name    = $avatar['name'];
-               $this->locked          = (bool)$avatar['manually-approve'];
-               $this->bot             = ($avatar['contact-type'] == Contact::TYPE_NEWS);
-               $this->discoverable    = !$avatar['unsearchable'];
-               $this->group           = ($avatar['contact-type'] == Contact::TYPE_COMMUNITY);
+                       strpos($account['url'], $baseUrl->get() . '/') === 0 ?
+                               $account['nick'] :
+                               $account['addr'];
+               $this->display_name    = $account['name'];
+               $this->locked          = (bool)$account['manually-approve'];
+               $this->bot             = ($account['contact-type'] == Contact::TYPE_NEWS);
+               $this->discoverable    = !$account['unsearchable'];
+               $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->url             = $avatar['url'];
-               $this->avatar          = Contact::getAvatarUrlForId($avatar['id'] ?? 0 ?: $avatar['pid'], Proxy::SIZE_SMALL, $avatar['updated'], $avatar['guid'] ?? '');
+               $this->note            = BBCode::convertForUriId($account['uri-id'], $account['about'], BBCode::EXTERNAL);
+               $this->url             = $account['url'];
+               $this->avatar          = Contact::getAvatarUrlForId($account['id'] ?? 0 ?: $account['pid'], Proxy::SIZE_SMALL, $account['updated'], $account['guid'] ?? '');
                $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->followers_count = $avatar['ap-followers_count'] ?? $avatar['diaspora-interacted_count'] ?? 0;
-               $this->following_count = $avatar['ap-following_count'] ?? $avatar['diaspora-interacting_count'] ?? 0;
-               $this->statuses_count  = $avatar['ap-statuses_count'] ?? $avatar['diaspora-post_count'] ?? 0;
+               $this->followers_count = $account['ap-followers_count'] ?? $account['diaspora-interacted_count'] ?? 0;
+               $this->following_count = $account['ap-following_count'] ?? $account['diaspora-interacting_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;
 
                // No custom emojis per account in Friendica