Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
[UST4188] DPT1: Vh4: MP: Du-Pont: rechtstr. transp.: leveradres overnemen op doorbestelde winkelkar/offerte...
[UST4188] DPT1: Vh4: MP: Du-Pont: rechtstr. transp.: leveradres overnemen op doorbestelde winkelkar/offerte...
[UST4188] DPT1: Vh4: MP: Du-Pont: rechtstr. transp.: leveradres overnemen op doorbestelde winkelkar/offerte

- LeveradresInvuller gebruiken in BestelWinkelkar

  1. … 1 more file in changeset.
Ja, in de allereerste test... daarin: VerwachteHoofding => VerkoopServiceMock.Verifieer()

Ja, in de allereerste test... daarin: VerwachteHoofding => VerkoopServiceMock.Verifieer()

Helemaal gelijk en done! http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif

Helemaal gelijk en done!

Meestal breng ik zo een blok naar een method eens voor besteller en eens voor bevestiger email. die een email adres teruggeeft zodat de code om email te bepalen, gescheiden is van de rest. Kwestie ...

Meestal breng ik zo een blok naar een method eens voor besteller en eens voor bevestiger email. die een email adres teruggeeft zodat de code om email te bepalen, gescheiden is van de rest. Kwestie van lange methods te vermijden. Denk ook dat je in de nieuwe methods een variabele bovenaan kan zetten Set email = "" (niks) en de voorwaarden maakt voor wanneer het wel ingevuld wordt, dan hoeven we de Emailadres.. = "" niet te herhalen.

en als ge nog een stapje leesbaarder wilt maken, maken we lokale ...StartsWith(Email, "Multi") die een boolean teruggeeft. Dan blijft de "if" statement kort.

Zou bv ook kunnen voor "wensdatum" in deze klasse..

Hebben we een test nodig om te bevestigen dat hikj wel overgenomen wordt in bepaalde situaties ? of zit die ergens in een standaard test ?

Hebben we een test nodig om te bevestigen dat hikj wel overgenomen wordt in bepaalde situaties ? of zit die ergens in een standaard test ?

[VRB562] VHIP504: ORDER2AX: Opvolging inhoud 2 email-velden die ITR doorstuurt:
[VRB562] VHIP504: ORDER2AX: Opvolging inhoud 2 email-velden die ITR doorstuurt:
[VRB562] VHIP504: ORDER2AX: Opvolging inhoud 2 email-velden die ITR doorstuurt:

- Refactor wegens extra variant => EmailAdresBevestiging moet ook leeg blijven indien het begint met multi

- Paar testjes

  1. … 1 more file in changeset.
[VRB538] [VRB538] VHIP504: ORDER2AX: SalesConfirmEmail invullen (en impersonificatie+... negeren):

- Na review => ProjectSettings via deploy overal aangezet behalve CACHE01 en CACHEACCEPT2010 en CACHETEST2010 => waar testen lopen mag hij aan staan => daarom ook zie volgend punt

- ProjectSettings uit testen gehaald

- Aanpassing testfile wegens verwijderen ProjectSettings uit testen

  1. … 3 more files in changeset.
Lijkt me niet verkeerd, maar heb ik nog niet gezien in testen bij ons. meestal maken we testen waarbij de vlag geforceerd wordt aan begin van de test. met als resultaat dat ge in dit geval 2 testen...

Lijkt me niet verkeerd, maar heb ik nog niet gezien in testen bij ons. meestal maken we testen waarbij de vlag geforceerd wordt aan begin van de test. met als resultaat dat ge in dit geval 2 testen kunt hebben. één met vlag aan en één met vlag uit. ik zou es rondhoren bij collega's.

Voorlopig ProjectSettings op gezet, wegens te grote onbetrouwbaarheid van wat ITR doorstuurt. Om aan te zetten: Do ##class(TECH.Config.ConfigMgr).Instance().ZetString("DOM.common.ProjectSettingsAPI...

Voorlopig ProjectSettings op gezet, wegens te grote onbetrouwbaarheid van wat ITR doorstuurt.
Om aan te zetten:
Do ##class(TECH.Config.ConfigMgr).Instance().ZetString("DOM.common.ProjectSettingsAPI_IsActiefVhisieEmailadressenNaarAx", 1)
Ondertussen wordt input gelogd in vhLib.Logging, totdat we zeker genoeg zijn dat de input correct is blijft de ProjectSettings op False staan.

[VRB538] VHIP504: ORDER2AX: SalesConfirmEmail invullen (en impersonificatie+... negeren):

- ProjectSettings toegevoegd => resultaat van VHISIE4 is te onbetrouwbaar momenteel => logging vhLib

  1. … 1 more file in changeset.
Is inclusief kaartje VRB534 !!!

Is inclusief kaartje VRB534 !!!

[VRB538] VHIP504: ORDER2AX: SalesConfirmEmail invullen (en impersonificatie+... negeren):
[VRB538] VHIP504: ORDER2AX: SalesConfirmEmail invullen (en impersonificatie+... negeren):
[VRB538] VHIP504: ORDER2AX: SalesConfirmEmail invullen (en impersonificatie+... negeren):

- EmailAdresBevestiging en EmailAdresBesteller => check of geldig emailadres => zoniet => leeg

  1. … 1 more file in changeset.
[VRB538] VHIP504: ORDER2AX: SalesConfirmEmail invullen (en impersonificatie+... negeren):

- ZetEmailAdresBevestiging => van VHISIE4 BestelWinkelkarRequest naar DTO

  1. … 1 more file in changeset.
[VRB538] VHIP504: ORDER2AX: SalesConfirmEmail invullen (en impersonificatie+... negeren):

- 2 nieuwe properties op Header => EmailAdresBesteller (ter vervanging EmailAdres) en EmailAdresBevestiging

- WSDL

  1. … 6 more files in changeset.
Het is "EmailAdresBesteller" geworden, na overleg met PRA en WV.

Het is "EmailAdresBesteller" geworden, na overleg met PRA en WV.

[VRB524] Vh4: BestelWinkelkar: emailadres gebruiker ontvangen van ITR en opslaan (in bestaande piece):

- Aanpassing naamgeving => EmailAdres => EmailAdresBesteller

  1. … 11 more files in changeset.
Als het echt gaat over het email adres van de gebruiker, zoals in de beschrijving van het kaartje, dan zou ik opteren om GebruikerEmail als naam te gebruiken. We zien gebruikerIp, GebruikerId terug...

Als het echt gaat over het email adres van de gebruiker, zoals in de beschrijving van het kaartje, dan zou ik opteren om GebruikerEmail als naam te gebruiken. We zien gebruikerIp, GebruikerId terugkomen in de header. Anderzijds zien we bij offerte variabelen die spreken over gegevens van de klant, zoals taal enz. en ook hier zien we Email adres terugkomen, wat bij een eerste indruk dan doet denken aan een email adres van de klant. (Beetje een ambetant issue om nog aan te passen, denk ik dan.)

[VRB524] Vh4: BestelWinkelkar: emailadres gebruiker ontvangen van ITR en opslaan:
[VRB524] Vh4: BestelWinkelkar: emailadres gebruiker ontvangen van ITR en opslaan:
[VRB524] Vh4: BestelWinkelkar: emailadres gebruiker ontvangen van ITR en opslaan:

- Property EmailAdres bijgezet op Hoofding-DTOke en RequestHeader en Offerte

- GeefEmailAdres en ZetEmailAdres bijgezet op interfaces van Offerte en Order en geïmplementeerd in derived

- MaakHoofding op BestelWinkelkar haalt EmailAdres op van de Header op de BestelWinkelkarRequest en zet het op het Hoofding-DTOke

- ZetHoofding op OfferteService haalt EmailAdres van Hoofding-DTOke en zet het op de Offerte

- MaakOrderVanOfferte op OrderIngaveService haalt EmailAdres op van Offerte en zet het op Order

  1. … 12 more files in changeset.