cino / laravel-chronos

src/Eloquent/Chronos.php

<?php

namespace Cino\LaravelChronos\Eloquent;

use Cino\LaravelChronos\Eloquent\Concerns\ChronosRelations;
use Cino\LaravelChronos\Eloquent\Concerns\ChronosTimestamps;
use Cino\LaravelChronos\Eloquent\Relations\Pivot;
use Illuminate\Database\Eloquent\Model as BaseModel;
use Illuminate\Support\Collection;

/**
 * @mixin \Illuminate\Database\Eloquent\Model
 */
trait Chronos
{
    use ChronosRelations;
    use ChronosTimestamps;

    /**
     * Create a new pivot model instance.
     *
     * @param \Illuminate\Database\Eloquent\Model $parent
     * @param array $attributes
     * @param string $table
     * @param bool $exists
     * @param string|null $using
     * @return \Cino\LaravelChronos\Eloquent\Relations\Pivot
     */
    public function newPivot(BaseModel $parent, array $attributes, $table, $exists, $using = null)
    {
        return $using ? $using::fromRawAttributes($parent, $attributes, $table, $exists)
            : Pivot::fromAttributes($parent, $attributes, $table, $exists);
    }

    /**
     * Reload the current model instance with fresh attributes from the database.
     *
     * @return \Illuminate\Database\Eloquent\Model
     */
    public function refresh()
    {
        if (!$this->exists) {
            return $this;

The expression return $this returns the type Cino\LaravelChronos\Eloquent\Chronos which is incompatible with the documented return type Illuminate\Database\Eloquent\Model.

        }

        $this->setRawAttributes(

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

            static::newQueryWithoutScopes()->findOrFail($this->getKey())->attributes

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

        );

        $this->load(Collection::make($this->relations)->reject(function ($relation) {

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

            return $relation instanceof Pivot;
        })->keys()->all());

        $this->syncOriginal();

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

        return $this;

The expression return $this returns the type Cino\LaravelChronos\Eloquent\Chronos which is incompatible with the documented return type Illuminate\Database\Eloquent\Model.

    }
}