Activity CacheAdminA_trunk2010/APPS/VKP/OrderIngave

10:00 am

Joannes Lemmens

More
Re: CR-2439
replied to Jo Willems

Heb de wijziging uitgevoerd!

3:58 pm

Jo Willems

More
Re: CR-2439
commented on review

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...

4:15 pm

Joannes Lemmens

More
CR-2439
started review

[rvJWI] OPO : nieuwe Verzendwijzen DOx, Transsmart activatie voor OPO en Dachser

3:19 pm

Tom Vermeulen

More
CR-1688
started review

[UW754] VH4 Dupont: verzendingwijze op order niet correct

11:21 am

Jo Willems

More
CR-1175
started review

[UW497] VHIP504 Verwijderen tijdelijk order - kan soms order niet verwijderen. - foutmelding

5:23 pm

Wim Vermeulen

More
Re: CR-1175
commented on review

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

5:06 pm

Wim Vermeulen

More
Re: CR-1175
commented on review

zie volgende comment, hieronder.

4:51 pm

Wim Vermeulen

More
Re: CR-1175
commented on review

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.

4:41 pm

Wim Vermeulen

More
Re: CR-1175
commented on review

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.

11:34 am