id	summary	reporter	owner	description	type	status	priority	component	resolution	keywords	cc
1061	Сброс флага несоответствия при контроле ОТК	alx	san	"В БД изделий АДС в коде [source:base/otk.php@141:107-112#L90 otk.php] я обнаружил такой фрагмент:

{{{#!sh
if ($_POST['status'] == 'fail')
    $query = ""UPDATE products set `otk` = ?, `mismatch` = 'yes' where `uid` = ?"";
else if ($_POST['status'] == 'ok')
    $query = ""UPDATE products set `otk` = ?, `mismatch` = 'no' where `uid` = ?"";
else
    $query = ""UPDATE products set `otk` = ? where `uid` = ?"";
}}}

Из этого кода следует, что контроль ОТК может устранять несоответствия изделий (см. вторую ветку условия)!!! Как такое возможно?

Моя логика подсказывает, что если изделие имеет какое-то несоответствие, то для устранения этого несоответствия требуется провести ремонт. Разве в обязанности контролера ОТК входит проведение ремонта? Я точно не знаю, но предполагаю, что нет - он только **контролирует** факт соответствия изделия КД, но ничего не ремонтирует (а если даже ремонтирует - есть интерфейс ""Ремонт""). Следовательно, в процессе **контроля** имеющееся у изделия несоответствие исчезнуть никак не может (может быть только выявлено новое несоответствие). И, следовательно, признак наличия несоответствия **не должен** сбрасываться при контроле ОТК.

**Предлагаю** убрать из второй ветки процитированного выше условия {{{, `mismatch` = 'no'`}}}

Более того, было бы логично вообще не допускать до контроля ОТК (выдавать ошибку при вводе S/N такого изделия) изделия, о которых известно, что они имеют несоответствия - зачем тратить время контролера, если изделие заведомо не пройдет контроль ОТК? **Предлагаю** реализовать такую проверку.

На данные размышления меня навел случай с одной платой, у которой я выявил (и занес в БД) несоответствие, однако плата **успешно** прошла контроль ОТК (в результате чего признак несоответствия был сброшен) и была отгружена заказчику, хотя ремонт, устраняющий обнаруженное несоответствие, проведен не был...

Добавил зам. директора по качеству в копию, так как ему тоже, наверное, будет интересно."	дефект	closed	major	БД изделий АДС	дубликат		andrei