[#364] Add canonical url

[jespy666]

#364

[fey]

@jespy666 если у нас сменится домен (вряд ли но все же), то придется править и код.
Плюс домен - это обычно какая-то константа, которая есть в приложении и ее можно использовать.
Плюс смотрите у нас ссылки страниц содержат query params, т.е. почти везде ссылка на страницу создается по одному подобию - просто берется текущая ссылка и вставляется без query param

[jespy666]

@fey Принял, на данный момент сформировал URL c помошью HttpRequest.build_absolute_uri(), в параметр location как раз передал путь без параметров, жду ревью)

[fey]

@sgmdlt посмотри прчик.

[jespy666]

@fey можете переслать последний комментарий? Не отображается вложение

[fey]

@jespy666
"можно на всех страницах прописать тег canonical сам на себя - а на параметрических - соответствующие страницы без параметров"
@jespy666 т.е. можно убрать дублирование и вшить в базовй шаблон каноникал урл, чтобы он в одном месте задавался.

[jespy666]

@fey Имеется ввиду, что в базовом шаблоне сделать {% block canonical_url %}, а в шаблонах параметрических страниц уже задавать в этот блок урл без параметров?

[fey]

Можно в принципе для всех ссылок задавать канонический урл.
Просто для ссылок с параметрами это будет ссылка без параметров.
А для тех, что без параметров - та же самая ссылка. Поэтому проще всегда задавать текущую ссылку без параметров.

[fey]

т.е. мы опускаем условие if canonical_url и логику формирования этого урла выносим куда-то выше, в общее место (не знаю, можно ли в джанге использовать функции-хелперы во вьюхе, но в рельсе например можно).

[jespy666]

@fey Могу предложить еще один вариант, добавить логику формирования canonical в templatetags, а в базовом шаблоне один раз вызываем наш тег шаблона и передаем в значение canonical_url. Итого:

1. URL формируется в одном месте (в templatetags)
2. Для обычных страниц получаем обычный URL
3. Для параметризованых страниц получаем canonical без параметров

[fey]

Ну вы код присылайте. Посмотрим))

[sgmdlt]

Салют, поправьте ошибку линтера и все оки

[jespy666]

@sgmdlt Привет, поправил