Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Opgelet: wanneer de While() over meerdere items loopt, dan geldt het Result van de laatste. Wellicht is de veronderstelling dat de ZoekViaKlantId() slechts één item oplevert. Dan toch beter geen Wh...

Opgelet: wanneer de While() over meerdere items loopt, dan geldt het Result van de laatste.
Wellicht is de veronderstelling dat de ZoekViaKlantId() slechts één item oplevert.
Dan toch beter geen While() uitvoeren, maar enkel een If HasNext() then ...
Ultiem kan je nog controleren of een nadien nog een HasNext() is en dan een Warning loggen of een error smijten.

Good one! Merci voor de hint!!

Good one! Merci voor de hint!!

Dat leek mij wat overkill, omdat de relevante change slechts op 1 plek in 1 klasse zit. Ik dacht dat een ProjectSetting gebruikt werd als je met vele changes in vele klassen zit, die tegelijkertijd...

Dat leek mij wat overkill, omdat de relevante change slechts op 1 plek in 1 klasse zit. Ik dacht dat een ProjectSetting gebruikt werd als je met vele changes in vele klassen zit, die tegelijkertijd aan/af moeten kunnen gezet worden.

Is voor het stukje hieronder eigenlijk de ProjectSettingsApi niet bruikbaar?

Is voor het stukje hieronder eigenlijk de ProjectSettingsApi niet bruikbaar?

klein detail (wrschl niet de moeite om aan te passen): In bovenstaand geval is het soms interessanter/properder om het volgende te doen: If $$$AssertEquals(..OpmerkingenLijst.Count(),2) {{ ... de ...

klein detail (wrschl niet de moeite om aan te passen): In bovenstaand geval is het soms interessanter/properder om het volgende te doen:

If $$$AssertEquals(..OpmerkingenLijst.Count(),2) {{
... de andere asserts
}} (dubbele accolades omdat die Jira anders denkt dat het een macro is)

Stel dat het aantal 1 is, krijg je maar 1 test-errormelding, anders gaat de .GetAt(2) ook nog eens een melding geven die je misschien op het verkeerde been zet bij het proberen fixen

[ICT-2289] [rv TVE] Bidi: Leverbon: datum backorders toevoegen:
[ICT-2289] [rv TVE] Bidi: Leverbon: datum backorders toevoegen:
ik zou onderstaande private method op de DOM.VKP.impl.LeverAdresRepository zetten. Dan kan ie hier proper uitgemocked worden. Die repo bevat ook een hoop testen die gemakkelijk aangepast kunnen wor...

ik zou onderstaande private method op de DOM.VKP.impl.LeverAdresRepository zetten. Dan kan ie hier proper uitgemocked worden. Die repo bevat ook een hoop testen die gemakkelijk aangepast kunnen worden om onderstaand gedrag te testen.

't is wel zo dat die Repository geen pure repo is, maar ook functionaliteit heeft die eigenlijk in een bovenliggende service zouden moeten zitten (maar dat is een ander verhaal)

Misschien kan de onderstaande functionaliteit dan ook vervangen worden door iets als in BestaatViaAXLeverAdresIdEnKlantNummer, maar dan met de query SELECT count(ID) from Derde_Klant.LevAdres where Klant = ? (zonder de Ax-dingen)

[ICT-2246] [rvTVE] Correctie loggingniveau betreffende LeverAdressen op Klant en BezwaarGevonden in...
[ICT-2246] [rvTVE] Correctie loggingniveau betreffende LeverAdressen op Klant en BezwaarGevonden in...
[ICT-1876] BIDI: Orderingave: Error zGeefIngegevenKenmerken in BLDTOE+29^FLOWORD7 (bij order2admin):...
[ICT-1876] BIDI: Orderingave: Error zGeefIngegevenKenmerken in BLDTOE+29^FLOWORD7 (bij order2admin):...
[ICT-1591] PA-ub-125: wegwerppallet op basis van klantinstellingen
[ICT-1591] PA-ub-125: wegwerppallet op basis van klantinstellingen
test of meerdere uitleveringen goed afgehandeld worden, cfr opm repository over flow.uitlevering.hoofding

test of meerdere uitleveringen goed afgehandeld worden, cfr opm repository over flow.uitlevering.hoofding

Repository maken over Flow.Uitlevering.Hoofding ? encapsulaties gebruiken waar het kan? overweeg met PRA om te zien of het de investering waard is. Indien niet gebruik maken van encapsulatie: nu wo...

Repository maken over Flow.Uitlevering.Hoofding ? encapsulaties gebruiken waar het kan? overweeg met PRA om te zien of het de investering waard is.
Indien niet gebruik maken van encapsulatie:
nu wordt 2 x $Order uitgevoerd voor één uitlevering. zo ook voor de volgende whiles
Dat kan beter
Na de rest van de code te bekijken : Repository maken (resulteert in één iterator en testbaarheid of meer dan één uitlevering correct afgehandeld worden.

Dit gaat alleen voor de eerste bon werken niet meer voor de 2de.

Dit gaat alleen voor de eerste bon werken niet meer voor de 2de.

ik zou dit toch over een paar lijnen spreiden, gaat wat leesbaarder worden dan.

ik zou dit toch over een paar lijnen spreiden, gaat wat leesbaarder worden dan.

Private?

Private?

Wat doen we met Data?

Wat doen we met Data?

grrrrr...

grrrrr...

grrrr....

grrrr....

Maak hier een specifieke method voor waarvan de naam betekenis heeft. 63 zegt niet veel. cfr andere opmerking op de impl hiervan.

Maak hier een specifieke method voor waarvan de naam betekenis heeft. 63 zegt niet veel. cfr andere opmerking op de impl hiervan.

cfr andere opmerking in implementatie

cfr andere opmerking in implementatie

Lijkt me 2 mooie methods om private te zetten en dan in specifiek genaamde methods voor speciefieke gegevens te gebruiken. Nu staat er ergens anders in de code een getal 63 te blinken....

Lijkt me 2 mooie methods om private te zetten en dan in specifiek genaamde methods voor speciefieke gegevens te gebruiken.
Nu staat er ergens anders in de code een getal 63 te blinken....