thinktomorrow / assetlibrary

src/AssetTrait.php

<?php

namespace Thinktomorrow\AssetLibrary;

use Spatie\MediaLibrary\InteractsWithMedia;
use Illuminate\Database\Eloquent\Relations\MorphToMany;
use Illuminate\Support\Collection;

trait AssetTrait
{
    use InteractsWithMedia;

    public static function bootAssetTrait()
    {
        static::deleted(function ($model) {

The method deleted() does not exist on Thinktomorrow\AssetLibrary\AssetTrait. Did you maybe mean deleteMedia()?

            $model->assetRelation->each(function ($asset) use ($model) {
                $model->assetRelation()->updateExistingPivot($asset->id, ['unused'=> true]);
            });
        });
    }

    public function assetRelation(): MorphToMany
    {
        return $this->morphToMany(Asset::class, 'entity', 'asset_pivots')->withPivot('type', 'locale', 'order')->orderBy('order');

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

    }

    public function asset(string $type, ?string $locale = null): ?Asset
    {
        return $this->assets($type, $locale)->first() ?? new NullAsset();
    }

    public function assets(?string $type = null, ?string $locale = null): Collection
    {
        $assets = $this->assetRelation;

        if ($type) {
            $assets = $assets->where('pivot.type', $type);
        }

        $locale = $locale ?? app()->getLocale();

The method getLocale() does not exist on Illuminate\Container\Container. Are you sure you never get this type here, but always one of the subclasses?

        $results = $assets->filter(function ($asset) use ($locale) {
            return $asset->pivot->locale == $locale;
        });

        if ($this->getUseAssetFallbackLocale() && $locale != $this->getAssetFallbackLocale() && $results->isEmpty()) {
            $results = $assets->filter(function ($asset) {
                return $asset->pivot->locale == $this->getAssetFallbackLocale();
            });
        }

        return $results->sortBy('pivot.order');
    }

    protected function getUseAssetFallbackLocale(): bool
    {
        return $this->useAssetFallbackLocale ?? config('thinktomorrow.assetlibrary.use_fallback_locale', false);
    }

    protected function getAssetFallbackLocale(): string
    {
        if (! config('thinktomorrow.assetlibrary.fallback_locale')) {
            config()->set('thinktomorrow.assetlibrary.fallback_locale', config('app.fallback_locale'));
        }

        return $this->assetFallbackLocale ?? config('thinktomorrow.assetlibrary.fallback_locale');
    }
}