Xwab
Форумыnavigate_nextСкрипты

DeForum v.1.0
Сообщения
KogekE

1) Куча echo это ну никак не тру
2) Зачем ECHO print_r()?
print_r() и сам справится
6)
@$type = abs(intval($_POST['type']));
@$sort = abs(intval($_POST['sort']));
@$type_sort = abs(intval($_POST['type_sort']));

24 Авг 2010, 21:32
Default_mo

иногда они выдавали ошибки

24 Авг 2010, 21:38
KogekE

Надо было убирать "ошибки" "естественными" путями, при error_reporting(E_ALL)
Ну это мои скромные замечания, смотрел только index.php)
- - - - - - - -
А слушать меня или нет, дело твое)

24 Авг 2010, 21:41
Default_mo

у тебя хоть критика адекватная, что будет учтено в след версии (за что респект)
, не такая как у некоторых личностей)

24 Авг 2010, 21:44
KogekE

И исчо, просто проходя мимо заметил
в _config.php session_start();
Так зачем же ты везде пишешь session_name('forumfree');, если ее можно написать _1_раз_ в _config.php?

24 Авг 2010, 21:47
Default_mo

это знаю
на скорую руку дописывал вот и вышло так

24 Авг 2010, 21:48
KogekE

Почему ты так не любишь массивы? Почему ты везде стараешься загнать индекс какого-либо массива в переменную? Используй прям массивами

24 Авг 2010, 21:50
jupsex

Хорошо бы, в следующей версии добавить возможность транслирования темы в rss. И общий список вложений.

24 Авг 2010, 21:53
Default_mo

28, сделаем
общий список вложений есть в админке.

25 Авг 2010, 6:32
kmsdsmk

Default_mo, видел бы ты код моими глазами то обассался бы
KogekE привел только малую часть говнокодинга.

25 Авг 2010, 7:30
Ответить на тему