Page 1 of 2
Баги при удалении пользователя и редактировании cron записей
Posted: Sun Aug 18, 2013 6:19 pm
by Shamilik
При удалении пользователя, от имени которого на данный момент сидит админ (admin->user), он (пользователь) удаляется и больше ничего нельзя сделать. Нельзя ни вернуться назад, ни просто выйти. Помогла только очистка кук.
Re: Ошибка при удалении пользователя
Posted: Mon Aug 19, 2013 8:38 am
by skid
Re: Ошибка при удалении пользователя
Posted: Mon Aug 19, 2013 2:09 pm
by Shamilik
При редактировании cron записи, если где-то был 0, например в минутах, он просто пропадает и приходится опять его там прописывать.
Re: Баги при удалении пользователя и редактировании cron зап
Posted: Tue Aug 20, 2013 11:19 am
by skid
Re: Баги при удалении пользователя и редактировании cron зап
Posted: Tue Aug 20, 2013 5:28 pm
by Shamilik
Хм, а случайно не...?
Типо как empty(), только пропускает нолик, но не пропускает пустое поле.
А isset() пропустит и '' (именно, пустоту)...
(Или мне уже пора спать и несу бред :D)
Re: Баги при удалении пользователя и редактировании cron зап
Posted: Tue Aug 20, 2013 6:23 pm
by skid
Как раз isset() пустоту не пропускает, а вот empty() воспринимает нули как пустое значение, потому и была проблема с нулями.
Re: Баги при удалении пользователя и редактировании cron зап
Posted: Wed Aug 21, 2013 7:32 am
by Shamilik
Фигово получается объяснять.
Я про то, что $_POST['something'] может быть передано пустым, в смысле оно будет установлено, но будет являться пустым.
http://codepad.org/g74a9kf7
Правка к предыдущему посту:
=>
Code: Select all
if (!isset($_POST['v_min']) || $_POST['v_min'] == '')
Re: Баги при удалении пользователя и редактировании cron зап
Posted: Wed Aug 21, 2013 9:09 am
by skid
Теперь понятно. Да, такая ситуация возможна. Добавил
проверку.
Re: Баги при удалении пользователя и редактировании cron зап
Posted: Wed Aug 21, 2013 9:12 am
by Shamilik
Code: Select all
((!isset($_POST['v_min'])) && ($_POST['v_min'] != ''))
=>
Code: Select all
(!isset($_POST['v_min']) || ($_POST['v_min'] == ''))
Пора меня приписать в commit как Shamil Yakupov xD
Re: Баги при удалении пользователя и редактировании cron зап
Posted: Wed Aug 21, 2013 9:15 am
by skid
Точно, логическая ошибка!