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
[rvJWI] OPO : nieuwe Verzendwijzen DOx, Transsmart activatie voor OPO en Dachser

- AdresUtils uitgebreid voor OPO en unit test bijgevoegd

  1. … 1 more file in changeset.
[ICT-2210] MP Van Raemdonck rechtstreekse leveringen

- Verzendwijze voor VanRaemdonck aanpassen naar DI1

  1. … 1 more file in changeset.
[ICT-1636] EC: MP: Häfele Duitsland:Bepalen verzendwijze bij export tijdelijk order naar AX

- Het bepalen van de verzendwijze bij creatie tijdelijk order voor Haefele DE was nog niet voor zien

  1. … 1 more file in changeset.
[UW754] VH4 Dupont: verzendingwijze op order niet correct
[UW754] VH4 Dupont: verzendingwijze op order niet correct
[UW754] VH4 Dupont: verzendingwijze op order niet correct

- aangepast in de AdresUtils: naar DI1 indien geen locationid gekegend (adhoc adres dus)

  1. … 2 more files in changeset.
[UW497] VHIP504 Verwijderen tijdelijk order - kan soms order niet verwijderen. - foutmelding

- reviewopm

    • -5
    • +5
    ./TijdelijkOrderVerwijderaar/Test.cls.xml
[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.

Reverse merged revision(s) 51210 from trunk2010/vhUnitTest/APPS/VKP/OrderIngave:

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

- move van test opkuisen

........

misschien de juiste test opkuisen....

    • -0
    • +146
    ./TijdelijkOrderVerwijderaar/Test.cls.xml
  1. … 2 more files in changeset.
[UW497] VHIP504 Verwijderen tijdelijk order - kan soms order niet verwijderen. - foutmelding

- move van test opkuisen

    • -146
    • +0
    ./TijdelijkOrderVerwijderaar/Test.cls.xml
[UW497] VHIP504 Verwijderen tijdelijk order - kan soms order niet verwijderen. - foutmelding

- refactor naar juiste package

    • -0
    • +146
    ./TijdelijkOrderVerwijderaar/Test.cls.xml
  1. … 19 more files in changeset.
[UST3653] DOM.VKP.Klant.impl.KlantAdresRepository en AxLocationId:
[UST3653] DOM.VKP.Klant.impl.KlantAdresRepository en AxLocationId:
[UST3653] DOM.VKP.Klant.impl.KlantAdresRepository en AxLocationId:

- Method GeefLeverAdres overgebracht naar LeverAdresRepository en hernoemd naar GeefViaKlantIDEnLeverAdresID

  1. … 7 more files in changeset.
  • 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