Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Reverse merged revision(s) 56420 from trunk2010:

Reverse merged revision(s) 56414 from trunk2010:

[ICT] Aanpassingen als gevolg van een shshsh die ik heb uitgevoerd lokaal...

........

revert omdat mijn xml export blijkbaar nogal verschillend is van de rest van de ventjes hier

revert van de revert omdat mijn wijzigingen wel goed waren, maar de rest van de ventjes niet :)

    • -14
    • +7
    ./TeVerwijderenTijdelijkeOrders.cls.xml
    • -4
    • +2
    ./TeVerwijderenTijdelijkeOrderRepository.cls.xml
  1. … 63 more files in changeset.
Reverse merged revision(s) 56414 from trunk2010:

[ICT] Aanpassingen als gevolg van een shshsh die ik heb uitgevoerd lokaal...

........

revert omdat mijn xml export blijkbaar nogal verschillend is van de rest van de ventjes hier

    • -7
    • +14
    ./TeVerwijderenTijdelijkeOrders.cls.xml
    • -2
    • +4
    ./TeVerwijderenTijdelijkeOrderRepository.cls.xml
  1. … 63 more files in changeset.
[ICT] Aanpassingen als gevolg van een shshsh die ik heb uitgevoerd lokaal...
    • -14
    • +7
    ./TeVerwijderenTijdelijkeOrders.cls.xml
    • -4
    • +2
    ./TeVerwijderenTijdelijkeOrderRepository.cls.xml
  1. … 63 more files in changeset.
[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.

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

- refactor naar juiste package

    • -0
    • +72
    ./TeVerwijderenTijdelijkeOrders.cls.xml
    • -0
    • +57
    ./TeVerwijderenTijdelijkeOrderRepository.cls.xml
    • -0
    • +17
    ./TijdelijkOrderVerwijderaar.cls.xml
  1. … 18 more files in changeset.
[UST2483]Vh4_SVC: "elk product zo snel mogelijk" aka Deelleveringen

-dependencies mockable maken

    • -0
    • +19
    ./impl/AdresUtils.cls.xml
  1. … 4 more files in changeset.