constants.php глюк или так должно быть

[Andry]

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

Скачал phpBB-2.0.21.zip(http://surfnet.dl.sourceforge.net/sourc ... 2.0.21.zip)
и в файле constants.php в строке 29-30:

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

//define('DEBUG', 1); // Debugging on
define('DEBUG', 1); // Debugging off

Наверное, это есть не очень хорошо?
Я так понимаю что по умолчанию включен дебаг режим?

[Nikel]

Я так понимаю что по умолчанию включен дебаг режим?

Неправильно понимаеш , дебагг нужен только при отлатке\тестирвании .

[Andry]

Я знаю для чего нужен этот флаг, вопрос в другом:
Насколько правомерно в дистрибутиве по умолчанию ставить "define('DEBUG', 1);", тем более что предполагается (судя по комментарию и предыдущим версиям), что по умолчанию будет "define('DEBUG', 0);".
Я не уверен, что все кто, будет ставить/обновлять phpBB, полезут в constants.php и изменят данный флаг. А включенный DEBUG может сказать на безопасности форума.

[Nikel]

Насколько правомерно в дистрибутиве по умолчанию ставить "define('DEBUG', 1);",

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

[avm]

Nikel, он говорит, что вместо 0 в дистрибе стоит 1 ! Так оно и есть.
Хотя по смыслу должен быть 0...

Но судя по испольванию этой константы в коде (всего дважды и только в одной функции!)
includes/functions.php, function message_die():

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

//
// Get SQL error if we are debugging. Do this as soon as possible to prevent
// subsequent queries from overwriting the status of sql_error()
//
if ( DEBUG && ( $msg_code == GENERAL_ERROR || $msg_code == CRITICAL_ERROR ) )
{
  сформировать $debug_text
}

//
// Add on DEBUG info if we've enabled debug mode and this is an error. This
// prevents debug info being output for general messages should DEBUG be
// set TRUE by accident (preventing confusion for the end user!)
//
if ( DEBUG && ( $msg_code == GENERAL_ERROR || $msg_code == CRITICAL_ERROR ) )
{
   добавить сформированный ранее $debug_text к $msg_text
}

ее наличие не только не критично, но и не оправдывает ожиданий от ее имени
т.е. она просто при возникновении ошибки в SQL, выводит сам sql-запрос и диагностику на страницу отображения ошибки...
define(DEBUG,0) позволит убрать вывод SQL запроса в сообщении об ошибке.

имхо эту константу просто нужно было назвать по другому...