fomvasss / laravel-translatable

src/Traits/HasTranslations.php

<?php

namespace Fomvasss\LaravelTranslatable\Traits;

use Illuminate\Database\Eloquent\Relations\HasMany;
use Illuminate\Support\Str;
use Illuminate\Support\Collection;

trait HasTranslations
{
    use HasLanguage;

    protected static function bootHasTranslations()
    {
        static::saved(function ($model) {
            if ($model->{$model->getLangcodeColumn()} && empty($model->{$model->getTranslationUuidColumn()})) {
                $model->{$model->getTranslationUuidColumn()} = Str::uuid();
                $model->save();
            }
        });
    }

    /**
     * @return string
     */
    public function getTranslationUuidColumn(): string
    {
        return config('translatable.db.columns.translation_uuid');

The function config was not found. Maybe you did not declare it correctly or list all dependencies?

    }

    /**
     * @return \Illuminate\Database\Eloquent\Relations\HasMany
     */
    public function translations(): HasMany
    {
        return $this->hasMany(static::class, $this->getTranslationUuidColumn(), $this->getTranslationUuidColumn());

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

    }

    public function saveTranslatable(string $langcode, ?string $uuid = null)
    {
        $this->setAttribute($this->getLangcodeColumn(), $langcode);

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

        if ($uuid) {
            $this->setAttribute($this->getTranslationUuidColumn(), $uuid);
        }

        return $this->save();

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

    }

    /**
     * @return Collection
     */
    public function getTranslationList(): Collection
    {
        $languages = config('translatable.languages', []);

The function config was not found. Maybe you did not declare it correctly or list all dependencies?

        $translations = $this->translations;
        $result = new Collection();

        foreach ($languages as $langcode => $language) {
            $model = $translations->where($this->getLangcodeColumn(), $langcode)->first();
            $result->push([
                'model_key' => optional($model)->{$this->getKeyName()},

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

                'model_type' => Str::snake(class_basename(static::class)),
                'translation_uuid' => optional($model)->{$this->getTranslationUuidColumn()},
                'langcode' => $langcode,
                'language' => $language,
                'status' => $this->getTranslationStatus($langcode, $model)
            ]);
        }

        return $result;
    }

    /**
     * @param string $langcode
     * @param null $model

Are you sure the doc-type for parameter $model is correct as it would always require null to be passed?

     * @return string
     */
    protected function getTranslationStatus(string $langcode, $model = null): string
    {
        if (empty($model)) {
            return 'empty';
        }

        if ($this->{$this->getLangcodeColumn()} === $langcode) {
            return 'current';
        }

        return 'translation';
    }
}