Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
[ICT-1769] EC: MP: ORGALUX: HTTP POST HandelInformeerder: productlijnen opnemen

- testje erbij

  1. … 2 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

- test data mee vergeten committen

[UST3920] EC: app: herbestel bij probleem: DI1 en nog niet verwerken

- Test data beschikbaar stellen winkelkar met maatwerklijn met HerbestellingBijProbleem

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
[UST3798] HNLT: Mogelijke leverdatums aanpassen

- VerzendWijzeBepaler uitgebreid met Häfele-logica

- Poging gedaan wat testjes toe te voegen

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

- testjes doen slagen

- testjes moeten nog herwerkt worden om de $$$ElkeWaardes weg te krijgen (die stonden er al!!!)

  1. … 5 more files in changeset.
[UST2939] Vh4: MaatwerkLijnProductBerekenaar en MP voor TBX fixen
[UST2939] Vh4: MaatwerkLijnProductBerekenaar en MP voor TBX fixen
[UST2405r] Vh4_SVC: MaakOrderVanOfferte: Maatwerkproducten reeds maken na bestellen

- '= -> $$$Not

- taxstandaardlade

  1. … 3 more files in changeset.
  • 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
[UST2405] Vh4_SVC: MaakOrderVanOfferte: Maatwerkproducten reeds maken na bestellen

- refact - maatwerklijn naar aparte klasse.

  1. … 1 more file in changeset.
  • More
  • CR-196
  • finished reviewing