Uitgaand

[ICT] Aanpassingen als gevolg van een shshsh die ik heb uitgevoerd lokaal...

........

revert omdat mijn xml export blijkbaar nogal verschillend is van de rest van de ventjes hier

- Na review => Mockable gemaakt om AxProductValidator onder test te brengen

- Fake Translator gemoved => heeft specifieke implementatie => willen we andere naam geven dan als code zelf Mockable gemaakt zou worden => beveiliging tegen eventueel overschrijven

- Fake Translator nodig voor UnitTests

- Velden AltItemId en UseAltItemId

- DTO VrijgegProduct => properties erbij gezet en alle properties gesorteerd volgens volgorde PakketReferentie VrijgegProducten

- De VeiligheidsVoorraadCalculator is nu Mockable

- Aanpassing van de Converter => verwijst nu naar VeiligheidsVoorraadCalculator

- extraleverdagen in mindering brengen in de berekening van de levertermijn

- opkuis

- Code aangeleverd door PV afgezonderd in aparte klasse met het oog op mogelijkheid tot uitmocken => ArtikelGroepBepaler

- Implementatie ArtikelGroep in converter

- Bijhorende test

- Nieuwe filter die oude (GenerischOrgaluxProductFilter) implementeert

- UnitTest nieuwe filter => functionaliteit GenerischOrgaluxProductFilter

- Oude filter in migratiecode Product vervangen door nieuwe => controle op ongewijzigd resultaat => check

- Aanpassen UnitTests migratie Product => gebruik nieuwe filter

- Bepaler ExtraLeveringsDagen volgens LeverancierNummer in VoorraadInstellingCalculator

- Filter om te bepalen of een product een generisch Orgalux product is

- Deze filter geïnjecteerd in ProductDixfConverter

- Implementatie ADUCalcTechnologyBOMBOL aangepast volgens filter

- OmschrijvingSamensteller in ProductDixfConverter verwijderd en als implementatie GetOmschrijving() gebruikt

- UnitTests ProductDixfConverter aangepast aan injectie en implementatie filter

- Ter voorbereiding van aanpassingen, refactoring en UnitTests

- Aanpassingen ter voorbereiding (methods -> public gemaakt + herschikking public/private methods)

- 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

- Aanpassingen hieromtrent in code PV

- Refactoring (types,dims)

- do ##class(Tools.CacheSourceUtils).MoveClass("vhTest.Mock.AX.Uitgaand.PM.impl.ConvertorLeveranciersnummerVanHoeckeNaarHalux","vhTest.Mock.AX.Uitgaand.PM.impl.Convertor.ConvertorLeveranciersnummerVanHoeckeNaarHalux")

- do ##class(Tools.CacheSourceUtils).MoveClass("vhTest.Fake.AX.Uitgaand.PM.impl.ConvertorLeveranciersnummerVanHoeckeNaarHalux","vhTest.Fake.AX.Uitgaand.PM.impl.Convertor.ConvertorLeveranciersnummerVanHoeckeNaarHalux")

- Verwijderen eerder ingevoegd veld ter controle (controle werd overbodig)

- Vestiging is nu enum ipv string

- MoveClass("vhTest.Fake.AX.Uitgaand.common.impl.MessageManager","vhTest.Fake.AXimpl.MessageManager")

- MoveClass("vhTest.Fake.AX.Uitgaand.common.impl.AifGenerator","vhTest.Fake.AXimpl.Generator")

- MoveClass("AX.common.impl.ConvertAXToAdmin","AXimpl.AxToAdminUtils")

- Code Paul:

+ Als LeveringsDagen = 0 dan BLID = ""

+ Voor Blum: ExtraLeveringsDagen = 3

- Returntype method ConvertorLeveranciersnummerVanHoeckeNaarHalux ontbrak