Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
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....

Geen engelstalige stukken in naamgeving Code wordt overbodig want GeefBon gebruiken en dan verder ook Bon.GeefVerzendwijze

Geen engelstalige stukken in naamgeving
Code wordt overbodig want GeefBon gebruiken en dan verder ook Bon.GeefVerzendwijze

// GeefVerzendwijze methode gebruiken uit apps.common.dataservice ipv zelf rechtstreeks %OpenId te doen

// GeefVerzendwijze methode gebruiken uit apps.common.dataservice ipv zelf rechtstreeks %OpenId te doen

//$Not van maken ipv '

//$Not van maken ipv '

// GeefBon --> Geef klant --> via objecten oproepen, niet rechtstreeks via de global

// GeefBon --> Geef klant --> via objecten oproepen, niet rechtstreeks via de global

//$Not van maken in plaats van ..VerzendwijzeUitgesteldPakbonBoekenAx(bon, klant) = $$$False

//$Not van maken in plaats van ..VerzendwijzeUitgesteldPakbonBoekenAx(bon, klant) = $$$False

een else kant toevoegen met een log else{ // log toevoegen dat event uitgesteld zal worden verzonden }

een else kant toevoegen met een log

else

Unknown macro: { // log toevoegen dat event uitgesteld zal worden verzonden }
// var hernoemen naar GeneratePakbonMagGeboektWordenResult en ook het vereder gebruik , argument in methode aanpassen

// var hernoemen naar GeneratePakbonMagGeboektWordenResult en ook het vereder gebruik , argument in methode aanpassen

Niet gewoon gebruik maken van de globals maar de gemapte objecten gebruiken zoals Geefbon Daaraan een methode toevoegen als en Zetvlag(new) De zet vlag zal dan onderstaande set piece bevatten

Niet gewoon gebruik maken van de globals maar de gemapte objecten gebruiken zoals Geefbon
Daaraan een methode toevoegen als en Zetvlag(new)
De zet vlag zal dan onderstaande set piece bevatten

ctor toevoegen en injecteren van de geeforderapiservice niet via context

ctor toevoegen en injecteren van de geeforderapiservice niet via context

VerwerkRecord hernoemen naar VerwerkBon (dit is iets duidelijker)

VerwerkRecord hernoemen naar VerwerkBon (dit is iets duidelijker)

Naamgevingen verduidelijken *Geen engelse stukken vb VerzendwijzeLONG.. *{} plaatsen volgens cache standaard *Zie ook confluence notities voor dit ticket
  • Naamgevingen verduidelijken
  • Geen engelse stukken vb VerzendwijzeLONG..
  • {} plaatsen volgens cache standaard
  • Zie ook confluence notities voor dit ticket
[ICT-1078] Fase 2 sturen bericht dat pakbon mag geboekt worden
[ICT-1078] Fase 2 sturen bericht dat pakbon mag geboekt worden
  • More
  • CR-946
  • finished reviewing
Do ipv do

Do ipv do

Do ipv DO (ook hieronder)

Do ipv DO (ook hieronder)

ProductApiMock ipv ProductAPI

ProductApiMock ipv ProductAPI

Kunt ook de Logger injecteren en een test met een VerwachtMethodCall("Error") indien de geïnjecteerde OrderAPI.VerwijderOrder() $$$False teruggeeft Best de ORDNr die in de string gestoken wordt, me...

Kunt ook de Logger injecteren en een test met een VerwachtMethodCall("Error") indien de geïnjecteerde OrderAPI.VerwijderOrder() $$$False teruggeeft
Best de ORDNr die in de string gestoken wordt, met single quotes omringen