CalculatedProduct

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

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

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

Telegramstijl, omdat ze anders te lang worden en de variaties nu eenmaal soms ingewikkeld zijn

Idem reply hierboven.

Zoals besproken... Dit ging over de property-namen, maar... toch liever niet. In de properties kunnen soms mocks, soms stubs gestoken worden, afhankelijk van wat de UnitTest nodig heeft.

Done

Done! (smartass )

Done!

Done, maar staat momenteel al nieuwe openstaande wijziging op die klasse, die nog niet gecommit mag worden, dus komt daarna wel.

Done, maar staat momenteel al nieuwe openstaande wijziging op die klasse, die nog niet gecommit mag worden, dus komt daarna wel.

Done en done en done!

Normaal binnen de scope blijven, ik weet het, maar ik dacht de while op deze manier zo proper en leesbaar mogelijk te houden. Uiteindelijk wordt elk van die dim'ekes toch in die while gebruikt en omhelst die while de volledige method.

Denk het niet. Nu krijgt ge alle OrderLijnIterators netjes op een rijtje via code completion. Eentje is dan de volledige, sommige zijn "via" dit, andere "via" dat, ...

Had ik al aangepast inmiddels, zit al in volgende review