MaakWinkelkar

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.

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.

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.

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.