AriasBros / eloquent-locale

src/Database/LocalizableBuilder.php

<?php

namespace Locale\Database;

use Illuminate\Database\Eloquent\Builder;
use Locale\Models\Localizable;

class LocalizableBuilder extends Builder
{
    /**
     * @var bool
     */
    protected $translationJoined = false;

    /**
     * Add an "order by" clause to the query.
     *
     * @param  string $column
     * @param  string $direction
     * @return Builder|LocalizableBuilder
     */
    public function orderBy($column, $direction = 'asc')
    {
        /** @var Localizable $localizableModel */
        $localizableModel = $this->model;

        if ($localizableModel->isLocalizableAttribute($column)) {
            $this->joinWithTranslation();
        }

        return parent::orderBy($column, $direction);

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

    }

    /**
     * @since 1.0.0
     */
    public function joinWithTranslation()
    {
        if (!$this->translationJoined) {
            /** @var Localizable $localizableModel */
            $localizableModel = $this->model;

            $localeTable = config("locale.model");
            $modelTable = $localizableModel->getTable();
            $modelKeyName = $localizableModel->getKeyName();
            $joiningTable = $localizableModel->joiningLocaleTable($localeTable, $modelTable);
            $modelForeignKey = $localizableModel->getModelForeignKey();

            $this->join($joiningTable, "{$modelTable}.{$modelKeyName}", "=", "{$joiningTable}.{$modelForeignKey}");

The method join() does not exist on Locale\Database\LocalizableBuilder. Since you implemented __call, consider adding a @method annotation.

        }
    }
}