недочёт в документации

Нашли ошибку в CI? — пишем сюда!

недочёт в документации

Сообщение michail1982 » 13 июл 2009, 00:33

причём, как в русской, так и в оригинальной
никак не необозначена функция $this->db->close();
не критично конечно, но тогда было-бы логично вставить $this->db->close(); в деструктор...
я знаю, что пхп подчищает за собой, но при неправельных конфигах (как у меня в МуСКуЛе ) подвисают идл-процессы
Аватара пользователя
michail1982
 
Сообщения: 533
Зарегистрирован: 25 ноя 2008, 15:36

Re: недочёт в документации

Сообщение denis909 » 13 июл 2009, 01:50

в загрузчике codeigniter (system/codeigniter/codeigniter.php) есть:

Код: Выделить всё
if (class_exists('CI_DB') AND isset($CI->db))
{
    $CI->db->close();
}
 


т.е. как-бы соединения должны закрываться, но..! действительно процессы бывают что и висят, м.б. сервер перегружен запросами...
о-па... пока писал идея пришла, при конфигах, когда php не закрывает соединения, висеть будут страницы 404 вызванные из контроллера через show_404() или show_error(), т.е. имеет смысл закрывать соедиение в application/errors, а не в деструкторе.
Аватара пользователя
denis909
 
Сообщения: 185
Зарегистрирован: 27 фев 2009, 00:58

Re: недочёт в документации

Сообщение michail1982 » 13 июл 2009, 11:11

Спасибо, попробую..
правильно ли я понял?
в файл system\codeigniter\Common.php в функции show_error() и show_404() добавить
Код: Выделить всё
$CI = &get_instance();
if (
class_exists('CI_DB') AND isset($CI->db))
{
    
$CI->db->close();
}
    exit;
Аватара пользователя
michail1982
 
Сообщения: 533
Зарегистрирован: 25 ноя 2008, 15:36

Re: недочёт в документации

Сообщение denis909 » 13 июл 2009, 13:39

Более правильно будет расширить библиотеку CI_Exceptions через MY_Exceptions.
Аватара пользователя
denis909
 
Сообщения: 185
Зарегистрирован: 27 фев 2009, 00:58


Вернуться в Баг-репорты

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

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