Ошибка при генерации телефона на страницы печати

Пишите о найденных ошибках в эту тему

Модераторы: Xpycm, Koduc

Правила форума
ВНИМАНИЕ! Форум не является средством гарантированной поддержки клиентов и пользователей. Поэтому на быстрый ответ тут рассчитывать не нужно, как и на ответ вообще. Сотрудники отвечают по мере своих возможностей.

На форуме действует ограничение в 3 сообщения / сутки.
Если Вы хотите сказать "спасибо", то воспользуйтесь функцией "Повысить репутацию" - зелёная иконка "плюс" под ником ответившего.

Ошибка при генерации телефона на страницы печати

Сообщение monarch » 19 янв 2019, 16:54

Добрый день!

На печатной версии объявления не генерится телефон.
Пример: /property/177?printable=1
Инспектор в Google Chrome показывает вот такую ошибку:

4.png
4.png (243.46 КБ) Просмотров: 16487


Спасибо!
--
С уважением, monarch.
Аватара пользователя
monarch
Прохожий
 
Сообщения: 39
Зарегистрирован: 30 ноя 2018, 11:54
Очки репутации: 17

Re: Ошибка при генерации телефона на страницы печати

Сообщение Xpycm » 19 янв 2019, 19:30

Здравствуйте.

Телефона в печатной версии быть не должно, иначе его легко будет парсить.

Для этого в файле protected\modules\formdesigner\models\HFormEditor.php
вместо:
Код: Выделить всё
if (!$model->canShowInView($row['field'], $row->type)) {

вставьте:
Код: Выделить всё
if (!$model->canShowInView($row['field'], $row->type, $isPrintable)) {


Затем в файле protected\modules\apartments\models\Apartment.php
вместо:
Код: Выделить всё
public function canShowInView($field, $type = false)

вставьте:
Код: Выделить всё
public function canShowInView($field, $type = false, $isPrintable = false)


и в этом же методе, после:
Код: Выделить всё
case 'phone':

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

if 
($isPrintable)
    return false;


Спасибо, что отписываетесь об ошибках и недочётах
Dropbox
Open Real Estate CMS: FAQ | FAQ 2 | FAQ 3
Изображение
Xpycm
Разработчик
Разработчик
 
Сообщения: 1592
Зарегистрирован: 30 дек 2011, 11:06
Откуда: Йошкар-Ола
Очки репутации: 50

Re: Ошибка при генерации телефона на страницы печати

Сообщение monarch » 24 янв 2019, 15:09

Поправил эти файлы - телефон (вместе с ошибкой при его генерации) из печатной версии пропал :)
Спасибо!

Так же интересно, есть ли возможность фиксы вышеуказанных ошибок включить в новые версии (или они уже включены).
--
С уважением, monarch.
Аватара пользователя
monarch
Прохожий
 
Сообщения: 39
Зарегистрирован: 30 ноя 2018, 11:54
Очки репутации: 17

Re: Ошибка при генерации телефона на страницы печати

Сообщение andipas » 25 янв 2019, 15:24

Да в новой версии такой ошибки нет.
Часто задаваемые вопросы. Для правки файлов notepad++, netbeans, phpStorm. Для правки CSS firebug.
Аватара пользователя
andipas
Разработчик
Разработчик
 
Сообщения: 695
Зарегистрирован: 28 дек 2011, 22:37
Очки репутации: 30

Re: Ошибка при генерации телефона на страницы печати

Сообщение Dmitry » 12 май 2020, 02:59

Xpycm писал(а):Телефона в печатной версии быть не должно, иначе его легко будет парсить.

Сейчас же вроде везде отображается телефон картинкой и если его разместить на печатной версии (естественно без кликабельной ссылки) это как-то облегчит работу желающим спарсить базу телефонов? Ведь картинка, вроде, генерируется по ID объявления (его владельца) и выдается как data:image/png , а не с адреса, содержащего сам номер. Что может его скомпрометировать на печатной странице?

Тогда уж проще спарсить с обычной (НЕ печатной) страницы в 2 строчки кода, ну и еще несколько строк, чтобы это потом сохранить.

Но только представьте как будет материться пользователь, который распечатает десяток-другой вариантов, чтобы потом обзвонить и не найдет там телефона :-D

PS. Мне больше спама сыпется на телефон ни разу не засвеченный в сети, чем на висящий в открытую на нескольких десятков сайтов (кстати на нем что-то я вообще особо спама и не припомню :-[ )
Dmitry
Гражданин
Гражданин
 
Сообщения: 135
Зарегистрирован: 29 мар 2020, 17:04
Откуда: Крым
Очки репутации: 22

Re: Ошибка при генерации телефона на страницы печати

Сообщение Foton » 16 май 2020, 14:51

В следующей версии мы добавим отображение телефона в версию для печати
Foton
Разработчик
Разработчик
 
Сообщения: 431
Зарегистрирован: 26 янв 2012, 14:58
Очки репутации: 8


Вернуться в Ошибки

Кто сейчас на конференции

Сейчас этот форум просматривают: нет зарегистрированных пользователей и гости: 50

cron