Controle

- 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

- AX-query sorteert nu op VerkoopOrderId, zodat de compare wat handiger is

- Try Catch => Indien GeefOrder() op de AdminOrderLijn geen order geeft => GEEN ORDER GEVONDEN in eerste cel van csv op die lijn ipv ingevuld record, zodat volgende records wel nog lukken

- De AdminOrderLijn-parameter in afzonderlijke variabele gestoken => ZoekeOrderLijn => voor debugging mits invoeging break

- 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

- 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)

- Poging vier tot webservice (zodat niet-Caché-mensen bestanden kunnen genereren) -> moet gecommit worden om te kunnen uittesten: Write verwijderd

- Voor snel resultaat aan Admin-kant: betreffende Gepickte- en OpenstaandeHoeveelheid -> op 0 gezet

- Verwerking review-opmerkingen

- Sopr/ProductConvertor TECH.Mockable gemaakt voor testdoeleinden

- Sopr/ProductConvertor -> Dummy gemaakt voor testdoeleinden

- Sopr geîmplementeerd in conversie

- Todo: Test Sopr indien wèl een BlumProduct staat in comment -> probleem om leverancier in test op Blum te zetten

- 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)

- Disconnect() en refactoringske

- Veld: LeverDatum: Aanpassing formatering

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

- Refactoring (vooral verzamelen van dependencies)

- Opzet: Iteratie 1