Асессор/мутатор getFirstNameAttribute() не мутирует поля с числом?

Question

[Nikolino]

Laravel 6.3
У модели есть поле вида: $model->photo_1

Хочу сделать асессор, который мутирует значение поля.

public function getPhoto1Attribute($value)
{
return 'test';
}

Когда запрашиваю поле $model->photo_1, получаю исходное (немутированное) значение, хотя ожидаю увидеть 'test'.
Почему?
В доке не указано как работать с такими полями, но согласно вот этому методу, название get метода у меня верное:

public function hasGetMutator($key)
    {
        return method_exists($this, 'get'.Str::studly($key).'Attribute');
    }

Comments

[Игорь Воротнёв]

Не асессор, а аксессор, раз уж вы так решили писать. Асессор это от слова assess, а не access.

[Александр Аксентьев]

Проверил на локалке, всё отлично работает, значит что-то недоговариваете или что-то по другому делаете.

[Nikolino]

Александр Аксентьев, версия Laravel как в посте?

[Александр Аксентьев]

Nikolino, версия старая, но этот код не менялся 3 года https://github.com/laravel/framework/blame/1bbe552...

[Nikolino]

Александр Аксентьев,
Модель

class SOProperty extends Model
{
    protected $fillable = [
        'city', 'postal_code', 'address_1', 'address_2', 'street', 'description', 'avatar',
        'photo_1', 'photo_2', 'photo_3', 'photo_4', 'photo_5'
    ];
    protected $casts = [
        'business_hours' => 'array',
    ];

    public function getPhoto1Attribute($value)
    {
        return 'test';
    }

    public function getAvatarAttribute($value)
    {
        return 'test';
    }

    public function getAddress1Attribute($value)
    {
        return 'test';
    }
}

Как видим, $model->avatar мутировался, а $model->photo_1 и $model->address_1 нет.

[Александр Аксентьев]

Nikolino, Str::studly("photo_1")
Что выдаст?

Я проверял getPhoto1Attribute, у меня сработало.
Может код который ответ генерирует как-то необычно работает, и не в модели дело?

[Nikolino]

Александр Аксентьев, я проверял уже 'get'.Str::studly("photo_1").'Attribute' , выдает всё верно, то есть getPhoto1Attribute
Но почему не срабатывает, пока не могу понять. У меня моделька подгружается через eager loading, может в этом дело, хз.
То есть так:

public function index()
    {
        $salonOwners = User::where('role', UserRoles::SO)->with('SOProfile')->get();

        return response()->json(['success' => $salonOwners], 200);
    }

А в модели User так:

public function SOProfile()
    {
        return $this->hasOne(SOProperty::class);
    }

[Nikolino]

Максим Федоров, уже проверял, так и есть, строка названия метода верная, но не мутирует)

[Максим Федоров]

Nikolino,
нашел схожую проблему
https://github.com/laravel/framework/issues/112

но есть способ обойти (через массив $appends)
https://stackoverflow.com/questions/17232714/add-a...

[Nikolino]

Максим Федоров Александр Аксентьев

В общем смысл в том, что toJson() отбрасывает "виртуальные" атрибуты аксессора, то есть когда у нас нет поля $model->avatar, а мы создаем аксессор getAvatarAttribute()

Но, когда название поля содержит цифру в конце $model->photo_1, и мы используем getPhoto1Attribute(), то несмотря на то что поле photo_1 реальное, мутируется не текущее поле, а создается новое виртуальное $model->photo1, которое должно быть добавлено в массив $appends, чтобы toJson() его не отбросил.
Но вообще странное поведение, это ведь не виртуальное поле.

class SOProperty extends Model
{
    protected $fillable = [
        'city', 'postal_code', 'address_1', 'address_2', 'street', 'description', 'avatar',
        'photo_1', 'photo_2', 'photo_3', 'photo_4', 'photo_5'
    ];
    protected $casts = [
        'business_hours' => 'array',
    ];
    protected $appends = ['photo1'];

    public function getPhoto1Attribute($value)
    {
        return 'test';
    }

    public function getAvatarAttribute($value)
    {
        return 'test';
    }

    public function getAddress1Attribute($value)
    {
        return 'test';
    }
}

С тем же успехом можно было создать аксессоры типа getFirstPhotoAttribute(), getSecondPhotoAttribute() и т.д. и зааппендить их в модель, но хотелось бы мутировать именно $model->photo_1 (исходное), судя по всему сделать это нельзя.
Taylor написал что "This is intentional and for performance reasons.". То есть, чтобы не жертвовать производительностью, можно исходные поля переименовать как $model->photo1, $model->photo2 и т.д. и тогда не нужно их аппендить в модель.

[Nikolino]

Максим Федоров, это не вам, это на будущее себе и тем кто в english не силен)
Проблема там описана про кастомные атрибуты, а вот почему Laravel расценивает поля с цифрой как кастомные, это уже другой вопрос.

По итогу убрал нижние подчеркивания из полей $model->photo_1 и всё заработало без танцев с бубном и $appends.