ricardosierra / translation

src/Traits/TranslationTrait.php

<?php

namespace Translation\Traits;

trait TranslationTrait
{
    /**
     * The belongsTo locale relationship.
     *
     * @return \Illuminate\Database\Eloquent\Relations\BelongsTo
     */
    abstract public function locale();

    /**
     * The belongsTo parent relationship.
     *
     * @return \Illuminate\Database\Eloquent\Relations\BelongsTo
     */
    abstract public function parent();

    /**
     * Returns true/false if the current translation
     * record is the parent translation.
     *
     * @return bool
     */
    public function isParent()
    {
        if (!$this->getAttribute($this->getForeignKey())) {

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

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

            return true;
        }

        return false;
    }

    /**
     * Returns the translations of the current
     * translation record.
     *
     * @return mixed
     */
    public function getTranslations()
    {
        if ($this->isParent()) {
            return $this->query()->where($this->getForeignKey(), $this->getKey())->get();

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

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

        }

        return false;
    }
}