WSimpl.AX.CalculatedProduct

Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Die WD die bij alle testen wegvalt zal wel oke zijn denk ik? ��

Die WD die bij alle testen wegvalt zal wel oke zijn denk ik? 😋

hier ook

hier ook

nog een wlipke te verwijderen

nog een wlipke te verwijderen

[ICT-4623] [rvPVR] Integratietesten aansturingacties via Encoway afwerken:
[ICT-4623] [rvPVR] Integratietesten aansturingacties via Encoway afwerken:
TestFiles reviewen, is in dit soort stories bijna onbegonnen werk. Ik heb zo goed en zo kwaad mogelijk de changes bekeken. Alvast bedankt om het aantal testfiles te filteren. Het geheel lijkt me we...

TestFiles reviewen, is in dit soort stories bijna onbegonnen werk. Ik heb zo goed en zo kwaad mogelijk de changes bekeken.
Alvast bedankt om het aantal testfiles te filteren.
Het geheel lijkt me wel oké te zijn.

Ook in eCon enkele commits gereviewed.

De dossiercode "LI7K" was blijkbaar een onbedoelde tussenstap (Tricky situatie met de TAOR Losse componenten) In combinatie met svn rev. 69413 is dit wel een correcte overgang. Hierbij is deze cha...

De dossiercode "LI7K" was blijkbaar een onbedoelde tussenstap (Tricky situatie met de TAOR Losse componenten)
In combinatie met svn rev. 69413 is dit wel een correcte overgang.

Hierbij is deze change nagekeken en goedgekeurd!

De meeste testfiles heb ik maar buiten de review gehouden. Een aantal zijn toch toegevoegd, bij wijze van check van het resultaat is dat, denk ik, wel ok en het is bovendien soms ook moeilijk om ui...

De meeste testfiles heb ik maar buiten de review gehouden. Een aantal zijn toch toegevoegd, bij wijze van check van het resultaat is dat, denk ik, wel ok en het is bovendien soms ook moeilijk om uit te maken a.d.h.v. de changesets of het al dan niet over code gaat of testfiles.
P.s.: Ook kleine change in Econ (zie svn).

[ICT-3120] [rvWV] PM: TAX: Front HoogteVerstelling MoventoKoppeling: release alle klanten:
[ICT-3120] [rvWV] PM: TAX: Front HoogteVerstelling MoventoKoppeling: release alle klanten:
OEPS !!! Great call!!! Thx!!!

OEPS !!! Great call!!! Thx!!!

Het oude productId komt nog op 2 andere plaatsen voor in caché studio. Deze best ook aanpassen. Searching for 1118152 in *.cls,*.mac,*.inc APPS.PM.Maatwerk.MaatwerkModel.Modellen.TaorboxModel.Rolle...

Het oude productId komt nog op 2 andere plaatsen voor in caché studio. Deze best ook aanpassen.

Searching for 1118152 in *.cls,*.mac,*.inc
APPS.PM.Maatwerk.MaatwerkModel.Modellen.TaorboxModel.Rollen.ZijkantBevestigingVeer.cls(GeefRol+2): ##class(APPS.PM.Maatwerk.MaatwerkModel.dto.SpecifiekProductBuilderType).%New(1118152)
DOM.PM.Maatwerk.Syncs.DotNet.MaatwerkCalculator.impl.ProductInfo.cls(GeefLijstVanProductIdVanNietSelectieKenmerken+102): Do ProductIds.Insert(1118152) 
Found 2 occurrence/s in 2 file/s.
[ICT-3034] [rvWV] Andere veer Vällinge koppeling TA'OR:
[ICT-3034] [rvWV] Andere veer Vällinge koppeling TA'OR:
zie andere review van dit kaartje

zie andere review van dit kaartje

Aangezien er met VerwachteUitkomst niks gebeurt tot op het einde van de test, is het niet per sé nodig om die hier al te dimmen. Je zou de assert evengoed kunnen reduceren tot Do $$$AssertEquals("O...

Aangezien er met VerwachteUitkomst niks gebeurt tot op het einde van de test, is het niet per sé nodig om die hier al te dimmen. Je zou de assert evengoed kunnen reduceren tot
Do $$$AssertEquals("Omschrijving in het Nederlands"$$$CRLF"Kleur:VertalingKleur",DescriptionBepaler.GeefDescription(ProductMock,RandomTaal))
Hoe dan ook is dat iets waarover je kan blijven discussiëren: langere lijnen en minder variabelen, of kortere lijnen en meer one-time-use variabelen...
Indien je toch gaat voor meer variabelen: de gewoonte is om de #dim's niet allemaal bovenaan te smijten, maar vlak voordat je ze nodig hebt

Maak hier maar ineens een Setterke van, want zoals je ziet kan je in dit geval (lange namen) niet zien of er dingen aan de constructor worden meegegeven. Idem voor andere #dim/Set combinaties in de...

Maak hier maar ineens een Setterke van, want zoals je ziet kan je in dit geval (lange namen) niet zien of er dingen aan de constructor worden meegegeven.
Idem voor andere #dim/Set combinaties in deze en de andere test

om goed te zijn zou je ook een test moeten voorzien die laat zien dat er geen Nederlands gebruikt wordt indien de omschrijving in andere taal niet ontbreekt

om goed te zijn zou je ook een test moeten voorzien die laat zien dat er geen Nederlands gebruikt wordt indien de omschrijving in andere taal niet ontbreekt

[ICT-883] UAT 1422: verkeerde tekst in engels document
[ICT-883] UAT 1422: verkeerde tekst in engels document
Variabele moet zijn : VerwachtPercentage

Variabele moet zijn : VerwachtPercentage

[VRB627] VHIP481 - UAT1305- uitval% en nettohoeveelheid doorgeven aan AX:
[VRB627] VHIP481 - UAT1305- uitval% en nettohoeveelheid doorgeven aan AX: