CacheAdminA_trunk2010/vhTest/Mock/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
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
Tuesday 17 Jan 2017
3:00 pm

Kurt Callebaut

11:43 am

Jo Claes

  • More
  • CR-23
  • resumed reviewing
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.

8:05 am

Kurt Callebaut

Monday 16 Jan 2017
2:26 pm

Kurt Callebaut

deze test is intussen gewijzigd.

deze test is intussen gewijzigd.

Friday 13 Jan 2017
5:27 pm

Jo Claes

Lijkt mij beter om de ExterneId mee te geven aan de Builder want anders hebben we een deel van de logica hier zitten en een ander deel in de builder

Lijkt mij beter om de ExterneId mee te geven aan de Builder want anders hebben we een deel van de logica hier zitten en een ander deel in de builder

8:39 am

Jo Claes

EdiConverter en LegacyConverter etc... samen nemen in klasse WSimpl.Vhisie4.Winkelkar.EconConfiguratieConverter.Convert(EconStream) As APPS.EC.Winkelkar.dto.MaatwerkLijn

EdiConverter en LegacyConverter etc... samen nemen in klasse WSimpl.Vhisie4.Winkelkar.EconConfiguratieConverter.Convert(EconStream) As APPS.EC.Winkelkar.dto.MaatwerkLijn

8:37 am

Jo Claes

Property EconConfiguratie As TECH.String (XMLNAME = "IngegevenKenmerken") [ Required ];

Property EconConfiguratie As TECH.String (XMLNAME = "IngegevenKenmerken") [ Required ];

8:36 am

Jo Claes

Afhankelijk van het antwoord van intracto maar ik vermoed dat deze er terug af mogen

Afhankelijk van het antwoord van intracto maar ik vermoed dat deze er terug af mogen

Thursday 12 Jan 2017
3:36 pm

Jo Claes

niet nodig

niet nodig

3:36 pm

Jo Claes

niet nodig

niet nodig

3:35 pm

Jo Claes

#dim lijn niet nodig

#dim lijn niet nodig

3:35 pm

Jo Claes

Request mag in-line gemaakt worden in de test zodat de test op zich duidelijk is

Request mag in-line gemaakt worden in de test zodat de test op zich duidelijk is

3:34 pm

Jo Claes

method niet nodig -> standaardlade kenmerken gebruiken

method niet nodig -> standaardlade kenmerken gebruiken

3:33 pm

Jo Claes

#dim Lijn niet nodig

#dim Lijn niet nodig

3:33 pm

Jo Claes

GeefWinkelkarIdViaExterneId mag eigenlijk weg en het zou gewoon een GeefViaID mogen zijn waarvan je een offerte zal terug krijgen ( omdat voorlopig de repository geen notie hebben van wat ze moeten...

GeefWinkelkarIdViaExterneId mag eigenlijk weg en het zou gewoon een GeefViaID mogen zijn waarvan je een offerte zal terug krijgen ( omdat voorlopig de repository geen notie hebben van wat ze moeten teruggeven , ( Strings, DataObjecten of Objecten ) op termijn zal het wel lukken als we de snelheid moeten aanpakken , maar voorlopig zou ik daar geen rekening met houden

3:31 pm

Jo Claes

Niet nodig om de bestaat via ExterneId op te roepen de GeefViaID is voldoende .. deze moeten dan wel de foutmelding encapsularen en zeggen dat er geen winkelkar bestaat met id.

Niet nodig om de bestaat via ExterneId op te roepen de GeefViaID is voldoende .. deze moeten dan wel de foutmelding encapsularen en zeggen dat er geen winkelkar bestaat met id.

3:17 pm

Jo Claes

set Kenmerken = ##class(Tools.Mock.ArgumentMatchers.DataTransferObjectMatcher).%New([StandaardKenmerken])

set Kenmerken = ##class(Tools.Mock.ArgumentMatchers.DataTransferObjectMatcher).%New([StandaardKenmerken])

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.

8:09 am

Kurt Callebaut

  • More
  • CR-23
  • started review
[UST2064] Vh4_SVC: BewaarMaatwerkLijn
[UST2064] Vh4_SVC: BewaarMaatwerkLijn
8:09 am

Kurt Callebaut

  • More
  • CR-22
  • started review
[UST2079] BewaarProductLijn moet ExterneId laten bewaren
[UST2079] BewaarProductLijn moet ExterneId laten bewaren
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.