Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
[ICT-1254] EC: VH4: MaakOrderVanOfferte: transactie scope kleiner maken (per Product-creatie en per Orderlijn-creatie i.p.v. gehele verwerking):

- Nothing but casing my dear Watson

  1. … 1 more file in changeset.
Indeed, stomme copy/paste http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/wink.gif

Indeed, stomme copy/paste

ieniemienie nitpicking: #dim ipv #Dim (staat ook nog in een andere klasse zo, wrschl copy-paste)

ieniemienie nitpicking: #dim ipv #Dim (staat ook nog in een andere klasse zo, wrschl copy-paste)

[ICT-1254] EC: VH4: MaakOrderVanOfferte: transactie scope kleiner maken (per Product-creatie en per...
[ICT-1254] EC: VH4: MaakOrderVanOfferte: transactie scope kleiner maken (per Product-creatie en per...
[ICT-1254] EC: VH4: MaakOrderVanOfferte: transactie scope kleiner maken (per Product-creatie en per Orderlijn-creatie i.p.v. gehele verwerking):

- MaakProductVanKenmerken gebeurt nu in een eigen OperatieScope.

- Accept op een WinkelkarLijn gebeurt nu in een eigen OperatieScope.

- Overkoepelende OperatieScope in MaakOrderVanOfferte is verwijderd => de 2 gevoelige sleutelacties hierboven hebben nu zelf elk hun OperatieScope

  1. … 4 more files in changeset.
[UST3920] EC: app: herbestel bij probleem: DI1 en nog niet verwerken
[UST3920] EC: app: herbestel bij probleem: DI1 en nog niet verwerken
[UST3920] EC: app: herbestel bij probleem: DI1 en nog niet verwerken

- Op winkelkar kunnen opvragen of er HerbestellingMetProbleem maatwerklijn aanwezig is

[DEF558] GeefOfferteSamenvattingenVoorKlant

- Probleem "andere winkelkarren" opvragen in de configurator. Hiervoor was reeds een try catch voorzien. Nu werd ook de data vervolledigd op cache01 en dit voor 5 winkelkarren. (fout opgetreden tot 19 febr 2017, daarna niet meer gezien.) 4 van de 5 winkelkarren zijn reeds besteld.

- Deze code was nodig om de data recht te zetten. -> stap 1 groeplijn toevoegen in ^DOM.EC.Winkelkar.WinkelkarD(xxxxx,"Inhoud",xxxxx), daarna groep corrigeren adhv ztryout klasse van JWI.

[KCA] uitgevoerd

voor Cache01

*** Queries to find maatwerk without groep !!!!

Select ID, lijst->aangemaaktop,lijst->klantid,Groep,Groep->Naam,* from "DOM_EC_WINKELKAR"."LIJN" where ID [ '180372';

Select ID, lijst->aangemaaktop,lijst->klantid,Groep,Groep->Naam,* from "DOM_EC_WINKELKAR"."LIJN" where groep->naam is null and len(groep)>0

Select klantid,id, BestelStatus from APPS_VKP_impl.Offerte where winkelkarid in (180372,181670,182355,182443,182854)

Select Naam from DOM_EC_Winkelkar.GroepLijn where naam = null

*** Zoeken in de globals

^DOM.EC.Winkelkar.WinkelkarD(180372

*** updaten in globals

Lijn "180372||823402"

Set ^DOM.EC.Winkelkar.WinkelkarD(180372,"Inhoud",823411) = $lb("~DOM.EC.Winkelkar.GroepLijn~","180372||823394",,,,,1,"")

Set ^DOM.EC.Winkelkar.WinkelkarD(180372,"Inhoud",823411,"GroepLijn") = $lb("Geen groepnaam",$lb($lb("180372||823402")))

Do ##class(zTryout.JWI.GroupnaamCorrectie).%New().VerwerkSubLijnByID("180372||823402")

181670||829937

Set ^DOM.EC.Winkelkar.WinkelkarD(181670,"Inhoud",830157) = $lb("~DOM.EC.Winkelkar.GroepLijn~","181670||823394",,,,,1,"")

Set ^DOM.EC.Winkelkar.WinkelkarD(181670,"Inhoud",830157,"GroepLijn") = $lb("Geen groepnaam",$lb($lb("181670||829937")))

Do ##class(zTryout.JWI.GroupnaamCorrectie).%New().VerwerkSubLijnByID("181670||829937")

182443||833947

Set ^DOM.EC.Winkelkar.WinkelkarD(182443,"Inhoud",833954) = $lb("~DOM.EC.Winkelkar.GroepLijn~","182443||833945",,,,,1,"")

Set ^DOM.EC.Winkelkar.WinkelkarD(182443,"Inhoud",833954,"GroepLijn") = $lb("Geen groepnaam",$lb($lb("182443||833947")))

Do ##class(zTryout.JWI.GroupnaamCorrectie).%New().VerwerkSubLijnByID("182443||833947")

Lijn 182854||835880

Groeplijn : 182854||835881

Set ^DOM.EC.Winkelkar.WinkelkarD(182854,"Inhoud",835881) = $lb("~DOM.EC.Winkelkar.GroepLijn~","182854||835878",,,,,1,"")

Set ^DOM.EC.Winkelkar.WinkelkarD(182854,"Inhoud",835881,"GroepLijn") = $lb("Geen groepnaam",$lb($lb("182854||835880")))

Do ##class(zTryout.JWI.GroupnaamCorrectie).%New().VerwerkSubLijnByID("182854||835880")

Lijn 182335||833395

Groeplijn 182335||833406

Set ^DOM.EC.Winkelkar.WinkelkarD(182335,"Inhoud",833406) = $lb("~DOM.EC.Winkelkar.GroepLijn~","182335||833370",,,,,1,"")

Set ^DOM.EC.Winkelkar.WinkelkarD(182335,"Inhoud",833406,"GroepLijn") = $lb("Geen groepnaam",$lb($lb("182335||833395")))

Do ##class(zTryout.JWI.GroupnaamCorrectie).%New().VerwerkSubLijnByID("182335||833395")

DONE - DONE - DONE - DONE - DONE - DONE - DONE - DONE - DONE - DONE

  1. … 2 more files in changeset.
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.

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.

[UST1937] Vhisie4: WebshopVersie op de Unishop Offertes

- ExterneId index unique

  • More
  • CR-3
  • started review
[UST1939] Vh4_SVC: MaakWinkelkar
[UST1939] Vh4_SVC: MaakWinkelkar
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.

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

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

Idem als LegeKenmerken.

Okay

Okay

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.

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.

Mock & Indien = Stub

Mock & Indien = Stub

Mock met $$$ElkeWaarde not done

Mock met $$$ElkeWaarde not done

Wat is de nood van deze klasse ?

Wat is de nood van deze klasse ?

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

  • More
  • CR-3
  • joined review
  • More
  • CR-3
  • started review
[UST1939] Vh4_SVC: MaakWinkelkar
[UST1939] Vh4_SVC: MaakWinkelkar