CacheAdminA_trunk2010/WS/Vhisie4

WinkelkarExport

Checkout Tools

Line History

Line Count Graph
Line Count Graph

Stats

Commits this week: 0
Total Committers: 5
Last Commit: 11 Jan 17
Commits this week: 0
Total Lines of Code (LoC): 465
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
Wednesday 07 Nov 2018
10:19 am

Jo Willems

Monday 05 Nov 2018
10:09 am

Martijn Vierstraete

10:09 am

Martijn Vierstraete

Naamgeving stub ipv mock

Naamgeving stub ipv mock

10:07 am

Martijn Vierstraete

Is deze test overbodig?

Is deze test overbodig?

10:06 am

Martijn Vierstraete

casing

casing

10:01 am

Martijn Vierstraete

Casing! (zeker bij nieuwe klasses)

Casing! (zeker bij nieuwe klasses)

Friday 26 Oct 2018
4:19 pm

Martijn Vierstraete

Bij nieuwe code mag je api's newen

Bij nieuwe code mag je api's newen

Tuesday 23 Oct 2018
11:34 am

Jo Willems

TECH.Interface is niet genoeg om een mock te laten genereren.... Open / closed principe : extend,not modify inherited behaviour...

TECH.Interface is niet genoeg om een mock te laten genereren....
Open / closed principe : extend,not modify inherited behaviour...

Monday 15 Oct 2018
12:09 pm

Martijn Vierstraete

Waarom nog een interface maken? Is de (stilzwijgende) afspraak niet om objecten te laten overerven van TECH.Mockable? In deze situatie zou je een Geen aanpasser als basis kunnen gebruiken of een ab...

Waarom nog een interface maken? Is de (stilzwijgende) afspraak niet om objecten te laten overerven van TECH.Mockable?
In deze situatie zou je een Geen aanpasser als basis kunnen gebruiken of een abstracte base-class.
Daarenboven kies je beter 1 manier om aan te geven dat je mocks/fakes wil: ofwel overerven van Interface ofwel van Mockable

12:08 pm

Martijn Vierstraete

Casing!

Casing!

12:07 pm

Martijn Vierstraete

Naamgeving kan iets explicieter. Bv. Test: NietMeeleverens worden aangepast

Naamgeving kan iets explicieter. Bv. Test: NietMeeleverens worden aangepast

11:58 am

Martijn Vierstraete

Naast het maken van een "Standaard" (aanpasser) kon je hier ook een "Geen" (aanpasser) maken. Dit is mogelijk nog anders dan het standaardgedrag.

Naast het maken van een "Standaard" (aanpasser) kon je hier ook een "Geen" (aanpasser) maken. Dit is mogelijk nog anders dan het standaardgedrag.

11:53 am

Martijn Vierstraete

Je zou kunnen aangeven in de test dat de KlantId en personalityId van de winkelkar gebruikt worden om de juiste aanpasser te maken.

Je zou kunnen aangeven in de test dat de KlantId en personalityId van de winkelkar gebruikt worden om de juiste aanpasser te maken.

9:36 am

Jo Willems

[UST4047] Vh4: MP: Beslag Design: niet meeleveren aanvinken bij "migratie" winkelkar voor doorbestellen...
[UST4047] Vh4: MP: Beslag Design: niet meeleveren aanvinken bij "migratie" winkelkar voor doorbestellen...
Friday 16 Mar 2018
9:15 am

Kurt Callebaut

Wednesday 14 Mar 2018
3:05 pm

Martijn Vierstraete

3:05 pm

Martijn Vierstraete

geen tests? http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif

geen tests?

3:05 pm

Martijn Vierstraete

[UW462] bestelopmerkingen toevoegen aan winkelkarexport voor vhisie4
[UW462] bestelopmerkingen toevoegen aan winkelkarexport voor vhisie4
Tuesday 22 Aug 2017
8:37 am

Tom Vermeulen

Wednesday 28 Jun 2017
4:57 pm

Jo Claes

  • More
  • CR-282
  • finished reviewing
Thursday 22 Jun 2017
3:24 pm

Jo Claes

Deze klasse mocht eigenlijk blijven bestaan en afhankelijk van welke offerte hij binnen kreeg ging door delegeren naar vhisie3 of vhisie4 versie, dan had je vorige opmerkingen qua fabriekje niet aa...

Deze klasse mocht eigenlijk blijven bestaan en afhankelijk van welke offerte hij binnen kreeg ging door delegeren naar vhisie3 of vhisie4 versie, dan had je vorige opmerkingen qua fabriekje niet aan uw been

3:22 pm

Jo Claes

Factory method hoort eigenlijk niet thuis op een service klasse

Factory method hoort eigenlijk niet thuis op een service klasse

3:21 pm

Jo Claes

Opkuisen

Opkuisen

3:21 pm

Jo Claes

Eigenlijk was het de moment om dit ServiceLocator patroon te vervangen door een fabriekje.

Eigenlijk was het de moment om dit ServiceLocator patroon te vervangen door een fabriekje.

3:17 pm

Jo Claes

Naamgeving van de klasse lijkt mij beter dat het gewoon OfferteConverter is en dat daar dan methods kunnen opkomen alla VanEcommerce en NaarEcommerce. Zodoende we maar één converter klasse hebben ...

Naamgeving van de klasse lijkt mij beter dat het gewoon OfferteConverter is en dat daar dan methods kunnen opkomen alla VanEcommerce en NaarEcommerce.

Zodoende we maar één converter klasse hebben die converteert in beide richtingen.

3:07 pm

Tom Vermeulen

Ik heb dit kaartje al afgesloten, maar ik weet wel dat er nog een hoop opkuiswerk volgt, plus nog wat extra functionaliteit (maar die staat in volgende kaartjes) Ook net opgevallen dat ik nog niet...

Ik heb dit kaartje al afgesloten, maar ik weet wel dat er nog een hoop opkuiswerk volgt, plus nog wat extra functionaliteit (maar die staat in volgende kaartjes)

Ook net opgevallen dat ik nog niet met Override-objectje werk.

2:13 pm

Tom Vermeulen

[UST2486] Vh4: Handels: Migratie na bestelling
[UST2486] Vh4: Handels: Migratie na bestelling
Monday 22 May 2017
3:44 pm

Jo Claes

3:43 pm

Jo Claes

2:48 pm

Tom Vermeulen

  • More
  • CR-242
  • finished reviewing