Форуми / National / Russian / Тех. поддержка / Бета-впечатления v.0.0.2

Тема обсуждения багов

dervan
#7815 13.02.2009 08:03

Contributors

# Asmo : Конечно надо фиксить, только не пойму почему ты у меня об этом спрашиваешь.
И правда, чё это я... (c)корова :)

# Asmo : Вчера проверял, баг полностью фиксится перестановкой проверки
Ещё одно соображение появилось, после общения в топике PFS Problem, там у человека что-то странное творится (возможно этот самый баг с подменой сообщения об ошибке - но не факт, файл на котором сбоит весит всего 800 kb). Последний скриншот там красноречивый - максимальный размер файла 2000 Gb. А вот какой код в system/core/pfs/pfs.inc.php:
	$maxfile = $row['grp_pfs_maxfile'];
и затем та самая проверка с этим $maxfile. Т.е. можно в админке поставить максимальный размер файла, превышающий настройки сервера - и опять начнётся цирк с конями. Поэтому $maxfile надо получать так:
	$ini_maxfile = ini_min(ini_get(upload_max_filesize), ini_get(post_max_size), ini_get(memory_limit));
	$maxfile = min($row['grp_pfs_maxfile'], $ini_maxfile);
Здесь функцища ini_min() - условная запись, там возможны сокращения для байтовых значений.
При этом ещё и от
$disp_upload .= "<input type=\"hidden\" name=\"MAX_FILE_SIZE\" value=\"".($maxfile*1024)."\" />";
будет больше толка.

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

Готов всё это доделать.