Controle

- OrdDiff-package van PV heeft nu zijn eigen AxOrderLijnIterator => Quit een ResultSet.

- De AxOrderLijnIterator gebruikt door THB staat terug in zijn oorspronkelijke staat => Quit een DTO-tje.

- Ongebruikte of onterecht toegewezen class-injecties verwijderd (AdminOrderLijnIterator en AxOrderLijnIterator)

- Ongebruikte properties verwijderd (idem)

- Query-aanpassingen en return-object-aanpassing van PV in iterator => returned nu een ResultSet ipv een DTO.

- Quickfix voor Orders die niet geopend kunnen worden.

- LegacyPartijAPI wordt nu gebruikt om LeverancierNummer te verkrijgen van de LeverancierId

- Wegwerken van de quickfix try catch => probleem is nu ten gronde opgelost

- ProductId en het Product zijn nu op meerdere plekken nodig, dus het zetten van die variabelen is verhuisd naar de Convert en ze worden nu meegegeven als parameters

- De logica is aangepast van wanneer en of de principaal moet opgehaald worden (dit is de ten gronde oplossing hierboven vermeld) => LevNr = BLUM en IsSoprInCache

- Aanpassen testen naargelang

- Na review => LegacyPartijAPI nu gebruikt voor GeefKlantNummerVoorPartijID

- Na review => GetDataByName ipv GetData in AxOrderLijnConverter

- Na review => Refactoring UnitTests voor leesbaarheid, performantie en uitbreidbaarheid

- Code van PV (geschreven op CacheAccept2010) in eerste instantie te syncen tussen alle omgevingen (vanuit lokale omgeving).

- Voor de AxConfigId aan Admin-kant mag de principaal niet opgehaald worden => de Blumkunde van de Sopr-variant moet geconverteerd worden

- Refactor naar oneliner

- Opkuis van enkele comments ivm iteratie van AX-query

- De inhoud van de mail in nu een Stream ipv een string => MAXSTRING-error vermijden

- Introductie AxInventTransId als extra property die deel uitmaakt van de unieke index

- Verfijning Repository => AxInventTransId en Early Quit als record niet bestaat in de VerwijderVolgensKlantNrOrderIdProductNrAxInventTransId()

- Verfijning en uitbreiding UnitTests van Repository

- Rechtzetting en uitbreiding Fake Repository

- Aanpassing na review => Afzonderen opbouwen data => niet meer in constructor van iterator, maar in method in Generator

- 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