ProductVolgensAxAttribuutBepaler

Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
[ICT-4920

Mail productupdate AX

    • -35
    • +2
    ./OrderService/ToeleveringService.cls.xml
  1. … 2 more files in changeset.
[ICT-2090] [rvTVE] PM: Maatwerk: BOMBOL VHIP481: Soms opvragen zonder product aan te maken:

- Na review: Extra testje voor RequestConverter indien NIET SalesPriceBOM

- Casing aangepast voor leesbaarheid => vermijden dat sommige mensen in 1e instantie denken dat het over een event gaat :-D

  1. … 1 more file in changeset.
Testje bijgezet als AttrVsAdminProdNr niet SalesPriceBOM is => dan moet er betreffende de ProductIdRecycleerder niets gebeuren.

Testje bijgezet als AttrVsAdminProdNr niet SalesPriceBOM is => dan moet er betreffende de ProductIdRecycleerder niets gebeuren.

Ik vond dat nogal overkill, omdat die list dan amper een codelijn of 6 later alweer omgezet zou moeten worden naar een delimited string. Moest de omzetting naar delimited string in een andere metho...

Ik vond dat nogal overkill, omdat die list dan amper een codelijn of 6 later alweer omgezet zou moeten worden naar een delimited string. Moest de omzetting naar delimited string in een andere method gebeuren, dan zou ik het zeker gedaan hebben

Ook zoals aan telefoon besproken... het gaat over de LadeVariant zoals gekend in AX, waarvoor overal in de BOMBOL de AXimpl gebruikt wordt.

Ook zoals aan telefoon besproken... het gaat over de LadeVariant zoals gekend in AX, waarvoor overal in de BOMBOL de AXimpl gebruikt wordt.

Zoals besproken aan de telefoon: Dat Request komt in de BOMBOL-interface ook al binnen als parameter, dus de afhankelijkheid is sowieso al een feit. Bovendien wordt er 1 laagje hoger ook een andere...

Zoals besproken aan de telefoon: Dat Request komt in de BOMBOL-interface ook al binnen als parameter, dus de afhankelijkheid is sowieso al een feit. Bovendien wordt er 1 laagje hoger ook een andere property van uitgelezen

testje(s) voor de andere gevallen?

testje(s) voor de andere gevallen?

ikzelf probeer constructies als deze meestal te vermijden omdat het om de zoveel tijd wel eens kan leiden naar een rariteit met die delimiter. Misschien beter om een list te gebruiken? Kunt ge gewo...

ikzelf probeer constructies als deze meestal te vermijden omdat het om de zoveel tijd wel eens kan leiden naar een rariteit met die delimiter. Misschien beter om een list te gebruiken? Kunt ge gewoon op adden indien nodig. Een list kan daarna 'gewoon' in een delimited string omgezet worden indien nodig.

minor nitpicking: ik las dit als een Event "OnJadajada" en het duurde toch een fractie van een seconde voor ik doorhad dat dat niet echt logisch zou zijn indeze context. Een kleine "B" was misschie...

minor nitpicking: ik las dit als een Event "OnJadajada" en het duurde toch een fractie van een seconde voor ik doorhad dat dat niet echt logisch zou zijn indeze context. Een kleine "B" was misschien logische geweest. Nogmaals: kommaneuken hoor

een beetje in de trand van de vorige opmerking: zou het niet properder zijn om de DOM....LadeVariant te gebruiken (qua hiërarchie hé). Je gaat dan wel een 'converter' moeten opzetten die de AX-vari...

een beetje in de trand van de vorige opmerking: zou het niet properder zijn om de DOM....LadeVariant te gebruiken (qua hiërarchie hé). Je gaat dan wel een 'converter' moeten opzetten die de AX-variant omzet naar een DOM-variant, maar daar kan de LadeVariantBepaler zich mee bezighouden. Op die manier verdwijnt de dependency op AXimpl.* package

Van deze request wordt enkel Request.Attribute gebruikt. Het zou beter zijn om die attribute als parameter mee te geven, anders introduceer je een (extra?) afhankelijkheid tss WSimpl en AXif packages.

Van deze request wordt enkel Request.Attribute gebruikt. Het zou beter zijn om die attribute als parameter mee te geven, anders introduceer je een (extra?) afhankelijkheid tss WSimpl en AXif packages.

[ICT-2090] [rvTVE] PM: Maatwerk: BOMBOL VHIP481: Soms opvragen zonder product aan te maken:
[ICT-2090] [rvTVE] PM: Maatwerk: BOMBOL VHIP481: Soms opvragen zonder product aan te maken:
[ICT-2090] [rvTVE] PM: Maatwerk: BOMBOL VHIP481: Soms opvragen zonder product aan te maken:

- In GeefLadeVariantVanRangeVolgensProductId, ENKEL gebruikt door GeefProductIdVrij => Indien LadeVariant niet gevonden => Error-logging afgezwakt naar Info => Kan immers normale verwachte case zijn =>

=> indien range op was bij uitbesteding heeft ProductIdRecycleerder lege ProductId teruggegeven => ProductVolgensAxAttribuutBepaler heeft daarop een geheel nieuw Product gemaakt, waarvan de ID vervolgens in de RequestConverter wordt gebruikt om GeefProductIdVrij-call mee te doen

  1. … 1 more file in changeset.
[ICT-2090] [rvTVE] PM: Maatwerk: BOMBOL VHIP481: Soms opvragen zonder product aan te maken:

- Check op timestamp en indien nodig automatische vrijgave van ProductId's en indien nodig logging

- Bijhorende UnitTest

  1. … 1 more file in changeset.
[ICT-2090] [rvTVE] PM: Maatwerk: BOMBOL VHIP481: Soms opvragen zonder product aan te maken:

- Logging => ProductId vrijgegeven voor LadeVariant

  1. … 1 more file in changeset.
[ICT-2090] [rvTVE] PM: Maatwerk: BOMBOL VHIP481: Soms opvragen zonder product aan te maken:

- Logging => ProductId werd uitbesteed voor LadeVariant

  1. … 1 more file in changeset.
[ICT-2090] [rvTVE] PM: Maatwerk: BOMBOL VHIP481: Soms opvragen zonder product aan te maken:

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

[ICT-2090] [rvTVE] PM: Maatwerk: BOMBOL VHIP481: Soms opvragen zonder product aan te maken:

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

  1. … 1 more file in changeset.
[ICT-2090] [rvTVE] PM: Maatwerk: BOMBOL VHIP481: Soms opvragen zonder product aan te maken:

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

[ICT-2090] [rvTVE] PM: Maatwerk: BOMBOL VHIP481: Soms opvragen zonder product aan te maken:

- In de dedicated Global wordt in de data nu een 2e piece bijgehouden bij gebruik van een ProductId met daarin een Horolog

  1. … 2 more files in changeset.
[ICT-2090] [rvTVE] PM: Maatwerk: BOMBOL VHIP481: Soms opvragen zonder product aan te maken:

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

[ICT-2090] [rvTVE] PM: Maatwerk: BOMBOL VHIP481: Soms opvragen zonder product aan te maken:

- Logging bij GeefLadeVariantVanRangeVolgensProductId

  1. … 1 more file in changeset.
[ICT-2090] [rvTVE] PM: Maatwerk: BOMBOL VHIP481: Soms opvragen zonder product aan te maken:

- Logging bij PersisteerRange

  1. … 1 more file in changeset.
[ICT-2090] [rvTVE] PM: Maatwerk: BOMBOL VHIP481: Soms opvragen zonder product aan te maken:

- Uitleg in comment ivm uitbreiding van ranges verduidelijkt

- Logger geïnjecteerd in constructor

- Logging uitgewerkt voor indien Schaarste in de range

- Bijhorende UnitTest

  1. … 2 more files in changeset.
[ICT-2090] [rvTVE] PM: Maatwerk: BOMBOL VHIP481: Soms opvragen zonder product aan te maken:

- CheckRangeOpBeschikbaarheidProductIds uitgebreid met 2e parameter => opgegeven komma delimited string van ProductIds checken => voor uitbreidingen

- Lokale BestaatProduct verwijderd => is door bovenstaande uitbreiding overbodig geworden

- 2 extra testjes hieromtrent

  1. … 2 more files in changeset.