CacheAdminA_trunk2010/vhUnitTest/CHUI/VKP

OrderIngave

Checkout Tools

Line History

Line Count Graph
Line Count Graph

Stats

Commits this week: 0
Total Committers: 5
Last Commit: 16 Aug 17
Commits this week: 0
Total Lines of Code (LoC): 777
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
2:36 pm

Tommy Hebb

2:04 pm

Martijn Vierstraete

1:52 pm

Martijn Vierstraete

naamgeving: OrderApiStub

naamgeving: OrderApiStub

1:46 pm

Tommy Hebb

[UST3556] OV: Doorsturen picking: rembours: enkel volledig order toelaten:
[UST3556] OV: Doorsturen picking: rembours: enkel volledig order toelaten:
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 25 Apr 2018
1:37 pm

Jo Willems

Thursday 08 Mar 2018
1:47 pm

Jo Willems

nee. Het probleem was dat er eerst doorgestuurd was voor picking. niet dat er het order gesloten was. Maakt niet uit hoe je het noemt. verkeerde handeling van gebruiker houdt daar geen rekening mee...

nee. Het probleem was dat er eerst doorgestuurd was voor picking. niet dat er het order gesloten was. Maakt niet uit hoe je het noemt. verkeerde handeling van gebruiker houdt daar geen rekening mee...
En als ze een order met klant halux naar AX willen sturen zijn ze al helemaal niet goe bezig.

12:11 pm

Kurt Callebaut

12:10 pm

Kurt Callebaut

Ik begrijp de oplossing, maar sluiten via F11 of S (Ax order) kan nu ook tegenstrijdig met wat menu aangeeft, geen AX order zijn. De volgende melding zal dan zijn ik heb een Ax order gemaakt en het...

Ik begrijp de oplossing, maar sluiten via F11 of S (Ax order) kan nu ook tegenstrijdig met wat menu aangeeft, geen AX order zijn.
De volgende melding zal dan zijn ik heb een Ax order gemaakt en het staat niet in AX ?

Waarom legacy die niet meer gebruikt werd terug in gebruik nemen ?

Ik stel voor dan een derde menupunt aan te maken
ofwel bevestigdAXorder te hernoemen naar BevestigdOrder en die te gebruiken ?

11:50 am

Kurt Callebaut

Monday 05 Mar 2018
9:12 am

Jo Willems

[BUS667] Voorkomen dat een tijdelijk aangemaakt order in admin doorgestuurd kan worden naar magazijn....
[BUS667] Voorkomen dat een tijdelijk aangemaakt order in admin doorgestuurd kan worden naar magazijn....
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