Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Geen idee of het overkill is, maar misschien is het handig als je een lijst of array van VHConfigs ook kan meegeven? Want ik vermoed dat als er iets fout is, dit voor meerder CIDs zal zijn?

Geen idee of het overkill is, maar misschien is het handig als je een lijst of array van VHConfigs ook kan meegeven? Want ik vermoed dat als er iets fout is, dit voor meerder CIDs zal zijn?

[ICT-4554] [rvJBA] Powertool maken om VhConfigs op te halen indien CIDs niet geldig
[ICT-4554] [rvJBA] Powertool maken om VhConfigs op te halen indien CIDs niet geldig
Ik had deze initieel bij Kris (als reviewer) gezet, maar dat was wellicht geen goed idee :-P

Ik had deze initieel bij Kris (als reviewer) gezet, maar dat was wellicht geen goed idee :-P

[ICT] Powertool ExterneVerkoopPowerservice
[ICT] Powertool ExterneVerkoopPowerservice
[ICT-610] [rvJWI] VRB650 - VH4 MP: ConfirmOrder - UpdateCartStatus via backgroudJob
[ICT-610] [rvJWI] VRB650 - VH4 MP: ConfirmOrder - UpdateCartStatus via backgroudJob
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....

In alle GeefOrderServiceInternal een BackGroundCartStatusUpdaterMock meegeven, anders wordt effectief een achtergrond process opgestart. Met indienMethodcalls().DoeNiets() processing verder laten g...

In alle GeefOrderServiceInternal een BackGroundCartStatusUpdaterMock meegeven, anders wordt effectief een achtergrond process opgestart.
Met indienMethodcalls().DoeNiets() processing verder laten gaan, indien er niets van wat in achtergrond uitgevoerd wordt moet gecontroleerd worden.

andere test method ook waar dit nog niet is.
geef een Mock mee en zie wat er aan fouten uit komt tijdens het uitvoeren van de testen.

[ICT-610] [rvJWI] VRB650 - VH4 MP: ConfirmOrder - UpdateCartStatus via backgroudJob
[ICT-610] [rvJWI] VRB650 - VH4 MP: ConfirmOrder - UpdateCartStatus via backgroudJob
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 '