CacheAdminA_trunk2010/vhUnitTest/APPS/EC/UNI/SFS/impl/OrderServiceInternal

ConfirmOrder

Checkout Tools

Line History

Line Count Graph
Line Count Graph

Stats

Commits this week: 0
Total Committers: 4
Last Commit: 06 Jul 16
Commits this week: 0
Total Lines of Code (LoC): 24
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 25 Jan 2019
5:05 pm

Wim Vermeulen

4:39 pm

Jo Willems

Thursday 24 Jan 2019
11:16 am

Wim Vermeulen

[UW700] ConfirmOrder: (UT) ECommerceAPI niet gebruiken bij $$$Inject
[UW700] ConfirmOrder: (UT) ECommerceAPI niet gebruiken bij $$$Inject
9:06 am

Wim Vermeulen

[UW700] ConfirmOrder: (UT) ECommerceAPI niet gebruiken bij $$$Inject

- SFS (oude) OrderServiceInternal : Constructor aangepast + UT

Options
    • -1
    • +1
    ./Test2.cls.xml
    • -1
    • +1
    ./Test1.cls.xml
    • -3
    • +1
    ./TestBase.cls.xml
  4. … 1 more file in changeset.
Tuesday 12 Sep 2017
9:00 am

Kurt Callebaut

8:59 am

Kurt Callebaut

excepties van gemaakt.

excepties van gemaakt.

Monday 11 Sep 2017
5:22 pm

Jo Claes

LeverDatum als %String terwijl de interface zegt dat het een %Date moet zijn ??? Kijk eens naar de andere test-cases in deze klasse ook want hier is er toch het een en ander niet juist. Ofwel krij...

LeverDatum als %String terwijl de interface zegt dat het een %Date moet zijn ???

Kijk eens naar de andere test-cases in deze klasse ook want hier is er toch het een en ander niet juist.
Ofwel krijgt de init een %String binnen en gaat deze de datum omvormen ofwel gaat iedere testcase de init correct aanroepen.

5:08 pm

Jo Claes

Een VanHoeckeOfferte is toch altijd een vhisie4 versie. Ofwel corrigeren ofwel is het dode code die opgekuist mag worden.

Een VanHoeckeOfferte is toch altijd een vhisie4 versie.

Ofwel corrigeren ofwel is het dode code die opgekuist mag worden.

5:05 pm

Jo Claes

Het zijn ..Stubs ipv ..Mocks - kwestie van het juist te hebben

Het zijn ..Stubs ipv ..Mocks - kwestie van het juist te hebben

4:05 pm

Kurt Callebaut

Dat durf ik niet met zekerheid te zeggen. De andere exceptieklasse voor de offertefoutmelding die overerft van invalidinput exceptie, past ook de exceptiecode niet aan.Mogelijks rekent andere softw...

Dat durf ik niet met zekerheid te zeggen. De andere exceptieklasse voor de offertefoutmelding die overerft van invalidinput exceptie, past ook de exceptiecode niet aan.Mogelijks rekent andere software op deze exceptiecode. ???

4:01 pm

Kurt Callebaut

excepties gebruikt ipv tekst of codes.

excepties gebruikt ipv tekst of codes.

11:15 am

Jo Claes

  • More
  • CR-511
  • finished reviewing
11:15 am

Jo Claes

Lijkt mij kweet niet hoe gevaarlijk om hier enkel op tekst te vergelijken. Hoe zou beter zijn dat de foutmeldingen een lijst van meldingen worden met ipv een string een melding object met daarin ee...

Lijkt mij kweet niet hoe gevaarlijk om hier enkel op tekst te vergelijken. Hoe zou beter zijn dat de foutmeldingen een lijst van meldingen worden met ipv een string een melding object met daarin een code en een string en kan er per code makkelijker gemapped worden welke exceptie te gooien.
Als code kunnen we misschien de exceptie klasse naam gebruiken zo kunnen we iets generiek maken om de exceptie te throwen

10:31 am

Jo Claes

We er voor zorgen dat je een andere foutcode naar buiten gooit

We er voor zorgen dat je een andere foutcode naar buiten gooit

10:31 am

Jo Claes

TECH.Interface ipv TECH.Mockable

TECH.Interface ipv TECH.Mockable

Wednesday 06 Sep 2017
3:51 pm

Kurt Callebaut

[UST2758] Vh4: Geldigheid prijzen: beslag-winkelkarren niet controleren en BestelWinkelkar
[UST2758] Vh4: Geldigheid prijzen: beslag-winkelkarren niet controleren en BestelWinkelkar
10:59 am

KurtC

[UST2758] Vh4: Geldigheid prijzen: beslag-winkelkarren niet controleren en BestelWinkelkar

- fixen falende testen.

Options
    • -1
    • +3
    ./Test2.cls.xml
Tuesday 05 Sep 2017
9:58 am

Kurt Callebaut

9:57 am

Kurt Callebaut

Zal Als apart kaartje moeten behandelen want de geldigheidsprijzen op maatwerklijnen worden niet ingevuld en is iets meer werk als enkel refactoring.

Zal Als apart kaartje moeten behandelen want de geldigheidsprijzen op maatwerklijnen worden niet ingevuld en is iets meer werk als enkel refactoring.

Monday 04 Sep 2017
1:51 pm

Jo Claes

  • More
  • CR-506
  • finished reviewing
1:50 pm

Jo Claes

Beetje onletten met benamingen -> Offerte en Winkelkar niet door elkaar halen Ook al wordt Vhisie3 vorm ( Offertes ) gebruikt , het blijven winkelkarren in Vhisie4.

Beetje onletten met benamingen -> Offerte en Winkelkar niet door elkaar halen

Ook al wordt Vhisie3 vorm ( Offertes ) gebruikt , het blijven winkelkarren in Vhisie4.

12:02 pm

Jo Claes

IsVerschillend lijkt mij een betere naam.

IsVerschillend lijkt mij een betere naam.

11:59 am

Jo Claes

Waarom wordt dit terug van de Offerte gehaald ?? Is het niet logischer dat de MaatwerkLijn correct aangepast wordt ???

Waarom wordt dit terug van de Offerte gehaald ?? Is het niet logischer dat de MaatwerkLijn correct aangepast wordt ???

11:23 am

Kurt Callebaut

[UST2682] vh4 : vervallen offertes
[UST2682] vh4 : vervallen offertes
Thursday 24 Aug 2017
4:16 pm

KurtC

[UST2682] - vervallen offertes.

Na bespreking met PRA - Niet op deze manier geldigheid utischakelen. er moet een odnerscheid komen prod en maatwedrklijnen, maar eerst nog verder te bepalen. mogelijks gaat bv intracto ook herbereken uitvoeren als er enkel productlijen zijn, etc... ! revert !

Options
    • -2
    • +1
    ./Test2.cls.xml
  2. … 4 more files in changeset.
8:36 am

KurtC

[UST2682] - vervallen offertes.

Voor Vhisie4 prijscontrole uitschakelen bij validatie van de offerte geldigheid.

Options
    • -1
    • +2
    ./Test2.cls.xml
  2. … 2 more files in changeset.
Tuesday 04 Jul 2017
6:28 pm

PieterVR

[UW386] Cosmetica...
Options
    • -0
    • +2
    ./TestBase.cls.xml
  2. … 2 more files in changeset.
Wednesday 24 May 2017
4:10 pm

Tom Vermeulen

Tuesday 23 May 2017
1:39 pm

Jo Claes

  • More
  • CR-243
  • finished reviewing
Monday 22 May 2017
4:00 pm

Jo Claes

Eigenlijk geen goed idee om de GeefWebshopVerwerker op de personality te plaatsen. Beter ergens op een klasse te zetten die dichter bij de webshop staat en daar de verwerker te vragen voor een bep...

Eigenlijk geen goed idee om de GeefWebshopVerwerker op de personality te plaatsen.

Beter ergens op een klasse te zetten die dichter bij de webshop staat en daar de verwerker te vragen voor een bepaalde personality

Het domein dient niet te weten dat er ergens een webshop bestaat.