Яндекс нашел ошибку в магазине

[Katrinz 0]

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

Вот ссылка на эту страницу: http://napolspb.ru/articles.php?tPath=1

а вот, что на ней написано:

Warning: Division by zero in /usr/home/napolspb/public_html/includes/classes/split_page_results.php on line 55

Warning: error_log(mysql_db_error.log) [function.error-log]: failed to open stream: Permission denied in /usr/home/napolspb/public_html/inc/vam_db_error.inc.php on line 40

Интернет-магазин закрыт на техническое обслуживание, заходите позже!

Проблема с MySQL. Как узнать причину ошибки.

Прдскажите, как эту страницу закрыть от яндекс/ просто удалить или ещё что-нить сделать, чтобы такого не было....

[Bububu 0]

Это статья "Напольные покрытия" даёт Вам такое.

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

-------

а нет, ссылка не корявая...

[support 447]

Для начала можно узнать текст ошибки, вот так - http://vamshop.ru/faq.php/faq_id/36/question/V-magazine-vyvoditsya-soobshenie--Internet-magazin-zakryt-na-tehnicheskoe-obsluzhivanie--zahodite-pozzhe----chto-yeto-za-soobshenie-i-kak-ego-otklyuchit--

[Bububu 0]

Почему - то значение MAX_NEW_ARTICLES_PER_PAGE выставлено в 0 и ошибка от этого.

Щас поищем....

[support 447]

Эта опция настраивается в Админке - Разное - Статьи - Настройка - Количество статей на одной странице новых статей, вот в этой опции не должно быть 0.

[Katrinz 0]

Ой, спасибо :)))

[support 447]

Не за что

[mosquito 0]

в этой строке может быть и 0 и что угодно, нужно только добавить в скрипте обработку исключений а не заставлять пользователя вводить то что от него ожидает скрипт.. ведь это не так сложно делается, хотя таких проблем\ошибок очень много, правятся они не так сложно что бы их игнорировать имхо

[support 447]

А что тогда выводить, если 0 укажут?!

Просто интересно.

[mosquito 0]

в большинстве вариантов 0 и другие неприемлемые данные используються как значение "по умолчанию" а каким поведением магазина будет на это значение решать тибе: или не отображать ничего, или отображать все (по умолчанию - выводим все статьи)

но в любом случае магазин не должен падать из за таких ошибок)

или не согласен?)

[support 447]

Лично я не вижу в этом проблемы, по умолчанию нет нулей нигде.

[ABerezin 0]

в большинстве вариантов 0 и другие неприемлемые данные используються как значение "по умолчанию" а каким поведением магазина будет на это значение решать тибе: или не отображать ничего, или отображать все (по умолчанию - выводим все статьи)

но в любом случае магазин не должен падать из за таких ошибок)

или не согласен?)

Поддерживаю

[support 447]

В /includes/classes/split_page_results.php вобщем-то можно определить, что если не задано количество, просто назначить, что б нуля не было, но как быть например с /templates/шаблон/source/boxes/articles_new.php и другими боксами, где в sql запросе используются константы, а именно в limit, т.е. вот так:

LIMIT " . MAX_NEW_ARTICLES_PER_PAGE . "";
[/code]
Если значение в админке не задано, всё равно ведь ошибка будет.
Я например не знаю как по всему магазину подобные места изменить, не ковыряя половину магазина.
Если кто знает, был бы признателен за совет.    

[Bububu 0]

Может, на случай проблем сделать SQL-запрос (несколько) и запускать их в сомнительных ситуациях подобного рода.

В запросе править недопустимые значения мягко - до минимальных безопасных или грубо - ставить в default.

Тупо и надёжно в таких ситациях.

MAX_NEW_ARTICLES_PER_PAGE <= 0  тогда    MAX_NEW_ARTICLES_PER_PAGE = 1

Правда эти запросы тоже писать надо, но объём работы меньше, чем прописывать в скрипт движка "защиту от дурака".

Это что-то вроде аптечки для первой мед. помощи. Появились проблемы - запустил и проверил.