CacheAdminA_trunk2010/vhTest/Fake/APPS/EC/Winkelkar/impl

WinkelkarService.cls.xml

Checkout Tools

Line History

Line Count Graph
Line Count Graph

Stats

Commits this week: 0
Total Committers: 3
Last Commit: 03 Feb 17
Commits this week: 0
Total Lines of Code (LoC): 0
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 28 Feb 2017
2:12 pm

Kurt Callebaut

Monday 13 Feb 2017
4:55 pm

Jo Claes

  • More
  • CR-46
  • finished reviewing
Friday 03 Feb 2017
11:31 am

JoC

[UST2175] : Vh4_SVC : Test services klaar hebben voor intracto

- ProductLijnen hebben nu ook notie van de groeplijn

Options
    • -179
    • +0
    ./WinkelkarService.cls.xml
  2. … 9 more files in changeset.
Monday 30 Jan 2017
2:50 pm

Jo Claes

Dummy terminologie is hier niet meer juist.. daar het over een specifieke maatwerk lijn gaat . met die specifieke Id.

Dummy terminologie is hier niet meer juist.. daar het over een specifieke maatwerk lijn gaat . met die specifieke Id.

2:45 pm

Jo Claes

Gebruikte klassen in property en definiëren in de constructor

Gebruikte klassen in property en definiëren in de constructor

2:44 pm

Jo Claes

Wat is er verkeerd aan de originele lijn ? En als je aanpast gelieve ze te dimmen. Btw wat betekent a ?

Wat is er verkeerd aan de originele lijn ? En als je aanpast gelieve ze te dimmen. Btw wat betekent a ?

Tuesday 24 Jan 2017
9:59 am

Kurt Callebaut

  • More
  • CR-46
  • started review
[UST2064] Vh4_SVC: BewaarMaatwerkLijn
[UST2064] Vh4_SVC: BewaarMaatwerkLijn
Monday 23 Jan 2017
8:18 am

KurtC

[UST2064] Vh4_SVC: BewaarMaatwerkLijn

- Refactoring winkelkar VoegToeMaatwerkLijn geeft object weer ipv ID.

Options
    • -1
    • +1
    ./WinkelkarService.cls.xml
  2. … 18 more files in changeset.
Tuesday 17 Jan 2017
10:48 am

Kurt Callebaut

10:48 am

Kurt Callebaut

Bedenking : GemaaktofferteEvent zou zo dicht mogelijk bij het dataobject moeten zitten zodat een event geraiset wordt bij het aanmaken van een offerte, aangemaakt vanuit eender welk stuk code. (bv ...

Bedenking : GemaaktofferteEvent zou zo dicht mogelijk bij het dataobject moeten zitten zodat een event geraiset wordt bij het aanmaken van een offerte, aangemaakt vanuit eender welk stuk code. (bv op repository) Indien er enkel geraiset wordt vanuit verkoopservice, dan heeft de event waarschijnlijk een andere bedoeling/betekenis.

Thursday 12 Jan 2017
12:03 pm

Jo Claes

RaiseEventOfferte is een implementatie detail en hoort niet op een interface klasse Naamgeving is niet ok om het gaat om een OfferteGemaaktEvent Beter om een EventRaiser klasse te gebruiken ( zoal...

RaiseEventOfferte is een implementatie detail en hoort niet op een interface klasse
Naamgeving is niet ok om het gaat om een OfferteGemaaktEvent

Beter om een EventRaiser klasse te gebruiken ( zoals daar is : BL.Derde.Agenda.event.EventRaiser )

Verder is het niet nodig om deze klasse expliciet te gaan uitmocken daar in het testing-framework het gehele pubsub gedeelte uitgemockt is.

Monday 09 Jan 2017
11:00 am

Kurt Callebaut

  • More
  • CR-3
  • started review
[UST1939] Vh4_SVC: MaakWinkelkar
[UST1939] Vh4_SVC: MaakWinkelkar
Saturday 07 Jan 2017
12:31 pm

Kurt Callebaut

Melding is veraglemeend naar "ExterneId x bestaat reeds". Daar er op data niveau geen relatie is tussen offerte en winkelkar, moet er een controle voor beide offerte en winkelkar gebeuren. Reposito...

Melding is veraglemeend naar "ExterneId x bestaat reeds".
Daar er op data niveau geen relatie is tussen offerte en winkelkar, moet er een controle voor beide offerte en winkelkar gebeuren. Repository geeft enkel bestaat viaexterneId en het is niet de bedoeling de repository een foutmelding te laten genereren. De meldingen zitten dus op niveau van de services. 1 controle enkel op ofwel offerte of winkelkar lijkt met gevaarlijk omdat het op data niveau niet opgevangen wordt. We willen ook geen dependency creeëren tussen offerte service en winkelkar service. Vandaar elk hun controle; dus de plaats waar de controles samenkomen is dan op verkoopservice niveau. Later misschien te refactoren.

Friday 06 Jan 2017
3:27 pm

Jo Claes

  • More
  • CR-3
  • resumed reviewing
3:11 pm

Kurt Callebaut

VerkoopService gebruikt geen repositories op dit moment. OfferteBewaarder niet vervangen door repository omdat dit op veel plaatsen zo gebruikt wordt, dan is er meer refactorinbg nodig dan enkel de...

VerkoopService gebruikt geen repositories op dit moment.
OfferteBewaarder niet vervangen door repository omdat dit op veel plaatsen zo gebruikt wordt, dan is er meer refactorinbg nodig dan enkel de verkoopservice.

2:06 pm

Kurt Callebaut

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

Idem als LegeKenmerken.

2:05 pm

Kurt Callebaut

Okay

Okay

1:59 pm

Kurt Callebaut

Winkelkar was hier niet van belang aangezien de wijzigingen en testen enkel over externid gaan. Testen zijn zoals besproken aangepast naar 1 test waarin alle controles gebeuren op de gebruikte meth...

Winkelkar was hier niet van belang aangezien de wijzigingen en testen enkel over externid gaan. Testen zijn zoals besproken aangepast naar 1 test waarin alle controles gebeuren op de gebruikte methods binnen de MaakOfferte.

Thursday 05 Jan 2017
1:16 pm

Jo Claes

Het feit dat er op de DOM.EC.. winkelkar en APPS.VKP.Offerte een ExterneId staat is een intern gegeven dus de controle op winkelkar en offerte horen samen te zitten. Het zou in principe niet kunnen...

Het feit dat er op de DOM.EC.. winkelkar en APPS.VKP.Offerte een ExterneId staat is een intern gegeven dus de controle op winkelkar en offerte horen samen te zitten.
Het zou in principe niet kunnen dat de ExterneId reeds op de winkelkar staat en niet op de offerte dus is eigenlijk een controle op de offerte voldoende.
Dus de foutmelding die we naar de buitenwereld sturen is eerder één van alla "ID reeds gebruik" want intracto heeft geen weet van onze offertes en Winkelkarren.

1:12 pm

Jo Claes

Mock & Indien = Stub

Mock & Indien = Stub

1:11 pm

Jo Claes

Mock met $$$ElkeWaarde not done

Mock met $$$ElkeWaarde not done

1:10 pm

Jo Claes

Wat is de nood van deze klasse ?

Wat is de nood van deze klasse ?

1:09 pm

Jo Claes

OfferteBewaarder mag vervangen worden door OfferteRepository - OfferteBewaarder mag eventueel integraal verwijderd worden als het niet teveel inpakt heeft

OfferteBewaarder mag vervangen worden door OfferteRepository - OfferteBewaarder mag eventueel integraal verwijderd worden als het niet teveel inpakt heeft

11:07 am

Jo Claes

  • More
  • CR-3
  • joined review
9:38 am

Jo Claes

  • More
  • CR-3
  • started review
[UST1939] Vh4_SVC: MaakWinkelkar
[UST1939] Vh4_SVC: MaakWinkelkar
Friday 30 Dec 2016
10:49 am

KurtC

[UST1939] Vh4_SVC: MaakWinkelkar

- controleerwinkelkarviaexterneid. op winkelkar gezet omdat daar repository gebruikt wordt.

Bedenking : externeid ook gezet op winkelkar (en neit op externewinkelkar) omdat de winkelkar bewaard wordt in de maakwinkelkar. Als we externe id zetten op externe winkelkar, dan bestaat de mogelijkheid dat de winkelkar gemaakt en bewaard wordt end at de externeid door bv een fout, niet gezet wordt. Nu garanderen we externeid gezet bij eht bewaren van de winkelkar.

Options
    • -0
    • +5
    ./WinkelkarService.cls.xml
  2. … 4 more files in changeset.
Wednesday 26 Oct 2016
3:38 pm

Tom Vermeulen

[UST1862R] DKG beslag: pakketreferentie in EDI omzetten in de nodige productlijnen

-VoegToeMaatwerk: het aantal van string naar integer (op offerteservice, winkelkarservice en maatwerklijnbuilder (dit fakeske had de bus gemist)

Options
    • -7
    • +4
    ./WinkelkarService.cls.xml
1:59 pm

Tom Vermeulen

[UST1862R] DKG beslag: pakketreferentie in EDI omzetten in de nodige productlijnen

-APPS.VKP.impl.OfferteService : VoegToeProduct -> method lijkt mij niet echt unittestbaar door de %OpenId .. beter hiervoor de repository gebruiken - DOM.EC.Winkelkar.LijnRepository.GeefViaID of DOM.EC.Winkelkar.WinkelkarRepository GeefLijnViaID - Eerste mogelijkheid lijkt mij toch net iets beter - kwestie van de ene repo is voor de winkelkarren en dan de andere specifiek voor de lijnen

- fake en mock ProductLijnRepository ook ineens weggehaald

- moet nog eens denken over een schone testbenaming in OfferteService/VoegToeProduct/Test

Options
    • -0
    • +177
    ./WinkelkarService.cls.xml
  2. … 10 more files in changeset.