Xetaravel\Livewire\Forms\AccountForm

Complexity

Total Complexity

3

Size/Duplication

Total Lines	121
Duplicated Lines	0 %

Importance

Changes	2
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	update()	0	39	2
A	load()	0	10	1

<?php

declare(strict_types=1);

namespace Xetaravel\Livewire\Forms;

use Illuminate\Support\Facades\Auth;
use Illuminate\Support\Facades\DB;
use Livewire\Attributes\Validate;
use Livewire\Features\SupportFileUploads\TemporaryUploadedFile;
use Livewire\Form;
use Xetaio\Mentions\Parser\MentionParser;
use Xetaravel\Models\Account;
use Throwable;
use Xetaravel\Models\User;

class AccountForm extends Form
{
    /**
     * The last name of the user.
     *
     * @var string|null
     */
    #[Validate('max:50')]
    public ?string $last_name = null;

    /**
     * The first name of the user.
     *
     * @var string|null
     */
    #[Validate('max:50')]
    public ?string $first_name = null;

    /**
     * The twitter of the user.
     *
     * @var string|null
     */
    #[Validate('max:50')]
    public ?string $twitter = null;

    /**
     * The facebook of the user.
     *
     * @var string|null
     */
    #[Validate('max:50')]
    public ?string $facebook = null;

    /**
     * The biography of the user.
     *
     * @var string|null
     */
    #[Validate('max:2500')]
    public ?string $biography = null;

    /**
     * The signature of the user.
     *
     * @var string|null
     */
    #[Validate('max:350')]
    public ?string $signature = null;

    /**
     * The avatar of the user.
     *
     * @var TemporaryUploadedFile|null
     */
    #[Validate('nullable|image|max:10240')]
    public ?TemporaryUploadedFile $avatar = null;

    /**
     * Load the fields.
     *
     * @return void
     */
    public function load(): void
    {
        $user = Auth::user();

        $this->first_name = $user->first_name;

Accessing first_name on the interface Illuminate\Contracts\Auth\Authenticatable suggest that you code against a concrete implementation. How about adding an instanceof check?

        $this->last_name = $user->last_name;

Accessing last_name on the interface Illuminate\Contracts\Auth\Authenticatable suggest that you code against a concrete implementation. How about adding an instanceof check?

        $this->twitter = $user->twitter;

Accessing twitter on the interface Illuminate\Contracts\Auth\Authenticatable suggest that you code against a concrete implementation. How about adding an instanceof check?

        $this->facebook = $user->facebook;

Accessing facebook on the interface Illuminate\Contracts\Auth\Authenticatable suggest that you code against a concrete implementation. How about adding an instanceof check?

        $this->biography = $user->biography;

Accessing biography on the interface Illuminate\Contracts\Auth\Authenticatable suggest that you code against a concrete implementation. How about adding an instanceof check?

        $this->signature = $user->signature;

Accessing signature on the interface Illuminate\Contracts\Auth\Authenticatable suggest that you code against a concrete implementation. How about adding an instanceof check?

    }

    /**
     * Function to create the post.
     *
     * @return Account
     *
     * @throws Throwable
     */
    public function update(): Account
    {
        return DB::transaction(function () {
            $user = User::find(Auth::id());

            $account = Account::updateOrCreate(
                [
                    'user_id' => $user->getKey(),
                ],
                [
                    'first_name' => $this->first_name,
                    'last_name' => $this->last_name,
                    'facebook' => $this->facebook,
                    'twitter' => $this->twitter,
                    'biography' => $this->biography,
                    'signature' => $this->signature,
                ]
            );

            $parser = new MentionParser($account, [

It seems like $account can also be of type Illuminate\Database\Eloquent\PendingHasThroughRelationship; however, parameter $model of Xetaio\Mentions\Parser\MentionParser::__construct() does only seem to accept Illuminate\Database\Eloquent\Model, maybe add an additional type check?

                'regex' => config('mentions.regex'),
                'mention' => false
            ]);
            $signature = $parser->parse($account->signature);

The property signature does not seem to exist on Illuminate\Database\Eloquent\PendingHasThroughRelationship.

            $biography = $parser->parse($account->biography);

The property biography does not seem to exist on Illuminate\Database\Eloquent\PendingHasThroughRelationship.

            $account->signature = $signature;
            $account->biography = $biography;
            $account->save();

            if (!is_null($this->avatar)) {
                $user->clearMediaCollection('avatar');
                $user->addMedia($this->avatar->getRealPath())
                    ->setName(mb_substr(md5($user->username), 0, 10))
                    ->setFileName(mb_substr(md5($user->username), 0, 10) . '.' . $this->avatar->getClientOriginalExtension())
                    ->toMediaCollection('avatar');
            }

            return $account;
        });
    }
}