Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
[UW700] ConfirmOrder: (UT) ECommerceAPI niet gebruiken bij $$$Inject
[UW700] ConfirmOrder: (UT) ECommerceAPI niet gebruiken bij $$$Inject
excepties van gemaakt.

excepties van gemaakt.

LeverDatum als %String terwijl de interface zegt dat het een %Date moet zijn ??? Kijk eens naar de andere test-cases in deze klasse ook want hier is er toch het een en ander niet juist. Ofwel krij...

LeverDatum als %String terwijl de interface zegt dat het een %Date moet zijn ???

Kijk eens naar de andere test-cases in deze klasse ook want hier is er toch het een en ander niet juist.
Ofwel krijgt de init een %String binnen en gaat deze de datum omvormen ofwel gaat iedere testcase de init correct aanroepen.

Een VanHoeckeOfferte is toch altijd een vhisie4 versie. Ofwel corrigeren ofwel is het dode code die opgekuist mag worden.

Een VanHoeckeOfferte is toch altijd een vhisie4 versie.

Ofwel corrigeren ofwel is het dode code die opgekuist mag worden.

Het zijn ..Stubs ipv ..Mocks - kwestie van het juist te hebben

Het zijn ..Stubs ipv ..Mocks - kwestie van het juist te hebben

Dat durf ik niet met zekerheid te zeggen. De andere exceptieklasse voor de offertefoutmelding die overerft van invalidinput exceptie, past ook de exceptiecode niet aan.Mogelijks rekent andere softw...

Dat durf ik niet met zekerheid te zeggen. De andere exceptieklasse voor de offertefoutmelding die overerft van invalidinput exceptie, past ook de exceptiecode niet aan.Mogelijks rekent andere software op deze exceptiecode. ???

excepties gebruikt ipv tekst of codes.

excepties gebruikt ipv tekst of codes.

  • More
  • CR-511
  • finished reviewing
Lijkt mij kweet niet hoe gevaarlijk om hier enkel op tekst te vergelijken. Hoe zou beter zijn dat de foutmeldingen een lijst van meldingen worden met ipv een string een melding object met daarin ee...

Lijkt mij kweet niet hoe gevaarlijk om hier enkel op tekst te vergelijken. Hoe zou beter zijn dat de foutmeldingen een lijst van meldingen worden met ipv een string een melding object met daarin een code en een string en kan er per code makkelijker gemapped worden welke exceptie te gooien.
Als code kunnen we misschien de exceptie klasse naam gebruiken zo kunnen we iets generiek maken om de exceptie te throwen

We er voor zorgen dat je een andere foutcode naar buiten gooit

We er voor zorgen dat je een andere foutcode naar buiten gooit

TECH.Interface ipv TECH.Mockable

TECH.Interface ipv TECH.Mockable

[UST2758] Vh4: Geldigheid prijzen: beslag-winkelkarren niet controleren en BestelWinkelkar
[UST2758] Vh4: Geldigheid prijzen: beslag-winkelkarren niet controleren en BestelWinkelkar
Zal Als apart kaartje moeten behandelen want de geldigheidsprijzen op maatwerklijnen worden niet ingevuld en is iets meer werk als enkel refactoring.

Zal Als apart kaartje moeten behandelen want de geldigheidsprijzen op maatwerklijnen worden niet ingevuld en is iets meer werk als enkel refactoring.

  • More
  • CR-506
  • finished reviewing
Beetje onletten met benamingen -> Offerte en Winkelkar niet door elkaar halen Ook al wordt Vhisie3 vorm ( Offertes ) gebruikt , het blijven winkelkarren in Vhisie4.

Beetje onletten met benamingen -> Offerte en Winkelkar niet door elkaar halen

Ook al wordt Vhisie3 vorm ( Offertes ) gebruikt , het blijven winkelkarren in Vhisie4.

IsVerschillend lijkt mij een betere naam.

IsVerschillend lijkt mij een betere naam.

Waarom wordt dit terug van de Offerte gehaald ?? Is het niet logischer dat de MaatwerkLijn correct aangepast wordt ???

Waarom wordt dit terug van de Offerte gehaald ?? Is het niet logischer dat de MaatwerkLijn correct aangepast wordt ???

[UST2682] vh4 : vervallen offertes
[UST2682] vh4 : vervallen offertes
  • More
  • CR-243
  • finished reviewing
Eigenlijk geen goed idee om de GeefWebshopVerwerker op de personality te plaatsen. Beter ergens op een klasse te zetten die dichter bij de webshop staat en daar de verwerker te vragen voor een bep...

Eigenlijk geen goed idee om de GeefWebshopVerwerker op de personality te plaatsen.

Beter ergens op een klasse te zetten die dichter bij de webshop staat en daar de verwerker te vragen voor een bepaalde personality

Het domein dient niet te weten dat er ergens een webshop bestaat.

[UST2393] Vh4: Handels: RegistreerContactGegevens implementeren
[UST2393] Vh4: Handels: RegistreerContactGegevens implementeren