CacheAdminA_trunk2010/vhUnitTest/CHUI/VKP/OrderIngave

TijdelijkOrderSluiterv01

Checkout Tools

Line History

Line Count Graph
Line Count Graph

Stats

Commits this week: 0
Total Committers: 2
Last Commit: 17 Aug 17
Commits this week: 0
Total Lines of Code (LoC): 282
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
Tuesday 05 Jun 2018
10:24 am

Jo Willems

Friday 01 Jun 2018
11:28 am

Kurt Callebaut

Thursday 31 May 2018
4:09 pm

Jo Willems

[VRB511]Tijdelijk order niet sluiten indien er geen productorderlijnen zijn.
[VRB511]Tijdelijk order niet sluiten indien er geen productorderlijnen zijn.
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.

Wednesday 31 Jan 2018
11:09 am

Jo Willems

Tuesday 23 Jan 2018
1:59 pm

Tom Vermeulen

  • More
  • CR-946
  • finished reviewing
1:59 pm

Tom Vermeulen

Do ipv do

Do ipv do

1:58 pm

Tom Vermeulen

Do ipv DO (ook hieronder)

Do ipv DO (ook hieronder)

1:58 pm

Tom Vermeulen

ProductApiMock ipv ProductAPI

ProductApiMock ipv ProductAPI

1:57 pm

Tom Vermeulen

Kunt ook de Logger injecteren en een test met een VerwachtMethodCall("Error") indien de geïnjecteerde OrderAPI.VerwijderOrder() $$$False teruggeeft Best de ORDNr die in de string gestoken wordt, me...

Kunt ook de Logger injecteren en een test met een VerwachtMethodCall("Error") indien de geïnjecteerde OrderAPI.VerwijderOrder() $$$False teruggeeft
Best de ORDNr die in de string gestoken wordt, met single quotes omringen

1:53 pm

Tom Vermeulen

returnt http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif

returnt

1:52 pm

Tom Vermeulen

$$$AssertEquals(,$$$True) kunt ge vervangen door $$$AssertTrue()

$$$AssertEquals(,$$$True) kunt ge vervangen door $$$AssertTrue()

1:51 pm

Tom Vermeulen

$$$DummyString

$$$DummyString

10:19 am

Jo Willems

[ICT349] VHIP504 Tijdelijk order sluiter order niet verwijderen indien fout in processing
[ICT349] VHIP504 Tijdelijk order sluiter order niet verwijderen indien fout in processing
Tuesday 28 Nov 2017
4:34 pm

Jo Willems

Thursday 16 Nov 2017
9:51 am

Kurt Callebaut

  • More
  • CR-738
  • finished reviewing
Wednesday 15 Nov 2017
3:49 pm

Jo Willems

[UST3138] VHIP504: ORDER2AX: Validatie bij het sluiten van een tijdelijk order
[UST3138] VHIP504: ORDER2AX: Validatie bij het sluiten van een tijdelijk order
Friday 13 Oct 2017
9:38 am

Jo Willems

Thursday 12 Oct 2017
8:24 am

Kurt Callebaut

Geen issue, maar ik zou denken dat we eens een ExistsId op TECH.Persistent moeten maken ook. http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif

Geen issue, maar ik zou denken dat we eens een ExistsId op TECH.Persistent moeten maken ook.

Thursday 05 Oct 2017
3:44 pm

Kurt Callebaut

  • More
  • CR-592
  • finished reviewing
2:24 pm

Jo Willems

[UST2935] VHIP504: ORDER2AX: order opkuisen
[UST2935] VHIP504: ORDER2AX: order opkuisen
Friday 25 Aug 2017
12:03 pm

Jo Willems

Monday 21 Aug 2017
4:38 pm

Kurt Callebaut

  • More
  • CR-448
  • finished reviewing
3:28 pm

Kurt Callebaut

Hiervoor zou ik een private method maken "IsTijdelijkOrder" die boolean weergeeft.

Hiervoor zou ik een private method maken "IsTijdelijkOrder" die boolean weergeeft.