Как поправить костыль с реляцией и where в yii2?

Question

[nepster-web]

Есть 3 таблицы:
- Статьи
- Теги
- Связи (Тег-Статья)

У меня была задача показать все записи конкретного тега. Я написал реляцию и наткнулся на проблему.
Таблицы Статьи и Теги содержат после URL, поэтому при написании:

$query->innerJoinWith(['tagsAll'])->andWhere(['url'=>$tag]);

Возникает ошибка:
Integrity constraint violation: 1052 Column 'url' in where clause is ambiguous

Поэтому я подключил модель тегов и сделал как-то так:

$query->innerJoinWith(['tagsAll'])->andWhere([Tags::tableName().'.url'=>$tag]);

При таком варианте все работает. Но я просто жопой чую, что это слоупочный костыль. Подскажите, пожалуйста, можно ли решить проблему более оптимальным вариантом?

Answer 1

[Сергей Протько]

проблема решается алиасами таблиц.

В yii1 по дефолту таблице, которая идет в from присваивался алиас t, для таблиц, которые вы прописываете джойном можно явно указать алиас. И тогда в where можно будет писать
andWhere('t.url=:url')

Но в Yii1 были проблемы с этим. Если к таблице ничего не джойнится, то и алиас не назначается, а значит запросы, где проставлены элиасы падают. Это решалось так же кастылями (как и любая проблема Yii). Решили ли они эту проблему во второй ветке я не знаю, но помниться мне реализация AR второй версии не сильно далеко ушла от реализации первой версии. Это была одна из причин по которой я ушел с Yii - изменения были анонсированы больше двух лет назад, зарелижены только сейчас.

p.s. используйте биндинг параметров.

Comments

[nepster-web]

@Fesor

тут запрос торчит вообще без алиасов:

SQLSTATE[23000]: Integrity constraint violation: 1052 Column 'url' in where clause is ambiguous
The SQL being executed was: SELECT COUNT(*) FROM `Pr715_articles` INNER JOIN `Pr715_articles_tags` ON `Pr715_articles`.`article_id` = `Pr715_articles_tags`.`article_id` INNER JOIN `Pr715_tags` ON `Pr715_articles_tags`.`tag_id` = `Pr715_tags`.`tag_id` WHERE `url`='friends'

[Сергей Протько]

@nepster09, mysql не знает из какой таблицы брать это поле, потому и нужно указать алиас.

[Сергей Протько]

@nepster09, короче, вместо названия таблицы укажите алиас t, должно заработать. Так же подучить SQL и почитайте документацию в кой-то веке. Не дело это использовать фреймворк и не пытаться даже его изучать (документация, исходники, благо исходники yii можно разобрать полностью за пару недель неспешного изучения)

[nepster-web]

@Fesor да в том, то и дело, что я дописал t и тогда запрос получается

SELECT COUNT(*) FROM `Pr715_articles` INNER JOIN `Pr715_articles_tags` ON `Pr715_articles`.`article_id` = `Pr715_articles_tags`.`article_id` INNER JOIN `Pr715_tags` ON `Pr715_articles_tags`.`tag_id` = `Pr715_tags`.`tag_id` WHERE t.`url`='friends'

алиас изначально не прописан. Я даже в доках не могу найти пример .

[Сергей Протько]

это не алиасы...

SELECT COUNT(*) FROM `Pr715_articles` t INNER JOIN `Pr715_articles_tags` t1 ON t.`article_id` = t1.`article_id` INNER JOIN `Pr715_tags` t2 ON t1.`tag_id` = t2.`tag_id` WHERE t.`url`='friends'

Что-то в этом духе. Если в документации не можете найти, поищите примеры в ишус трекера: #1791, #2687 ну и т.д.

[nepster-web]

@Fesor Спасибо, накопал. Таки вышло:

public function getTagsAll()
{
return $this->hasMany(Tags::className(), ['tag_id' => 'tag_id'])->from(['tagsTable' => Tags::tableName()])->viaTable(ArticlesTags::tableName(), ['article_id' => 'article_id']);
}

$query->innerJoinWith(['tagsAll'])->andWhere(['tagsTable.url'=>$tag]);

[Сергей Протько]

@nepster09, но вы всеравно используете полное имя таблицы... это не сосем правильно.

[nepster-web]

@Fesor всмысле? я так назвал алиас. Можно было конечно просто t, но там более понятно, какая именно таблица используется.

[Сергей Протько]

@nepster09, как хотите... А вообще поработали бы вы с нормальными ORM типа Doctrine, с которыми никогда нету таких проблем.

Answer 2

[Nikita Permin]

Для связей подобного рода (все записи тега, или все теги записи с использованием промежуточных таблиц) вам наверное лучше будет использовать viaTable.
для модели Article получение тегов будет выглядеть так:

public function getTags()
{
	return $this->hasMany(Tags::className(), ['id' => 'tag_id'])
		->viaTable('{{%article_tag}}', ['article_id' => 'id']);
}

Ну и для тега (в модели Tags) получение статей, использующих этот тег:

public function getArticles()
{
	return $this->hasMany(Article::className(), ['id' => 'article_id'])
		->viaTable('{{%article_tag}}', ['tag_id' => 'id']);
}

Использовать можно будет так:
$tag->articles;//результат - массив моделей статей
где $tag - например $tag = Tags::findOne(['url' => $tag), естественно после findOne не забываем проверить $tag, чтобы при случае его отсутствия не получить UnknownPropertyException

Comments

[nepster-web]

я так и использую. Только я делаю выборку не по tag_id, а по url. Ссылка вида: site.ru/acticles/friends. А поле url есть и в записях и в тегах, поэтому конфликтует.

[Сергей Протько]

@nepster09, алиасы таблиц проставьте явно в запросах.

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

Nikita Permin
Доброго времени !
Вывожу посты по тегу по Вашему примеру
$model = Tags::find()->andWhere(['slug'=> Yii::$app->getRequest()->getQueryParam('slug') ])->one();
А как теперь в Контроллере прописать чтобы еще и пагинация работала ?
Мой метод не работает
$pagination = new Pagination([
'defaultPageSize' => 3,
'totalCount' => $model->posti->count()
]);

$model = $model->posti->orderBy(['id' => SORT_DESC])
->offset($pagination->offset)
->limit($pagination->limit)
->all();
Спасибо.

Answer 3

[Сергей]

Прошу прощения, может я не понял. А нельзя сначала вытащить в контроллере теги из урла, а потом передать их в модель, в которой будет уже поиск делаться?
$tags = self::find()->where(['like', 'name', $keyword])->all();,
Ну и так далее. Или я не о том?

Comments

[nepster-web]

там 3 таблицы (еще одна таблица связей тег-статья), от лайков я отказался. Тогда все отлично работает в 1 запрос.

[Сергей]

А почему теги не писать в таблицу статей? с разделителем? а потом вытаскивать.

[nepster-web]

@TsarS Как оптимально реализовать теги к статье?

[Сергей Протько]

@TsarS, потому что нормализация данных