Ошибка Violating в Print-версии

[monarch]

Добрый день,

После обновления до версии 1.32 появилась ошибка в принт-версии с красным предупреждением "Violating ...".
Из исправлений от дефолтной установки изменено:
1. название сайта в переводах - модули seo
2. там же - description
3. keywords

Сам Print-шаблон не трогали. Копирайты везде стоят. В main-шаблоне добавили счётчики.

Скриншот:

printable_violating.png (170.8 КБ) Просмотров: 12516

[andipas]

В файле themes/atlas/views/layouts/print.php

добавьте перед тегом </body>

Код: Выделить всё

<center>Powered by <a href="https://open-real-estate.info/en/" target="_blank">Open Real Estate</a></center>
 

[monarch]

Спасибо! Всё "взлетело".

Поскольку теперь страницу стало видно нормально, есть вопрос с точки зрения клиента сайта :)
Два момента:

1) Если мне не изменяет память в печатной версии объявления (в шаблоне classic) была контактная информация о риэлторе.
Там ещё телефон был картинкой, но он отображался при нажатии (чтоб не парсили злостно).
Сейчас я так понимаю всё "упаковано" в QR-код, но им не все пользуются.

Линк:
https://volga.guru/baza/property/306?printable=1

На демке тоже нет:
https://demo-free-ru.open-real-estate.i ... rintable=1

2) В версии для печати, почему то отсутствует карта (у нас OpenStreet выбран):
https://volga.guru/baza/property/294?printable=1

Хотя, на демке карта в принт-версии есть:
https://demo-free-ru.open-real-estate.i ... rintable=1

Спасибо!

[Xpycm]

1) Нет, вывода телефона в печатной версии у нас не было ни в одной из тем.

2) В демке используется Яндекс.Карты.
у OSM, как оказалось, сервис, который генерировал статические изображения перестал корректно и стабильно работать.


Чтобы карта отображалась в themes/atlas/views/modules/apartments/views/view_print.php
вместо

Код: Выделить всё

<?php if (isset($staticImageUrl) && $staticImageUrl) : ?>
    <div>
        <!--<h2><?php // echo Yii::t('common', 'Map');?></h2>-->
        <img src="<?php echo $staticImageUrl; ?>" width="<?php echo $sWidth; ?>"
             height="<?php echo $sHeight; ?>"/>
    </div>
<?php endif; ?>

Вставьте:

Код: Выделить всё

<?php
echo '<div>';
$this->renderPartial('//modules/apartments/views/_tab_map', array(
    'data' => $model,
    'isPrintable' => true,
));
echo '</div><br />';
?>

Спасибо, что написали.

[monarch]

1) Нет, вывода телефона в печатной версии у нас не было ни в одной из тем.

В версии 1.25.6 в принт-версии, было поле "Телефон владельца:".
Тема classic была, кажется.

Вот топик с пруфом этого функционала, где была поправлена отрисовка картинки с номером телефона.
viewtopic.php?f=3&t=2065

Возможно, он был выпилен из печатной версии. Но, если клиент или риэлтор распечатал страницу с объявлением, то ему, чтоб контактные данные по объекту были известны, придется выходить на обычную версию и дописывать эти данные на напечатанном листе от руки. Не очень удобно.

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


2. Спасибо за фикс по картам, попробую прикрутить!

[monarch]

2) В демке используется Яндекс.Карты.
у OSM, как оказалось, сервис, который генерировал статические изображения перестал корректно и стабильно работать.

Чтобы карта отображалась в themes/atlas/views/modules/apartments/views/view_print.php
вместо
...

Внёс эти поправки, грохнул assets & runtime. Карта появилась, но адаптивность поломалась.
Карту нужно засунуть в какой-то div, как это сделано на странице обычного объявления.

[Foton]

Здравствствуйте!
Очень сомнительна необходимость адаптивности в версии для печати.
К тому же я не обнаружил на вашем сайте что неадаптивнось вызвана именно картой, не могли бы вы предоставить скриншоты?

[monarch]

К тому же я не обнаружил на вашем сайте что неадаптивнось вызвана именно картой, не могли бы вы предоставить скриншоты?

Добрый день,

Да, пардон, не картой, а div-ом с фотографиями.
Адаптивность не нужна, если поисковики не узнали бы об этих ссылках, но у нас был немного урезанный robots.txt и они прошлись по ?printable версиям.
Щас есть запрет, но они продолжают ходить :) И гугл, и яндекс.
Если бы вписать в шапку

Код: Выделить всё

<link rel canonical = "полный адрес объявления без ?printable"> 

тогда можно было бы убрать и запрет на индексацию, и гугл бы не ругался.
Но если ставить canonical, то его и на выборках и нескольких других страницах надо ставить.

Я боюсь лезть в код, т.к. это уже серьёзная правка движка будет :)


Скриншоты:
До:

1.jpg (174.58 КБ) Просмотров: 11971

После:

2.jpg (121.31 КБ) Просмотров: 11971