Controle

- Aanpassing na review => GeefAantalVolgensKlantNrOrderIdProductNr geeft nu 0 als BestaatVolgensKlantNrOrderIdProductNr False geeft

- Aanpassing na review => GeefAantalVolgensKlantNrOrderIdProductNr => Returned nu werkelijk de som ipv het aantal op de 1e lijn die voldoet (wat voor deze implementatie genoeg was, wegens unieke combo's, maar feitelijk incorrect was)

- Na review => UnitTests

- Query moet gesorteerd zijn volgens OrderLijnId/inventtransid, zodat de compare wat handiger is

- Lichte aanpassing AX-query (aangereikt door KPR)

- Converter AX => ConfigId en Sopr => zijn soms NUL => geeft CRLF => testen op $Char(0) => dan lege string

- Converter AX => Hoeveelheid => OpenstaandeHoeveelheid min GepickteHoeveelheid ipv som

- Converter Admin => Kleine opkuis => Ongebruikte dependency => OrderWMSLijnRepository => verwijderd

- Converter Admin => Product en Sopr => KlantNummer <> LeverancierNummer => tsssss

- Converter Admin => Product => AxMaatwerkGenerischItemId van OrderLijn halen indien ConfigId aanwezig

- Converter Admin => Product => GeefVervangProductDoorSOPRInAX mag/moet altijd

- Converter Admin =>Sopr => LeverancierNummer ophalen => Indien dan nodig (5005) => SoprContractId ophalen

- Er is enkel (1) Hoeveelheid nodig nu en deze wordt doorgegeven als parameter via de aangepaste iterator (zie volgend item)

- Er werd een nieuwe iterator tussen de bestaande iterator en converter geplaatst => AdminUniekeOrderLijnIterator => persisteert OrderLijnen tot unieke combinaties OrderNr-KlantNr-ProductNr en telt de aantallen op en geeft die mee door aan de converter

- Persistente klasse => ProductOrderLijn => met bijhorende repository => ProductOrderLijnRepository

- Lichte aanpassing aan AX-query volgens mail van KPR

- VerkochtAantal hoeft niet meer => property verwijderd

- ProductApi wordt nu gebruikt ipv ProductRolApi

- Blumkunde wordt nu netjes geconverteerd naar de SoprContractId

- Implementatie Sopr's => Ophalen SoprPrincipaal waar nodig

- DOM.PM.impl.ProductImpl voorzien van ClassMethod OpenId, die eerst %New uitsluit en dan method Open uitvoert die check doet of product bestaat

- Alle afgeleide klassen waar nodig aangepast: %New -> OpenId

- Voorziening OrderWMSLijnRepository in converter (implementatie staat voorlopig on hold tot bevestiging dat we doorgaan met gepickte hoeveelheid vergelijken)

- Leverdag: Weglaten uren, minuten, seconden en milliseconden om false negatives te vermijden

(in Admin hadden sommige lijnen blijkbaar uren, min, .... Bij de meeste was dat allemaal 0. Onduidelijk hoe dit in AX ging komen en hoe dit daarna via de query tot uiting zou komen)

- Verwerking review-opmerkingen

- Mini-refactoring

- Toevoeging velden iteratie 2 in DTO

- Herschikking velden iteratie 1 in generator (logischere volgorde in .csv)

- Toevoeging velden iteratie 2 in generator

- Toevoeging query iteratie 2 in AxOrderLijnIterator

- Iteratie 2 -velden voorlopig leeg gedefault in AdminOrderConverter (todo: conversie naar juiste waarden)

- Iteratie 2 -velden toegevoegd en uitgewerkt in AxOrderLijnConverter

- Comments in TestCase AdminOrderLijnConverter ter aanduiding UnitTests iteratie-groep

- De AdminOrderLijnIterator gebruikt nu niet meer de OrderApi (die verantwoordelijkheid is doorgegeven aan de (outer-)OrderIterator). De OrderIterator wordt gemockt in de UnitTests.

- Naamgeving in UnitTests (Stub/Mock en Dummy)

- Veld: LeverDatum: Aanpassing formatering

- Todo: VerkoopOrderId, VerkoopOrderLijnId: Te checken eens velden voorzien zijn door andere interface

- Move class -> DTO naar bovenliggend level: Aanpassingen aan de klassenaam in stringvorm

- Test voor Iterator

- We converteren eigenlijk over de orderlijnen

- AxInventTransId

- Utils: GeefLeverdag() = 06/07/2017 (zijnde 64470 in Cache)

- Veld: LeverDatum