APPS.PM.Maatwerk.End2End.MAT

Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
[ICT] Update masterdata door PM
[ICT-3368] Testrunner Maatwerkcalculator

- Afronding op 6 getallen brengen

  1. … 47 more files in changeset.
[ICT-2836][rvPVR] LBX BomBol .Net klaarzetten en testen vanuit AX:

- End2End bestanden na omleiden End2End naar .Net. End2End testen voor de matten gaan nu ook langs .Net

[ICT-2873][rvNVT] ICT: De Grote Prijswijziging van 01/05/2022 : ICT automatische testen aanpassen

- UnitTests End2End MAT

[ICT-2522] Update testbestanden voor prijswijzigingen 1 oktober 2021
De vhUtils standaard-kenmerken kiezen is soms nogal tricky: *liefst zo standaard mogelijk (behalve bij speciallekes zoals LadeMetVeelExtras) *liefst zo ondubbelzinnig mogelijk : bvb. X en Y versc...

De vhUtils standaard-kenmerken kiezen is soms nogal tricky:

  • liefst zo standaard mogelijk (behalve bij speciallekes zoals LadeMetVeelExtras)
  • liefst zo ondubbelzinnig mogelijk : bvb. X en Y verschillend bij sifonlade --> check.
  • in de meeste gevallen zo werkelijkheidsgetrouw mogelijk


in deze method vind ik enige verwarring :

  • StandaardMat heeft LadeType=TAX, die heeft (nog) geen spoelbak.
  • Spoelbak voor LBX? of eerder voor TBX/MVX? --> wellicht zijn beide methods zinvol, en dan zou ik één method expliciet MatVoorLbxSpoelbakLade() noemen.
  • bij deze LBX moeten de uitsparingBreedtes X en Y dan best wel de juiste afmeting bevatten, want dat is ook een "vaste maat"


In eerdere gesprekken had ik al wel aangegeven dat een goede keuze van kenmerkwaarden, niet zo vanzelfsprekend is, en dat ik je hierbij zeker wel zou bijstaan. Bij deze, call me

Ik heb nog wat uitleg over die "E" "Exceptions" in de Json-export, maar dat is wat veel om hier in de review-opmerking te zetten. Contacteer me maar eens. Eventueel zelfs interessant voor een meeti...

Ik heb nog wat uitleg over die "E" "Exceptions" in de Json-export, maar dat is wat veel om hier in de review-opmerking te zetten.
Contacteer me maar eens. Eventueel zelfs interessant voor een meeting met JBA en THB.

copy-paste typo (2x) : #dim ... As MVX-kenmerken ??

copy-paste typo (2x) :
#dim ... As MVX-kenmerken ??

copy-paste 2x zelfde

copy-paste 2x zelfde

copy-paste 2x zelfde

copy-paste 2x zelfde

Mijn eerste review-opmerking ging zijn : "In een AssertEquals-expressie zou geen $Select() mogen staan." Dat topic wil ik eventueel nog met jou bespreken. Door dit op te merken, viel m'n oog wel o...

Mijn eerste review-opmerking ging zijn : "In een AssertEquals-expressie zou geen $Select() mogen staan."
Dat topic wil ik eventueel nog met jou bespreken.

Door dit op te merken, viel m'n oog wel op een typo in bovenstaande lijn: RugBreedteX en in de $Select() staat BreedteM :-O

Bij voorkeur Z1 en Z2 verschillend kiezen. idem voor X en Y

Bij voorkeur Z1 en Z2 verschillend kiezen.
idem voor X en Y

Uitsparing Breedte.M moet je verplaatsen naar Dubbele uitsparing.

Uitsparing Breedte.M moet je verplaatsen naar Dubbele uitsparing.

Deze IF legt een onnodige dependency, en heeft geen meerwaarde binnen de converter. Integendeel, bij uitbreiding voor LBX sifon of TBX sifon moet je hier iets toevoegen, terwijl de code binnen de I...

Deze IF legt een onnodige dependency, en heeft geen meerwaarde binnen de converter.
Integendeel, bij uitbreiding voor LBX sifon of TBX sifon moet je hier iets toevoegen, terwijl de code binnen de IF toch "universeel" is.

Ook de MatKenmerken hebben property "NietMeeleveren", dus ik zou deze method wel laten oproepen, ook al is de impl dan enkel met de eerste en de laatste lijn : %New() en Quit. Oftewel rechtstreeks ...

Ook de MatKenmerken hebben property "NietMeeleveren", dus ik zou deze method wel laten oproepen, ook al is de impl dan enkel met de eerste en de laatste lijn : %New() en Quit.
Oftewel rechtstreeks in de method Converteer() , dan mag de commentaar hier volledig weg.

VH code conventions : VulAanMatKenmerken()

VH code conventions : VulAanMatKenmerken()

Als het object MatKenmerken één en dezelfde oref blijft (dus geen %Clone ofzo) dan is het niet nodig om de SetAt() te doen, want de pointer in de GekoppeldeKenmerken verwjist nog steeds naar hetzel...

Als het object MatKenmerken één en dezelfde oref blijft (dus geen %Clone ofzo) dan is het niet nodig om de SetAt() te doen, want de pointer in de GekoppeldeKenmerken verwjist nog steeds naar hetzelfde object in memory.

Een kleine bedenking: ik ben niet helemaal zeker of de method wel thuishoort in deze klasse, omdat die enkel geschikt is voor MatKenmerken. Voorlopig is dit oké, en functioneel zeker geen probleem...

Een kleine bedenking:
ik ben niet helemaal zeker of de method wel thuishoort in deze klasse, omdat die enkel geschikt is voor MatKenmerken.

Voorlopig is dit oké, en functioneel zeker geen probleem.
Ik wil niet verder uitweiden nu, over deze bedenking, dus enkel als review-opmerking, en om te vermijden dat we vanalles in deze klasse gaan steken dat niet voor alle IngegevenKenmerken van toepassing is.

Er is geen IF nodig, want : if "true" or "false" --> altijd dus :-P

Er is geen IF nodig, want :
if "true" or "false" --> altijd dus
:-P

toegevoegde parameter "IsBerekendViaDotNet" ook doorgeven via ##super()

toegevoegde parameter "IsBerekendViaDotNet" ook doorgeven via ##super()

toegevoegde parameter "IsBerekendViaDotNet" ook doorgeven via ##super()

toegevoegde parameter "IsBerekendViaDotNet" ook doorgeven via ##super()

[ICT-2296] Uitbreidingen in cache voor nieuwe mattenconfigurator
[ICT-2296] Uitbreidingen in cache voor nieuwe mattenconfigurator
[ICT-2296] Uitbreidingen in cache voor nieuwe mattenconfigurator

Update testbestanden voor:

- Defaulting corpusbreedte en wanddikte 0 verwijderen, dit wordt nu opgevangen in de DommNaaarDotNet convertor

  1. … 17 more files in changeset.
[ICT-2437] Update testbestanden voor prijswijzigingen 1 juli 2021
[ICT-2296] Uitbreidingen in cache voor nieuwe mattenconfigurator

- Updaten testen voor: Initial values invullen voor corpuswanddikte en corpusbreedte, anders wordt die afhankelijk van het pad leeg gelaten en .NET kan daar niet mee om

  1. … 15 more files in changeset.
[ICT-2296] Uitbreidingen in cache voor nieuwe mattenconfigurator

- Nieuwe merge van testbestanden

  1. … 147 more files in changeset.
[ICT-2296] Uitbreidingen in cache voor nieuwe mattenconfigurator

- Reverse merge want ik heb een klasse te veel gecommit ivm TAOR ladeOnderdeelnfoBepaler

  1. … 147 more files in changeset.