Переопределить метод find через функцию в Модели?

Question

[Роман Савицкий]

Всем Добрый День !
Подключил вот это расширение: https://github.com/OmgDef/yii2-multilingual-behavior
В modules\catalog\models\Item.php - подключил поведение
Автор этого расширения просить, в модели переопределить метод find(), вот так

public static function find()
    {
        return new MultilingualQuery(get_called_class());
    }

Либо же в моем случае, чтобы выбрать все переводы нужно так сделать

public static function find()
    {
$q = new MultilingualQuery(get_called_class());
$q->multilingual();
return $q;
}

Затем если в контроллере сделать так:
$model = Item::findOne($id);
Все переводы моей записи подтягиваются.

Все это конечно хорошо, но переопределив метод find() таким образом в модели, у меня начали вылазить другие не нужные мне ошибки в других местах сайта где я тоже использую класс Item.
Вопрос, как это все организовать через некую функцию в классе Item, чтобы потом в контроллере, вот так допустим запись из БД доставать ?

$model = new Item;
$model = $model->findMultilingual($id);

Заранее спасибо !

Comments

[Максим Тимофеев]

Все это конечно хорошо, но переопределив метод find() таким образом в модели, у меня начали вылазить другие не нужные мне ошибки в других местах сайта где я тоже использую класс Item

Ошибки покажите?

Вопрос, как это все организовать через некую функцию в классе Item, чтобы потом в контроллере, вот так допустим запись из БД доставать ?

Там все это реализовано в поведении. Так что вместо изобретения велосипеда, исправьте ошибки.

[Роман Савицкий]

Максим Тимофеев
Вот такая ошибка:
Calling unknown method: omgdef\multilingual\MultilingualQuery::sortDate()

[Максим Тимофеев]

Ну так нет у MultilingualQuery метода sortDate. Чего Вы вообще обращаетесь к классу MultilingualQuery, если надо к своей модели обращаться.

[Роман Савицкий]

Максим Тимофеев, Я к своей модели и обращаюсь, и в ней же как написал выше, переопределил find(), как сказал автор, вот таким образом:

$q = new MultilingualQuery(get_called_class());
$q->multilingual();
return $q;

И вот такие ошибки и выскакивают.

[matperez]

Роман Савицкий, а до вашего вмешательства в модели был уже переопределен метод find()?

Если да, то найдите верните все как было, найдите класс, который ранее возвращался в методе find() и добавьте туда трейт MultilingualTrait
use MultilingualTrait;

[Роман Савицкий]

matperez, Доброго !
В свою модель записей Item.php, я только вот это добавил:

public static function find(){
$q = new MultilingualQuery(get_called_class());
$q->multilingual();
return $q;
}

///а до вашего вмешательства в модели был уже переопределен метод find()?
Нет, ничего переопределено не было, я переопределил исходя из мануала автора, чтобы переводы из БД все подгрузить.
Про это не понял, куда это нужно прописать ?
use MultilingualTrait;
в мой Класс в файле Item.php ? и все, переопределять ничего не нужно ? А как тогда мне все переводы подгрузить ?

[matperez]

Роман Савицкий, метода sortDate() на который ваш код ругается абсолютно точно нет в стандартном классе ActiveQuery, который возвращается по умолчанию из метода find(). Это значит, что где-то у вас в модели метод find() был переопределен из из него возвращался не ActiveQuery, а какой-нибудь его наследник. Ищите. Возможно ваша модель Item тоже наследует не ActiveRecord а какой-нибудь другой класс, к котором метод find() переопределен.

[Роман Савицкий]

matperez, Item у меня наследует от ActiveRecord,

class Item extends \yii\easyii\components\ActiveRecord

Все ок, я не понял что делать с MultilingualTrait.

[Максим Тимофеев]

В доках:

public static function find()
    {
        return new MultilingualQuery(get_called_class());
    }

у Вас:

public static function find(){
  $q = new MultilingualQuery(get_called_class());
  $q->multilingual();
  return $q;
}

Как бы совсем не так.
Зачем метод multilingual дернули? Хотя не понимаю, как он эту ошибку мог вызвать. Видимо в другом месте проблема. И скорее всего в том, что метод getImems есть в Вашей моделе и в Multilingual. Так что думаю надо переименовать в Вашей моделе getImems и будет Вам счастье.

[matperez]

Роман Савицкий, \yii\easyii\components\ActiveRecord ну вот. А штатный класс yii\db\ActiveRecord. Смотрите что возвращает метод find в \yii\easyii\components\ActiveRecord и после этого решайте.

Про трейты почитайте в гугле. Они позволяют добавить функциональность к классу без наследования.
https://habrahabr.ru/post/216955/
https://habrahabr.ru/post/258671/

Answer 1

[matperez]

Вам нужно создать свой класс. Какой-нибудь ItemQuery. Наследовать его от yii\easyii\components\ActiveQuery и добавить туда трейт MultilingualTrait и его уже возвращать из Item::find(). И будет вам счатье.

Comments

[Роман Савицкий]

Спасибо за дельные советы, буду пробовать.

[Роман Савицкий]

Получилось, а я мучался, сразу видно что Вы профессионал !)
В yii\easyii\components\ActiveRecord, действительно было переопределение find такое

public static function find(){
       return new ActiveQuery(get_called_class());
}

В расширении /yii2-multilingual-behavior/ везде заменил
use yii\db\ActiveQuery;
на
use yii\easyii\components\ActiveQuery;
В модели Item переопределил find(), и добавил функцию для выбора переводов записи

public static function find()
    {
        return new MultilingualQuery(get_called_class());
    }

    public function findMultilingual($item_id)
    {
        return $this::find()->multilingual()->where(['item_id' => $item_id])->one();
    }

Все ЗАРАБОТАЛО, Благодарю !!!

[matperez]

Роман Савицкий, это где вы yii2-multilingual-behavior изменили? в папке vendor?
нельзя так делать. создайте свой класс как я написал.
в папке vendor вообще менять код нельзя, он чужой.

[Роман Савицкий]

matperez, Хорошо, подправлю по правилам.

[Роман Савицкий]

matperez, Доброго, подскажите, из за чего такое может быть ?
При заходе на страницу Shop, перевод которой я организовал, в html lang, пусто, хотя на других страницах все Ок отображается.


Из за этого ссылки едут.

[matperez]

Роман Савицкий, не знаю. я этим продуктом не пользуюсь.
На вашем месте, я бы проверил шаблон (layout), чтобы выяснить откуда вообще берется значение для lang, а потом проанализировал был контроллер, чтобы понять какое конкретно значение должно туда попасть. А после уже бы решал как его туда вернуть.

[Роман Савицкий]

matperez, Разобрался, лишняя строка была в контроллере, переопределяло заново
Yii::$app->language