metasfresh / metasfresh-webui-frontend-legacy

metasfresh Webui Frontend
GNU General Public License v2.0
57 stars 35 forks source link

Same HU shown with picking processed and not processed #1942

Closed metas-dh closed 4 years ago

metas-dh commented 6 years ago

Is this a bug or feature request?

Bug

What is the current behavior?

The same HU is shown with picking processed and not processed, for 2 different orderlines in picking terminal. http://recordit.co/BMNyfQvBFa

Which are the steps to reproduce?

  1. Sales order with 2 orderlines, same product
  2. In picking terminal pick a HU of that product in a picking slot
  3. process the picking, while 1 orderline is selected: HU becomes read-only
  4. select the other orderline: HU is not read-only anymore, as if the picking was not processed

    What is the expected or desired behavior?

    HU shall be shown the same for all orderlines.

siemiatj commented 6 years ago

After looking for a suitable test case for a year, when I've finally found one I'm getting error from the server when running quick action :/ Any ideas ? @teosarca @metas-ts

http://recordit.co/Dqztnbj44n

metas-dh commented 6 years ago

afais the error says "More than 1 record found", can it be related to #1941? since the HU is displayed in the picking slot, but also still shown in the HU Editor for selection? @teosarca wdyt?

teosarca commented 6 years ago

@siemiatj @metas-dh The issue is not related with this task but ofc is blocking this one. The problem is described and shall be solved here: https://github.com/metasfresh/metasfresh/issues/4562

More, in this particular case (w101) there are 3 distribution networks with empties but we shall have only one: image

Atm, i've kept only the first one. If, while testing on w101, we got an error because empties could not be collected, we need to create the distribution line inside first distribution network: image

metas-dh commented 4 years ago

this is not happening anymore, was probably fixed with the several tasks on picking terminal we had in the recent weeks.