CacheAdminA_trunk2010/vhTest/Fake/APPS/VKP/Korting

impl

Checkout Tools

Line History

Line Count Graph
Line Count Graph

Stats

Commits this week: 0
Total Committers: 5
Last Commit: 19 Jun 15
Commits this week: 0
Total Lines of Code (LoC): 124
Net change in LoC this week: 0

Commit Activity

Commits By Day In Week
52 week commits volume
Commits By Day In Week
Commits by day
Commits By Day In Week
Commits by hour
Commits By Hour In Day
Commit calendar
 

Most active committers (90 days)

loading
loading
  • last updated a few seconds ago
All Commits Reviews
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Monday 18 Jul 2022
10:06 am

Joannes Lemmens

Wednesday 18 May 2022
2:17 pm

Tom Vermeulen

Monday 09 May 2022
10:36 am

Joannes Lemmens

Unit test toegevoegd die gedrag voor niet-Haefele personality test.

Unit test toegevoegd die gedrag voor niet-Haefele personality test.

10:23 am

Joannes Lemmens

Deze moet dan False teruggeven. Khad inderdaad wel wat moeite met een goeie naam te kiezen... en deze lijkt nogal afhankelijk van hoe ge het leest. Het idee was: "Niet-BE Haefele Personality", dus ...

Deze moet dan False teruggeven. Khad inderdaad wel wat moeite met een goeie naam te kiezen... en deze lijkt nogal afhankelijk van hoe ge het leest.
Het idee was: "Niet-BE Haefele Personality", dus een Haefele personality die niet HaefeleBE is. Ander idee is bijvoorbeeld "IsHaefelePersonalityMaarNietHaefeleBE" ofzo?

Nuja, het is volledig afhankelijk van hoe verschillende bedrijven die data doorsturen. Als er bvb een nieuwe dat exact zoals Haefele wilt doen dan moet die hier ook toegevoegd worden en dus moet de naam veranderd worden.

9:11 am

Tom Vermeulen

zie ook opmerking in kortingservicehelper zelf: ge gaat hier moeten duidelijk maken (met een of meerdere extra testjes) wat het gedrag gaat zijn als er met een willekeurige personality deze method ...

zie ook opmerking in kortingservicehelper zelf: ge gaat hier moeten duidelijk maken (met een of meerdere extra testjes) wat het gedrag gaat zijn als er met een willekeurige personality deze method wordt opgeroepen (leeg, error, defaultgedrag, ....)

9:09 am

Tom Vermeulen

wat moet deze doen als bv SFS binnenkomt (of een andere personality die niets met Häfele te maken heeft?

wat moet deze doen als bv SFS binnenkomt (of een andere personality die niets met Häfele te maken heeft?

Monday 04 Apr 2022
12:05 pm

Joannes Lemmens

[ICT-2786] Aparte korting voor Orgalux bij multipersonalities aanmaken:
[ICT-2786] Aparte korting voor Orgalux bij multipersonalities aanmaken:
Monday 23 Nov 2020
8:57 am

Sam Van Hoey

Tuesday 14 Jul 2020
8:06 am

Tom Vermeulen

Monday 22 Jun 2020
1:25 pm

Tom Vermeulen

normaalgezien geen early quits in een method (tenzij helemaal bovenaan)

normaalgezien geen early quits in een method (tenzij helemaal bovenaan)

1:02 pm

Tom Vermeulen

Grammarnazicomment: invoerbestand is 1 woord. http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif

Grammarnazicomment: invoerbestand is 1 woord.

12:58 pm

Tom Vermeulen

klein detail, maar het kan zeker geen kwaad om single quotes in de output rond Bestandsnaam te voorzien. Maakt het duidelijker dat er iets had moeten staan indien BestandsNaam leeg is. In dit concr...

klein detail, maar het kan zeker geen kwaad om single quotes in de output rond Bestandsnaam te voorzien. Maakt het duidelijker dat er iets had moeten staan indien BestandsNaam leeg is. In dit concreet geval maakt het niet zoveel uit, maar het is gewoon een goede gewoonte.

12:51 pm

Tom Vermeulen

Het verschil met de test hieronder is dat je in dit geval de VerwerkCustomer maar 1 gaat oproepen, hieronder 2x. Je kan dat gedrag afdwingen door nog een .ExactAantalKeer(1) (of (2) natuurlijk) toe...

Het verschil met de test hieronder is dat je in dit geval de VerwerkCustomer maar 1 gaat oproepen, hieronder 2x. Je kan dat gedrag afdwingen door nog een .ExactAantalKeer(1) (of (2) natuurlijk) toe te voegen aan bovenstaande lijn. Als je dat niet doet, zijn de beide testen virtueel hetzelfde (behalve de maxnumberofcustomers) en weet je als lezer niet echt waar het grote verschil zit.

(idem voor de testen hieronder natuurlijk)

12:48 pm

Tom Vermeulen

Test met 0 klanten niet vergeten. Het zou kunnen dat klant crappy data doorstuurt.

Test met 0 klanten niet vergeten. Het zou kunnen dat klant crappy data doorstuurt.

12:06 pm

Sam Van Hoey

[ICT-1528] EC: MP: Häfele DE: Kortingen-bestand uittesten
[ICT-1528] EC: MP: Häfele DE: Kortingen-bestand uittesten
Thursday 27 Apr 2017
1:38 pm

Jo Claes

1:38 pm

Jo Claes

Niet alle commits van ICT242 zijn opgenomen : 'k weet het - kzou de SVN repo hier ook moeten aan toevoegen ofwel die code in de common steken ben er nog niet 100% wat hier mee te doen. Geen testen ...

Niet alle commits van ICT242 zijn opgenomen : 'k weet het - kzou de SVN repo hier ook moeten aan toevoegen ofwel die code in de common steken ben er nog niet 100% wat hier mee te doen.
Geen testen ==> needs resolution -> De testen zitten dus in de svn-repo

Wednesday 26 Apr 2017
3:39 pm

Martijn Vierstraete

  • More
  • CR-188
  • finished reviewing
3:39 pm

Martijn Vierstraete

Niet alle commits van ICT242 zijn opgenomen in deze review *Geen testen ==> needs resolution http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif *Ik zie g...
  • Niet alle commits van ICT242 zijn opgenomen in deze review
  • Geen testen ==> needs resolution
  • Ik zie geen inhoudelijke wijzigingen, enkel code generatie aanpassingen (bv. package name, cdata)
Thursday 20 Apr 2017
4:01 pm

Jo Claes

4:00 pm

Jo Claes

[ICT242] - Synchronizer terug werkend krijgen
[ICT242] - Synchronizer terug werkend krijgen