CacheAdminA_trunk2010/vhTest/Fake/APPS/VKP/OrderIngave

TeVerwijderenTijdelijkeOrderRepository.cls.xml

Checkout Tools

Line History

Line Count Graph
Line Count Graph

Stats

Commits this week: 0
Total Committers: 2
Last Commit: 26 Apr 19
Commits this week: 0
Total Lines of Code (LoC): 55
Net change in LoC this week: 0

Commit Activity

Commits By Day In Week
52 week commits volume
Commits By Day In Week
Commits by day
Commits By Day In Week
Commits by hour
Commits By Hour In Day
Commit calendar
 

Most active committers (90 days)

loading
loading
  • last updated a few seconds ago
All Commits Reviews
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Friday 26 Apr 2019
2:09 pm

Tom Vermeulen

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 :)

Options
    • -4
    • +2
    ./TeVerwijderenTijdelijkeOrderRepository.cls.xml
  2. … 65 more files in changeset.
1:53 pm

Tom Vermeulen

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

Options
    • -2
    • +4
    ./TeVerwijderenTijdelijkeOrderRepository.cls.xml
  2. … 65 more files in changeset.
11:47 am

Tom Vermeulen

[ICT] Aanpassingen als gevolg van een shshsh die ik heb uitgevoerd lokaal...
Options
    • -4
    • +2
    ./TeVerwijderenTijdelijkeOrderRepository.cls.xml
  2. … 65 more files in changeset.
Wednesday 09 May 2018
11:21 am

Jo Willems

11:07 am

Jo Willems

[UW497] VHIP504 Verwijderen tijdelijk order - kan soms order niet verwijderen. - foutmelding
[UW497] VHIP504 Verwijderen tijdelijk order - kan soms order niet verwijderen. - foutmelding
Monday 07 May 2018
5:23 pm

Wim Vermeulen

5:09 pm

Wim Vermeulen

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

zie volgende comment, hieronder.

zie volgende comment, hieronder.

4:51 pm

Wim Vermeulen

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

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.

Friday 04 May 2018
3:55 pm

Jo Willems

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

- refactor naar juiste package

Options
    • -0
    • +57
    ./TeVerwijderenTijdelijkeOrderRepository.cls.xml
  2. … 20 more files in changeset.