CacheAdminA_trunk2010/vhUnitTest/APPS/EC/Winkelkar/impl/WinkelkarService

ControleerWinkelkarViaExterneId

Checkout Tools

Line History

Line Count Graph
Line Count Graph

Stats

Commits this week: 0
Total Committers: 1
Last Commit: 30 Dec 16
Commits this week: 0
Total Lines of Code (LoC): 20
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
Wednesday 15 Nov 2017
10:16 am

Kurt Callebaut

Friday 10 Nov 2017
11:51 am

Kurt Callebaut

Deze code was dus niet correct...http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif is intussen gewijzigd. Later misschien nog te refactoren.

Deze code was dus niet correct... is intussen gewijzigd. Later misschien nog te refactoren.

Wednesday 11 Oct 2017
8:01 am

Kurt Callebaut

ik ga ondanks de beslissing om hier niet aan verder te werken, toch een kaartje aanmaken.. ge weet maar nooit.

ik ga ondanks de beslissing om hier niet aan verder te werken, toch een kaartje aanmaken.. ge weet maar nooit.

Monday 09 Oct 2017
1:30 pm

Jo Claes

  • More
  • CR-580
  • finished reviewing
1:29 pm

Jo Claes

LogActie lijkt mij toch niet echt correct te zijn, het hoort allemaal in de detail te staan. Actie lijkt mij eerder ControleKenmerken - Kwestie dat de logging beetje ordentelijk is

LogActie lijkt mij toch niet echt correct te zijn, het hoort allemaal in de detail te staan.

Actie lijkt mij eerder ControleKenmerken - Kwestie dat de logging beetje ordentelijk is

1:18 pm

Jo Claes

DOM logica

DOM logica

1:17 pm

Jo Claes

DOM logica

DOM logica

1:17 pm

Jo Claes

APPS logica

APPS logica

1:17 pm

Jo Claes

APPS logica

APPS logica

1:16 pm

Jo Claes

APPS logica

APPS logica

1:16 pm

Jo Claes

APPS logica

APPS logica

1:16 pm

Jo Claes

DOM Logica

DOM Logica

1:16 pm

Jo Claes

DOM Logica

DOM Logica

1:16 pm

Jo Claes

DOM Logica

DOM Logica

Wednesday 04 Oct 2017
4:00 pm

Kurt Callebaut

[UST2867] Vh4_SVC: Snelheid sync winkelkar naar Caché controleren en tot aanvaardbaar punt brengen
[UST2867] Vh4_SVC: Snelheid sync winkelkar naar Caché controleren en tot aanvaardbaar punt brengen
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