CacheAdminA_trunk2010/APPS/VKP/Korting

impl

Checkout Tools

Line History

Line Count Graph
Line Count Graph

Stats

Commits this week: 0
Total Committers: 9
Last Commit: 06 Jul 15
Commits this week: 0
Total Lines of Code (LoC): 279
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:
Thursday 24 Jun 2021
8:16 am

Jo Willems

Monday 11 Jan 2021
8:46 am

Tom Vermeulen

8:46 am

Tom Vermeulen

de injecties zijn er, maar ge zijt uw testen vergeten toevoegen aan de review http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif

de injecties zijn er, maar ge zijt uw testen vergeten toevoegen aan de review

Monday 23 Nov 2020
8:57 am

Sam Van Hoey

Tuesday 17 Nov 2020
5:04 pm

Jo Willems

[ICT-1953] EC: MP: Häfele: kortingbestanden ook naar Vhisie4 sturen
[ICT-1953] EC: MP: Häfele: kortingbestanden ook naar Vhisie4 sturen
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 25 Oct 2018
8:46 am

Tom Vermeulen

Tuesday 23 Oct 2018
12:28 pm

Martijn Vierstraete

12:25 pm

Martijn Vierstraete

Als het genoeg kriebelt mag alles na de eerste quit weg http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif

Als het genoeg kriebelt mag alles na de eerste quit weg

10:27 am

Tom Vermeulen

[UST4089] VH4 Hafele - korting TAX
[UST4089] VH4 Hafele - korting TAX
Monday 29 Jan 2018
3:51 pm

Wim Vermeulen

10:40 am

Martijn Vierstraete

  • More
  • CR-956
  • finished reviewing
10:40 am

Martijn Vierstraete

Niet nodig om aan te passen, enkel ter info: bij booleans zet ik in de naamgeving meestal Is..., Mag..., Moet..., Heeft... In dit geval zou dit worden: MoetKortingKopierenVanTbxNaarLbx

Niet nodig om aan te passen, enkel ter info: bij booleans zet ik in de naamgeving meestal Is..., Mag..., Moet..., Heeft... In dit geval zou dit worden: MoetKortingKopierenVanTbxNaarLbx

10:06 am

Wim Vermeulen

[UW427] UNI Haefele NL : Kortingen TBX ook voor LBX
[UW427] UNI Haefele NL : Kortingen TBX ook voor LBX