Ошибка добавления страниц

Ошибка при добавлении новой страницы, модуль page

Landgraft	#1 21. Januar 2014, 07:02
Members	У меня одного вылезает проблема при добавлении новой страницы "SQL error HY000: General error: 1366 Incorrect integer value: '' for column 'page_size' at row 1"??? И еще один вопрос, если для поля page_size (тег PAGEADD_FORM_SIZE, 153 строка) в page.add.php установлен параметр maxlength=255, а в бд это поле имеет значение int(11) - то может все таки стоит исправить page.add.php? А то после обновления с siena 0.9.15 на 0.9.16 опять вылезла ошибка...

Landgraft

#1 21. Januar 2014, 07:02

У меня одного вылезает проблема при добавлении новой страницы "SQL error HY000: General error: 1366 Incorrect integer value: '' for column 'page_size' at row 1"???

И еще один вопрос, если для поля page_size (тег PAGEADD_FORM_SIZE, 153 строка) в page.add.php установлен параметр maxlength=255, а в бд это поле имеет значение int(11) - то может все таки стоит исправить page.add.php? А то после обновления с siena 0.9.15 на 0.9.16 опять вылезла ошибка...

Dayver	#2 21. Januar 2014, 14:05
Team Thanked: 180 mal	А что вы вбиваете в это поле что у вас ошибка вылазит? Pavlo Tkachenko aka Dayver

Landgraft	#3 21. Januar 2014, 15:16
Members	Ничего, и в поле "прикрепить файл" селектор стоит в положении "нет".

Macik	#4 21. Januar 2014, 19:16
Team Thanked: 181 mal	Главная проблема, на сколько я понимаю в том, что в базе поле как INT, а модуль не фильтрует правильно значчение при вводе : $rpage['page_size'] = cot_import('rpagesize', $source, 'TXT'); Добавлено 4 минуты спустя: Создал тикет №1327. Добавлено 2 минуты спустя: to @Landgraft: Для быстрой проверки — исправь строку на: ``$rpage['page_size'] = cot_import('rpagesize',` `$source,` `'INT'`);` https://github.com/macik правильный хостинг — https://goo.gl/fjCa1F

Macik

#4 21. Januar 2014, 19:16

Team
Thanked: 181 mal

Главная проблема, на сколько я понимаю в том, что в базе поле как INT, а модуль не фильтрует правильно значчение при вводе :

	$rpage['page_size']     = cot_import('rpagesize', $source, 'TXT');

Добавлено 4 минуты спустя:

Создал тикет №1327.

Добавлено 2 минуты спустя:

to @Landgraft: Для быстрой проверки — исправь строку на: `$rpage['page_size'] = cot_import('rpagesize', $source, 'INT');`

https://github.com/macik
правильный хостинг — https://goo.gl/fjCa1F

Landgraft	#5 22. Januar 2014, 05:48
Members	А каким образом изменение типа фильтра поможет при добавлении, если значение все равно передается пустое? Хотя изменение типа фильтра - правильный подход, но не достаточный для избавления от ошибки, т.к. если придет не число - то фильтр его не пропустит, и значение опять будет пустым, соответственно опять ошибка запроса. ИМХО, добавить в page.add.php в 132 строку запись $rpage['page_size'] = (!is_int($rpage['page_size'])) ? '0' : $rpage['page_size']; будет достаточным, особенно при таком изменении фильтра)) А интерес остался:)) Что, никто не добавляет страницы, если никто больше об этом не писал? Или при нахождении бага просто у себя исправляют, не скидывая в багтрак или на форум? Кстати, если изменить тип фильтра на ИНТ - то в логе будут появляться постоянно записи об ошибке типа переменной.

Landgraft

#5 22. Januar 2014, 05:48

Members

А каким образом изменение типа фильтра поможет при добавлении, если значение все равно передается пустое? Хотя изменение типа фильтра - правильный подход, но не достаточный для избавления от ошибки, т.к. если придет не число - то фильтр его не пропустит, и значение опять будет пустым, соответственно опять ошибка запроса.

ИМХО, добавить в page.add.php в 132 строку запись

$rpage['page_size'] = (!is_int($rpage['page_size'])) ? '0' : $rpage['page_size'];

будет достаточным, особенно при таком изменении фильтра))

А интерес остался:)) Что, никто не добавляет страницы, если никто больше об этом не писал? Или при нахождении бага просто у себя исправляют, не скидывая в багтрак или на форум?

Кстати, если изменить тип фильтра на ИНТ - то в логе будут появляться постоянно записи об ошибке типа переменной.

Kort	#6 22. Januar 2014, 06:49
Administrators Thanked: 211 mal	#39043 Landgraft: А интерес остался:)) Что, никто не добавляет страницы, если никто больше об этом не писал? Или при нахождении бага просто у себя исправляют, не скидывая в багтрак или на форум? PFS уже мало кто использует SED.by - создание сайтов, разработка плагинов и тем для Котонти

Kort

#6 22. Januar 2014, 06:49

Administrators
Thanked: 211 mal

#39043 Landgraft:
А интерес остался:)) Что, никто не добавляет страницы, если никто больше об этом не писал? Или при нахождении бага просто у себя исправляют, не скидывая в багтрак или на форум?

PFS уже мало кто использует

SED.by - создание сайтов, разработка плагинов и тем для Котонти

Alex300	#7 22. Januar 2014, 07:03
Administrators Thanked: 222 mal	Я использую в некоторых проектах. И с подобной проблемой не стакливался. Есть миры, не здесь, там, где небеса горят, и моря засыпают, и реки дремлют; люди сделаны из дыма, а города – из песен. Где-то опасность, где-то несправедливость, даже где-то остыл чай. Идем Эйс, у нас много работы!... ...Sorry for my english... Бесплатные расширения для Cotonti: https://lily-software.com/free-scripts/

Alex300

#7 22. Januar 2014, 07:03

Administrators
Thanked: 222 mal

Я использую в некоторых проектах. И с подобной проблемой не стакливался.

Есть миры, не здесь, там, где небеса горят, и моря засыпают, и реки дремлют; люди сделаны из дыма, а города – из песен. Где-то опасность, где-то несправедливость, даже где-то остыл чай. Идем Эйс, у нас много работы!...
...Sorry for my english...
Бесплатные расширения для Cotonti: https://lily-software.com/free-scripts/

Landgraft	#8 22. Januar 2014, 11:58
Members	А я не прикрепляю никаких файлов, и пфс не установлен)

Roffun	#9 23. Januar 2014, 12:09
Contributors Thanked: 175 mal	#39044 Kort: PFS уже мало кто использует Я использую во всех проектах, кроме бага с всплывающим окном (исправили), проблем не наблюдал. Улетел на другую планету, а там почты нету.. https://www.cotonti.com/forums/45298?m=posts