bytic / http

src/Request/Traits/ArrayAccessTrait.php

<?php
declare(strict_types=1);

namespace Nip\Http\Request\Traits;

/**
 * Class ArrayAccessTrait
 * @package Nip\Http\Request\Traits
 */
trait ArrayAccessTrait
{
    /**
     * @param mixed $offset
     * @return bool
     */
    public function offsetExists($offset): bool
    {
        return $this->has($offset);

It seems like has() must be provided by classes using this trait. How about adding it as abstract method to this trait?

    }

    /**
     * @param mixed $offset
     * @return mixed
     */
    public function offsetGet($offset): mixed
    {
        return $this->get($offset);

It seems like get() must be provided by classes using this trait. How about adding it as abstract method to this trait?

    }

    /**
     * @param mixed $offset
     * @param mixed $value
     */
    public function offsetSet($offset, $value): void
    {
        $this->attributes->set($offset, $value);

The property attributes does not exist on Nip\Http\Request\Traits\ArrayAccessTrait. Since you implemented __get, consider adding a @property annotation.

    }

    /**
     * @param mixed $offset
     */
    public function offsetUnset($offset): void
    {
        $this->attributes->remove($offset);

The property attributes does not exist on Nip\Http\Request\Traits\ArrayAccessTrait. Since you implemented __get, consider adding a @property annotation.

    }

    /**
     * @param $name
     * @return mixed
     */
    public function __get($name)
    {
        return $this->get($name);
    }
}