inventory related fixes - Githubissues

metasfresh / metasfresh

We do Open Source ERP - Fast, Flexible & Free Software to scale your Business.

http://metasfresh.com/en

GNU General Public License v2.0

1.74k stars 592 forks source link

inventory related fixes #4876

Closed metas-ts closed 5 years ago

metas-ts commented 5 years ago

Is this a bug or feature request?

Bug follow-up of metasfresh/metasfresh#4448 and metasfresh/metasfresh#4450

What is the current behavior?

Which are the steps to reproduce?

What is the expected or desired behavior?

metas-dh commented 5 years ago

Results of IT1 tested in release

Inventur (Physical Inventory):
- automatische Nummernvergabe: OK
- Ergebnisse wenn Parameter gesetzt in Menge & Min. Wert / Min. Wert: OK
- älteste HUs zu erst: keine Daten bislang in Inventur: HUs were sorted by value, smallest value (= oldest HU) first and biggest value (= newest HU) last: OK
- sortiert nach Lagerort: zuerst die, welche am längsten nicht gezählt wurden:
- es waren aber andere Lagerorte noch nie in fertig gestellten Inventuren enthalten, sollten die nicht als erstes gelistet werden? => worked this time, OK
- Zeilen erstellen (Lagerort & Produkt) : Parameter "Lagerort" dropdown liefert keine Ergebnisse (Parameter "Lagerort" works only when warehouse is set in main window, not when left empty), NOK (Aktion funktioniert aber ohne Parameter / mit Parameter Produkt) => afais this was not considered in this task, right? do we need another task for this?
- Feld Merkmale was correctly set (in my case, Bio): OK

question: when running both available actions, w/o parameters set, i got different results:

Zeilen erstellen (Lagerort & Produkt) : got 13 lines, including 1 product in Verdichtungslager
Zeilen erstellen (Lagerort-Anz. & Min.-Warenwert): got 12 lines, w/o the product in Verdichtungslager => shouldn't the results be the same? @metas-ts wdyt?

Inventur-Position Zählen:
- Filter now also includes "Zugewiesen an": OK
- Filter for product and Lagerort also included now: OK
- Filter for Suchschlüssel (value) not included anymore: OK? don't we need that anymore?
- changes like marking as "Gezählt" directly displayed, no window refresh necessary: OK
- field UOM not included anymore: OK

metas-ts commented 5 years ago

Hi @metas-dh

when running both available actions, w/o parameters set, i got different results:

talked with @metas-mk

it's OK that the "Zeilen erstellen (Lagerort-Anz. & Min.-Warenwert)" results do not contain anything from Locator "Verdichtungslager"
- this stuff is picked and basically out the door when it comes to stock
the same should apply to "Zeilen erstellen (Lagerort & Produkt)"

=> creating a follow-up task => https://github.com/metasfresh/metasfresh/issues/4927

Filter for Suchschlüssel (value) not included anymore: OK? don't we need that anymore?

as talked; i now added a M_HU_ID to be filterable as well:

metas-dh commented 5 years ago

checked in mf15:

filter for M_HU_ID included in Inventur-Position Zählen: OK
working fine: OK