Не работает rel="canonical"

[brisks 2]

Версия Вамшоп - 1.59

Не работает атрибут - canonical

Создаются дубли страниц.

Например:

http://www.kladshop.ru/product_info.php?products_id=385

http://www.kladshop.ru/product_info.php?products_id=385&ref=yml

 

Вебмастер пишет -  Укажите роботу предпочтительный адрес с помощью 301 редиректа или атрибута rel="canonical"

 

Как исправить ошибку?

[support 447]

Так у Вас старая версия, не было ещё тогда, видимо canonical тэга.

Вот посмотрите текущую версию http://demo.vamshop.ru/samsung-galaxy-tab-3.html

Есть тэг canonical:

<link rel="canonical" href="http://demo.vamshop.ru/samsung-galaxy-tab-3.html" />

 

[brisks 2]

Да, я это подозревал.

Как мне исправить это?

 

[support 447]

Вообще, рекомендую всегда иметь свежую версию VamShop, т.е. правильнее всего - обновляться до текущей версии.

[brisks 2]

Не получается. Слишком много изменений было.

Надо ведь еще и базу было апгрейдить.

[support 447]

Тогда переносите только код canonical адресов.

В /includes/application_top.php его найти можно по canonical

и в /includes/header.php

[brisks 2]

ok

Попробую.

 

[support 447]

Удачи.

[brisks 2]

Этого не достаточно.

ПОсле подгрузки  нового  /includes/header.php

Сайт перестает работать.

Видимо еще что-то надо поменять!!!

(еще в каких-то файлах)

[support 447]

Так Вы не файлы переписывайте, а только код canonical переносите.

Понятное дело, что целиком нельзя файлы переписывать, если Вы не обновляетесь полностью.

[brisks 2]

Так я только код и переносил. (конечно)

 

[support 447]

Чудес ведь не бывает.

Значит не всё перенесли или неправильно перенесли.

Надо смотреть текст ошибки, что пишет.

[brisks 2]

Т.е. Вы хотите сказать. Что эти 2-а файла и больше нигде ничего не меняется?!

 

[support 447]

9 часов назад, brisks сказал:

Т.е. Вы хотите сказать. Что эти 2-а файла и больше нигде ничего не меняется?!

 

Насколько я помню, да.

В /includes/application_top.php прописан код функции CanonicalUrl

В /includes/header.php она используется.

[brisks 2]

Да, спасибо.

Сделал.

[support 447]

Не за что

[madvolkov 0]

а как прописывать этот canonical ?

есть страница категории например :minute.ru/canape.html и там выведены 30 товаров а отображается по 15 и при переходе на след старницу с товарами url меняется на : minute.ru/canape.html?page=2 , а в коде, что у страницы  minute.ru/canape.html cannical равен minute.ru/canape.html , что у старницы minute.ru/canape.html?page=2 canonical равен  minute.ru/canape.html?page=2 как это исправить ? 

[support 447]

23 минуты назад, madvolkov сказал:

а как прописывать этот canonical ?

есть страница категории например :minute.ru/canape.html и там выведены 30 товаров а отображается по 15 и при переходе на след старницу с товарами url меняется на : minute.ru/canape.html?page=2 , а в коде, что у страницы  minute.ru/canape.html cannical равен minute.ru/canape.html , что у старницы minute.ru/canape.html?page=2 canonical равен  minute.ru/canape.html?page=2 как это исправить ? 

В /includes/application_top.php поменяйте:

$search = '/\&sid.[^\&\?]*|\?sid.[^\&\?]*|\?sort.[^\&\?]*|\&sort.[^\&\?]*|\?direction.[^\&\?]*|\&direction.[^\&\?]*|\?on_page.[^\&\?]*|\&on_page.[^\&\?]*|\?page==1|\&page==1|\&cat.[^\&\?]*|\&filter_id.[^\&\?]*|\&manufacturers_id.[^\&\?]*|\&params.[^\&\?]*|\?q.[^\&\?]*|\&q.[^\&\?]*|\?price_min.[^\&\?]*|\&price_min.[^\&\?]*|\?price_max.[^\&\?]*|\&price_max.[^\&\?]*/'; // searches for the session id in the url

на:

$search = '/\&sid.[^\&\?]*|\?sid.[^\&\?]*|\?sort.[^\&\?]*|\&sort.[^\&\?]*|\?direction.[^\&\?]*|\&direction.[^\&\?]*|\?on_page.[^\&\?]*|\&on_page.[^\&\?]*|\?page.[^\&\?]*|\&page.[^\&\?]*|\?page==1|\&page==1|\&cat.[^\&\?]*|\&filter_id.[^\&\?]*|\&manufacturers_id.[^\&\?]*|\&params.[^\&\?]*|\?q.[^\&\?]*|\&q.[^\&\?]*|\?price_min.[^\&\?]*|\&price_min.[^\&\?]*|\?price_max.[^\&\?]*|\&price_max.[^\&\?]*/'; // searches for the session id in the url

 

[madvolkov 0]

не помогает (

http://a-la-minute.ru/canape.html?page=2

[support 447]

А точно код поменяли?!

У меня на странице http://vamshop.loc/smartphones.html?page=2

Тэг canonical такой http://vamshop.loc/smartphones.html

Вот мой файл /includes/application_top.php

application_top.php

[madvolkov 0]

Заработало ! Изменил в своём файле ток строку $search, которую взял из вашего файла !

Спасибо ! 

[support 447]

Не за что

[madvolkov 0]

снова проблема ! как исправить ? 

есть раздел с новостями...при переходе на новость в cononical отображается url данной новости а нужно что бы отображался урл каталога новостей :

http://www.actio.ru/news.php

этот вот урл можно сделать что бы отображался внутри новостей ?  

[support 447]

4 часа назад, madvolkov сказал:

снова проблема ! как исправить ? 

есть раздел с новостями...при переходе на новость в cononical отображается url данной новости а нужно что бы отображался урл каталога новостей :

http://www.actio.ru/news.php

этот вот урл можно сделать что бы отображался внутри новостей ?  

А почему это проблема?!

По-моему, правильно, что в canonical показывает url страницы новости.

Например http://www.actio.ru/proizvodstvennaja-programma-fpt-industrial-.html

Код canonical в /includes/application_top.php, внизу, в функции CanonicalUrl()

[madvolkov 0]

для того чтобы в новостях поменять canonical на news.php  сделал так вот:

условие :   if ($_GET['news_id']) ВЕРНО ?

вопрос...верно ли сделано ? нигде больше ничего править не надо ? 

function CanonicalUrl()
{                            // это для того чтобы в новостях canonical был news.php
    if ($_GET['news_id']){   
    $domain = substr((($request_type == 'SSL') ? HTTPS_SERVER : HTTP_SERVER), 0); // gets the base URL minus the trailing slash
    $string = 'news.php'; // gets the url
    $search = '/\&sid.[^\&\?]*|\?sid.[^\&\?]*|\?sort.[^\&\?]*|\&sort.[^\&\?]*|\?direction.[^\&\?]*|\&direction.[^\&\?]*|\?on_page.[^\&\?]*|\&on_page.[^\&\?]*|\?page=1|\&page=1|\&cat.[^\&\?]*|\&filter_id.[^\&\?]*|\&manufacturers_id.[^\&\?]*|\&params.[^\&\?]*|\?q.[^\&\?]*|\&q.[^\&\?]*|\?price_min.[^\&\?]*|\&price_min.[^\&\?]*|\?price_max.[^\&\?]*|\&price_max.[^\&\?]*/'; // searches for the session id in the url
    $replace = ''; // replaces with nothing i.e. deletes
    echo preg_replace($search, $replace, $string); // merges the variables and echoing them
    }
    else
    {
    $domain = substr((($request_type == 'SSL') ? HTTPS_SERVER : HTTP_SERVER), 0); // gets the base URL minus the trailing slash
    $string = $_SERVER['REQUEST_URI']; // gets the url
    $search = '/\&sid.[^\&\?]*|\?sid.[^\&\?]*|\?sort.[^\&\?]*|\&sort.[^\&\?]*|\?direction.[^\&\?]*|\&direction.[^\&\?]*|\?on_page.[^\&\?]*|\&on_page.[^\&\?]*|\?page=1|\&page=1|\&cat.[^\&\?]*|\&filter_id.[^\&\?]*|\&manufacturers_id.[^\&\?]*|\&params.[^\&\?]*|\?q.[^\&\?]*|\&q.[^\&\?]*|\?price_min.[^\&\?]*|\&price_min.[^\&\?]*|\?price_max.[^\&\?]*|\&price_max.[^\&\?]*/'; // searches for the session id in the url
    $replace = ''; // replaces with nothing i.e. deletes
    echo $domain . preg_replace($search, $replace, $string); // merges the variables and echoing them    
    }
}