Яндекс.Вебмастер -> Товары и цены

[Wizard]

Денис, будет ли исправлен в последующих версиях баг с Яндекс.Вебмастер -> Товары и цены?

Проблема заключается в том что яндекс индексирует товар так:
http://demo.simplacms.ru/products/samsung-s5570-galaxy-mini

а мы передаем xml так:
http://demo.simplacms.ru/products/samsung-s5570-galaxy-mini?variant=12

в следствии - яндекс выдает ошибку.

проблема решается легко, к примеру:

$simpla->db->query("SELECT p.id, v.price...$variant[$p->id] = isset($variant[$p->id]) ? '?variant='.$p->variant_id : '';...<url>".$simpla->config->root_url.'/products/'.$p->url.$variant[$p->id]."</url>....

[pikusov]

Не совсем понял суть проблемы

?variant=12 передаётся яндекс-маркету, так как у него есть требование, чтобы при переходе человек попадал именно на выбранный вариант

[Wizard]

в поиске присутствует именно такая ссылка
http://demo.simplacms.ru/products/samsung-s5570-galaxy-mini

Яндекс.Вебмастер -> Товары и цены ни как не может автоматически привязать к этой ссылке ?variant=12

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

***так как у него есть требование, чтобы при переходе человек попадал именно на выбранный вариант

Именно этого он и требует
http://demo.simplacms.ru/products/samsung-s5570-galaxy-mini - это и есть основной вариант, тот на которой пользователь попадает переходя по ссылке как внутри сайта так и на поиске.

[Wizard]

Если мы на сайте во всех ссылках товара будем указывать ид варианта, тогда проблемы не будет, но по моему проще в xml не выводить параметр ?variant=12 там где вариант стоит по умолчанию - то есть первый.

[Wizard]

ссылка на поиске именно такая - http://www.zdoroff.com/products/SWB30
а мы передаем -http://www.zdoroff.com/products/SWB30?varian=522

это две разные страницы, как яндекс может привязать одно к другому?

и потому для обладателей симплы подобное не возможно:



Для Маркета это катит но не для Товары и цены - это разные вещи

просто скорее всего мало кто сталкивался с этой проблемой, так как для выгрузки надо не меньше 30 тиц

[pikusov]

То есть в yandex.xml просто не указывать variant_id для первых вариантов товара? Надо думать как это сделать

[Wizard]

я показал как это сделать в первом посту - проверенно, работает!

[pikusov]

Я непонял как оно работает честно говоря, что за переменная $variant например

[pikusov]

Попробуйте так http://pastebin.com/S6bkyxWi
Но я не уверен что добавление ORDER BY p.id, v.position не замедлит работу

[Wizard]

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

[pikusov]

А, вроде понял.
По сути вы проверяете, выводились ли уже варианты этого товара, если нет - то добавлять не нужно?
Это не будет работать, к сожалению

[pikusov]

Потому что не факт что:
1) подряд идут варианты одного и того же товара
2) первым идет вариант который действительно первый по полю position

[Wizard]

*** ORDER BY p.id, v.position
это конечно надо добавить, для актуальной релевантности вариантов, то как они стоят в выводе на сайте, так как нам необходима именно первый вариант

[pikusov]

Да, я так и сделал по ссылке
> Попробуйте так http://pastebin.com/S6bkyxWi

У вас много товаров? Вы можете сравнить скорость старого и нового запроса?

[Wizard]

*** Это не будет работать, к сожалению

перед тем как создавать тему я проверил))) - работает

[Wizard]

товаров не много, я не думаю что для яндекса не будет критична разница в скорости загрузки xml в 0,1 сек

[pikusov]

Попробуйте поменять местави варианты в админке, скорее появится ошибка если не сделать ORDER BY p.id, v.position. В общем мы друг друга поняли

[pikusov]

> товаров не много, я не думаю что для яндекса не будет критична разница в скорости загрузки xml в 0,1 сек
Конечно, просто хотел добавить это в след релиз, но как бы оно не начало тормозить в магазинах, где много товаров

[Wizard]

ошибка, все работает по приведенному коду...

[Wizard]

***Конечно, просто хотел добавить это в след релиз, но как бы оно не начало тормозить в магазинах, где много товаров

ок, я сейчас протестирую

[pikusov]

Почему не работает?

[Wizard]

все, все работает - это я проморгал

[Wizard]

Денвер

товаров - 1634
вариантов - 5219

ДО memory peak usage: 3143416 bytes, page generation time: 2.966 seconds
ПОСЛЕ memory peak usage: 3326936 bytes, page generation time: 2.16 seconds

[Wizard]

даже не знаю почему после дополнения кода time меньше. перепроверил два раза.

[pikusov]

Вы сделали так?
$variant[$p->id] = isset($variant[$p->id]) ? '?variant='.$p->variant_id : '';

Просто нет смысла хранить этот массив, так что лучше возьмите мой вариант где для этого используется одна переменная вместо массива, будет кушать мешьше памяти.
На счет времени - лучше сделайте print этого запроса и посмотрите время через phpmyadmin, чтобы увидеть именно время запроса.