CalculatedProduct

- klasse KlantLinkKaderdeurDetailbladPdfBepaler aangemaakt

- input Product moet van het type "DOM.PM.KADProduct.KADKaderdeur" zijn, anders exception

- deze implementatie voorziet ook meteen logging

- Implementatie GeefProductIdVrij van ProductIdRecycleerder i.g.v. AttrVsAdminProdNr = SalesPriceBOM

- Bijhorende UnitTest

- Commentjes weggehaald en overgebleven comment verduidelijkt.

- Logging is hier in principe niet nodig, wegens al voldoende voorzien in helperklaskes

- Als ProductIdRecycleerder geen ProductId teruggeeft => ook volledig nieuw product maken => BOMBOL moet robust zijn en zoveel mogelijk AX dienen

- Al een 1e testje => ProductIdRecycleerder gaf ProductId => Product bestaat nog niet => Nieuw product volgens kenmerken

- Logging => ProductId werd uitbesteed voor LadeVariant

- Werkte wel al, maar juist is juist => gaat enkel over 1e piece in de data

- Error-logging i.g.v. IsVrijeProductIdGevonden = false

- Check op LadeVariant gebeurt al in de LadeVariantBepaler, dus niet hier nog eens nodig

- Klein refactorke => Logging bij schaarste afgezonderd naar private method

- Logging bij GeefLadeVariantVanRangeVolgensProductId

- Console geïnjecteerd in constructor => voor testdoeleinden

- Parameters vervangen door Properties => makkelijk te wijzigen in TestCase

- CheckRangeOpBeschikbaarheidProductIds => Uitgebreid met check op aanwezigheid van ProductId's in ^KPRO

- 1e TestCase-opzet met al 2 testjes voor CheckRangeOpBeschikbaarheidProductIds

- GlobalNaam is nu een klasse-parameter => makkelijk te wisselen in toekomstige TestCase

- Ranges van LadeTypes zijn nu allemaal Klasse-parameters => Opzichtiger met oog op uitbreidingen en eventueel ook makkelijker te manipuleren in UnitTesten

- In method GeefAantalVrijePlaatsenInRangeVolgensLadeVariant => Indien AantalNietGepersisteerdeProductIdsInRange > 0 => automatisch persisteren => geautomatiseerde werking bij uitbreiding van een range

- BestaatProduct toegevoegd om makkelijk los product te checken voor uitbreiding van een range => potentiële uitbreiding voor later => nieuwe range (meegegeven als parameter) checken die rapporteert welke ProductId's in die range nog vrij zijn

- Exceptie bij ongekende LadeVariant verwijderd => reeds voorzien in gebruikte LadeVariantBepaler

- Implementatie check op AantalVrijePlaatsenInRangeVolgensLadeVariant => na elke call => logging nog te vervolledigen

- Voor bovenstaande => implementatie GeefAantalVrijePlaatsenInRangeVolgensLadeVariant => stelt method call samen volgens LadeVariant => daarom methods zoals GeefLBXRange => LadeType in upper case gezet => is trouwens ook leesbaarder

- PersisteerRanges en PersisteerRange werken nu met parameter ForceerVolledigeReset met default False => Bij uitbreiding range enkel persistering van nieuwe, tenzij ForceerVolledigeReset = True, dan volledig nieuwe persistering (= alle switches op off = geen ProductId's in gebruik)

-

- Eerste opzet => Opsplitsing volgens verschillende mogelijkheden voorzien => implementatie in comment (pseudocode) => doet dus momenteel nog niets

- Wel al injectie van dependancies, waaronder nieuwe ProductIdRecycleerder (zie vorige commit)

- Eerste opzet ProductIdRecycleerder => Geeft voorlopig altijd lege string terug als ProductId.

- Al ineens mockable gemaakt voor testing => TODO

- Wel al aantal voorzieningen:

=> 4 ranges (LBX, MVX, TAX en TBX) gedefinieerd als komma seperated strings => voorziet in flexibele aanpassing en uitbreiding van ranges

=> Controle of initieel ranges wel effectief vrij zijn (per LadeType of ineens alle ranges) => via BestaatProduct op ProductApi => OPGELET: Manueel via PuTTy in gang te zetten => Volgt misschien nog een deployklaske voor

=> Persistering van 4 ranges in dedicated Global ^ProductIdRecycle => Om bij te houden welke ProductId's op dat moment in gebruik zijn => OPGELET: Manueel via PuTTy in gang te zetten => Volgt misschien nog een deployklaske voor

=> Dit laatste kan ook gebruikt worden om, na indienstname, alle ProductId's (per LadeType of ineens alle ranges) weer volledig vrij te geven => USE WITH CAUTION