id	summary	reporter	owner	description	type	status	priority	component	resolution	keywords	cc
1010	"Добавить пользователям конфигурационный параметр ""каталог протоколов"""	alx	Denis_N	"В [56/base] пользователю дана возможность открывать протокол тестирования кликом ссылки. По этому поводу у меня есть сразу несколько соображений.

1. В сообщении коммита написано, что для правильной работы этой функции требуется, чтобы пользователь установил путь к каталогу, в котором у него содержатся протоколы, в 93 строке файла testing.php. Но, это вряд ли возможно, так как, во-первых, как пользователь сможет модифицировать хранящийся на сервере файл? У него нет (и не должно быть) прав модификации этого файла! А во-вторых (и главных), файл на сервере один, а пользователей - много. Следовательно, максимум один из пользователей сможет пользоваться этой функцией.

 **Предлагаю** добавить пользователям настройку пути к каталогу протоколов, которую хранить в базе данных **для каждого пользователя**, и сделать пользователям интерфейс для управления этой настройкой.

2. Насколько я вижу, файл протокола ищется по наличию номера протокола в имени файла. Здесь есть сразу две проблемы:
 - номер протокола может отсутствовать в имени файла (так как нет стандарта на именование файлов протоколов), и тогда файл не будет найден (false negative);
 - номер протокола может быть частью номера другого протокола (например протокол номер D12345 и протокол номер D12345-2), что приведет к отображению ""лишнего"" или просто не того файла (false positive).

 **Предлагаю** вместо реализованного в [56/base] механизма просто хранить протоколы в базе данных. Для этого: в форме тестирования вместо указания номера протокола пользователь сразу загружает на сервер **файл протокола**. В дальнейшем при выводе информации о тестировании, если в базе есть протокол, выводить ссылку для его скачивания. Плюсы такого варианта:
 - это снижает вероятность ошибки ввода номера протокола, в результате которой протокол потом невозможно найти;
 - это позволяет автоматизировать присваивание номеров протоколам, исключая возможность существования нескольких протоколов с одинаковыми номерами;
 - это затруднит нерадивым (я уверен, что у нас таких нет, но все-таки) пользователям выполнять проверки без составления протокола;
 - отпадает необходимость реализации предложения п. 1 выше."	улучшение	closed	major	БД изделий АДС	invalid