Opened 14 months ago

Closed 2 weeks ago

#1377 closed улучшение (готово)

Перемещение изделий туда, где они и так уже находятся

Reported by: alx Owned by: Denis_N
Priority: minor Component: БД изделий АДС
Keywords: Cc:

Description

Случайно обнаружил в БД запись в истории изделия S/N D02052 о его перемещении со склада на склад! :)

Очевидно, что в такой записи нет никакого смысла - если изделие было на складе и перемещено на склад, то это все равно, что никакого перемещения не было. Предполагаю, что данная запись была внесена в БД по ошибке.

Чтобы помочь пользователям избежать подобных ошибок предлагаю в веб-интерфейсе, а также в backend'е добавить проверку, не позволяющую добавить запись о перемещении изделия, если исходное (текущее) местоположение совпадает с новым (указанным пользователем) местоположением.

Change History (6)

comment:1 by san, 14 months ago

Со склада на склад перемещение может быть разумным если изменился номер полки на складе(вообще складов у нас теперь несколько и теоретически изделие могут перенести с одного склада на другой, номер склада содержится в номере полки). Номер полки задаётся в поле shelfdrawer.
Для изделия S/N D02052 операция перемещения действительно была бессмысленной, т.к. номер полки как был 1.2.3.6, так и остался.

comment:2 by Denis_N, 2 weeks ago

In 570/base:

Улучшение

Добавлена проверка в интерфейсе возврата и backend'е, чтобы не сохранять
change_location, если изделие уже находится в выбранном месте.

Для склада сравнивается не только location, но и shelfdrawer, поэтому
перемещение между разными полками остаётся разрешённым. При попытке
сохранить бессмысленное перемещение пользователь видит сообщение об ошибке.

See #1377

comment:3 by Denis_N, 2 weeks ago

In 572/base:

Улучшение универсальная панель

Добавлена проверка в интерфейсе возврата и backend'е, чтобы не сохранять
change_location, если изделие уже находится в выбранном месте.

Для склада сравнивается не только location, но и shelfdrawer, поэтому
перемещение между разными полками остаётся разрешённым. При попытке
сохранить бессмысленное перемещение пользователь видит сообщение об ошибке.

see #1377

comment:4 by Denis_N, 2 weeks ago

In 572/base забыл сразу указать, что еще в этом коммите в универсальной панели добавлено поле номера полки при перемещении на склад.

Version 0, edited 2 weeks ago by Denis_N (next)

comment:5 by Denis_N, 2 weeks ago

In 573/base:

Улучшение: В массовом сканере добавлена проверка серийников, которые уже находятся
в выбранном месте. Для интерфейсов с массовой обработкой серийных такие операции не блокируются жестко, в отличие от интерфейсов, которые работаю с одним серийным, чтобы не терять производственные скорости.

Backend возвращает список совпавших серийников, а интерфейс показывает подтверждение. После подтверждения пользователя сохранение выполняется как раньше. Для
занесения на склад сравнивается location + shelfdrawer, для смены
местоположения — текущий location.

See #1377

comment:6 by Denis_N, 2 weeks ago

Resolution: готово
Status: newclosed
Note: See TracTickets for help on using tickets.