cino / laravel-chronos

src/Eloquent/Concerns/ChronosInteractsWithPivotTable.php

<?php

namespace Cino\LaravelChronos\Eloquent\Concerns;

use Illuminate\Database\Eloquent\Relations\Pivot;

trait ChronosInteractsWithPivotTable
{
    /**
     * Get the pivot models that are currently attached.
     *
     * @return \Illuminate\Support\Collection
     */
    protected function getCurrentlyAttachedPivots()
    {
        return $this->newPivotQuery()->get()->map(function ($record) {

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

            $class = $this->using ? $this->using : Pivot::class;

            $pivot = $class::fromRawAttributes($this->parent, (array)$record, $this->getTable(), true);

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

            return $pivot->setPivotKeys($this->foreignPivotKey, $this->relatedPivotKey);
        });
    }
}