Почему hidden элементы видны в JsonResource?

Question

[Artem0071]

Laravel 5.6

Есть модель User
В нем есть

protected $hidden = [
        'email'
    ];

Так вот, создал UserResource (для API):

public function toArray($request)
{
  return [
     'email' => $this->email, // email отображается
     'email' => $this->resource->email, // email отображается
  ]
}

Пришлось делать следующее:

public function toArray($request)
{
  $hidden = $this->resource->getHidden();
  return [
     'email' => in_array('email', $hidden) ? null : $this->resource->email, // email не отображается
  ]
}

То есть, laravel знает что это скрытое поле, но все равно его показывает
Может я как то не так получаю параметр?

Comments

[miki131]

А зачем вы его показываете, если оно скрыто?

[Artem0071]

miki131, Мне иногда нужно его показывать (например владельцу аккаунта)
И я делаю $user->makeVisible(['email'])

[miki131]

Artem0071, попробуйте conditional-attributes

public function toArray($request)
{
    return [
        'email' => $this->when(auth()->user()->id == $this->resource->id, $this->resource->email),
    ];
}

[Вячеслав Плиско]

hidden был костылём для json сериализации до того, как во фреймворке появились ресурсы.
Так что если используете ресурсы, то лучше на hidden вовсе не смотреть и не использовать.

[Artem0071]

miki131, Причем тут это вообще?)
Я знаю про это, но я имел в виду совершенно другое
Вы снова тут используете логику именно в ресурсе, а не в контроллере

Answer 1

[Alex Wells]

Потому что вы обращаетесь к нему через магический геттер на обьекте ресурса ($this->email), который в свою очередь просто дергает тоже поле на ресурсе (модельке). $hidden у модели скрывает поле только при ->toArray(), но оно все еще есть в аттрибутах.

Все логично и так и должно работать) Выходы:
1) убирать 'email' => $this->email из ресурса
2) делать

class UserResource {
    private $includeEmail = false;

    public function toArray($request) {
        return [
            'email' => $this->when($this->includeEmail, $this->email)
        ];
    }

    public function includeEmail() {
        $this->includeEmail = true;

        return $this;
    }
}

И позже делаете return (new UserResource($model))->includeEmail(); в контроллере.

Comments

[Alex Wells]

Или совсем на крайняк, если очень хочется использовать именно $hidden, то можете скостылить свой BaseJsonResource, где напишите один метод whenNotHidden - но это плохое решение. Используйте второе и упростите себе жизнь в дальнейшем)