mostafaznv / larupload

src/Concerns/BaseLarupload.php

<?php

namespace Mostafaznv\Larupload\Concerns;

use Mostafaznv\Larupload\Storage\Attachment;
use Mostafaznv\Larupload\Storage\Proxy\AttachmentProxy;

trait BaseLarupload
{
    protected static string $laruploadNull;

    private bool $hideLaruploadColumns;

    /**
     * @var Attachment[]
     */
    private array $attachments = [];

    protected function initializeLarupload(): void
    {
        $this->hideLaruploadColumns = config('larupload.hide-table-columns');

        $this->attachments = $this->attachments();
        $table = $this->getTable();

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

        foreach ($this->attachments as $attachment) {
            $attachment->folder($table);
        }
    }

    public static function bootLarupload(): void
    {
        static::$laruploadNull = crc32(time());

        if (!defined('LARUPLOAD_NULL')) {
            define('LARUPLOAD_NULL', static::$laruploadNull);
        }
    }

    /**
     * Get the entities should upload into the model
     *
     * @return Attachment[]
     */
    abstract public function attachments(): array;
}