Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
[UST3920] EC: app: herbestel bij probleem: DI1 en nog niet verwerken
[UST3920] EC: app: herbestel bij probleem: DI1 en nog niet verwerken
beetje aangepast

beetje aangepast

1. aangepast 2. aangezien dat zo uit de pre-refactor-code komt, ga ik dat in eerste instantie niet aanpassen, want dan maskeer ik mischien gedrag dat ik nu niet ken. (en ondertussen hebben we hier ...

1. aangepast
2. aangezien dat zo uit de pre-refactor-code komt, ga ik dat in eerste instantie niet aanpassen, want dan maskeer ik mischien gedrag dat ik nu niet ken. (en ondertussen hebben we hier op den bureau er al een hele boom over opgezet )

voor de reformat http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/thumbs_down.gifvoor early quits

voor de reformat
voor early quits

Niets fout, maar http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif 1. Constructor injects als optionele parameters 2. Api's newen ipv ophalen van context,...

Niets fout, maar
1. Constructor injects als optionele parameters
2. Api's newen ipv ophalen van context, indien mogelijk

[UST3798] HNLT: Mogelijke leverdatums aanpassen
[UST3798] HNLT: Mogelijke leverdatums aanpassen
  • More
  • CR-593
  • finished reviewing
Gebruik eerder een dummy PersonalityKlantBepaler ipv een stub je doet er nikske mee -> kwestie van zo duidelijk mogelijk te zijn. Wat betreft ollekebollekeknol idem dat gaat op één of andere manier...

Gebruik eerder een dummy PersonalityKlantBepaler ipv een stub je doet er nikske mee -> kwestie van zo duidelijk mogelijk te zijn.
Wat betreft ollekebollekeknol idem dat gaat op één of andere manier met aandacht lopen terwijl het niet relevant is.

Dit geldt eigenlijk voor andere testen in deze klasse, je gebruikt Winkelkar.GeefKlantID() in de ene method en in de andere ollebolleknol waarom ?

was al gebeurd, maar blijkbaar niet mee in de review gestoken. zal wel fout van KCA zijn.

was al gebeurd, maar blijkbaar niet mee in de review gestoken. zal wel fout van KCA zijn.

Lijkt mij best om hiervoor een expliciet testje bij te maken

Lijkt mij best om hiervoor een expliciet testje bij te maken

[UST2939] Vh4: MaatwerkLijnProductBerekenaar en MP voor TBX fixen
[UST2939] Vh4: MaatwerkLijnProductBerekenaar en MP voor TBX fixen
  • More
  • CR-253
  • finished reviewing
Ik zou deze dan eerder TaxStandaardLade noemen zo is er meer en meer duidelijkheid, 't is dat dit in toekomst misschien wel nog van nut zou kunnen zijn.

Ik zou deze dan eerder TaxStandaardLade noemen zo is er meer en meer duidelijkheid, 't is dat dit in toekomst misschien wel nog van nut zou kunnen zijn.

als je dan toch iedere quote aan het aanpassen bent

als je dan toch iedere quote aan het aanpassen bent

[UST2405] Vh4_SVC: MaakOrderVanOfferte: Maatwerkproducten reeds maken na bestellen
[UST2405] Vh4_SVC: MaakOrderVanOfferte: Maatwerkproducten reeds maken na bestellen
  • More
  • CR-196
  • finished reviewing
Ik zou het eerder anders benoemen qua foutmelding : Ofwel expliciet testen op aantal lijnen in de winkelkar : en melden dat er geen lijnen zijn Ofwel de melding veranderen naar 'Er kon geen lever...

Ik zou het eerder anders benoemen qua foutmelding :

Ofwel expliciet testen op aantal lijnen in de winkelkar : en melden dat er geen lijnen zijn

Ofwel de melding veranderen naar 'Er kon geen leverdatum bepaald worden voor winkelkar 'Den_ID', mogelijks zitten er geen lijnen in de winkelkar'

[DEF574] Heldere exception indien geen lijnen, in GeefMogelijkeLeverdatums
[DEF574] Heldere exception indien geen lijnen, in GeefMogelijkeLeverdatums
  • More
  • CR-175
  • finished reviewing
[UST2330] Vh4: Dagelijks MailOverzicht naar VKP moet beslaglijnen van Vhisie4 winkelkarren bevatten
[UST2330] Vh4: Dagelijks MailOverzicht naar VKP moet beslaglijnen van Vhisie4 winkelkarren bevatten