Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Heb de wijziging uitgevoerd!

Heb de wijziging uitgevoerd!

Indien er nieuwe parameters in de constructor bij komen, deze achteraan in de parameterlijst toevoegen. Reden : Unittesten die aan deze constructor mocks meegeven moeten nu gerefactored worden...

Indien er nieuwe parameters in de constructor bij komen, deze achteraan in de parameterlijst toevoegen.
Reden : Unittesten die aan deze constructor mocks meegeven moeten nu gerefactored worden...

[rvJWI] OPO : nieuwe Verzendwijzen DOx, Transsmart activatie voor OPO en Dachser
[rvJWI] OPO : nieuwe Verzendwijzen DOx, Transsmart activatie voor OPO en Dachser
[UW754] VH4 Dupont: verzendingwijze op order niet correct
[UW754] VH4 Dupont: verzendingwijze op order niet correct
[UW497] VHIP504 Verwijderen tijdelijk order - kan soms order niet verwijderen. - foutmelding
[UW497] VHIP504 Verwijderen tijdelijk order - kan soms order niet verwijderen. - foutmelding
Nu komt die "1" precies uit de lucht gevallen. Men moet al moeite doen en verder zoeken, om te vinden dat die vanuit de method GeefTvwTijdelijkOrder() komt. Voorstel: ..GeefTeVerwijderenTijdelijkO...

Nu komt die "1" precies uit de lucht gevallen. Men moet al moeite doen en verder zoeken, om te vinden dat die vanuit de method GeefTvwTijdelijkOrder() komt.

Voorstel:
..GeefTeVerwijderenTijdelijkOrder(TvwOrderID)
Do TeVerwijderenOrderRepositoryMock.VerwachtMethodCall("VerwijderViaID",TvwOrderID).DanTralala

idem in de volgende methods

zie volgende comment, hieronder.

zie volgende comment, hieronder.

Deze review-opmerking mocht ik niet maken (van JoW) : klassenaam in t enkelvoud.

Deze review-opmerking mocht ik niet maken (van JoW) :
klassenaam in t enkelvoud.

Tja, ik moet het melden, hé ;-P $Inject() kan best de OrderAPI newen i.p.v. Context.Instance() ... maar als de discussie nog gesloten is, dan laat je het maar zo.

Tja, ik moet het melden, hé ;-P
$Inject() kan best de OrderAPI newen i.p.v. Context.Instance()
... maar als de discussie nog gesloten is, dan laat je het maar zo.

[UST3653] DOM.VKP.Klant.impl.KlantAdresRepository en AxLocationId:
[UST3653] DOM.VKP.Klant.impl.KlantAdresRepository en AxLocationId:
  • More
  • CR-394
  • finished reviewing
[UST2519] Vh4_SVC: AdresID gebruiken om juiste gekende leveringsadres te selecteren
[UST2519] Vh4_SVC: AdresID gebruiken om juiste gekende leveringsadres te selecteren