CacheAdminA_trunk2010/APPS/common

VKP

Checkout Tools

Line History

Line Count Graph
Line Count Graph

Stats

Commits this week: 0
Total Committers: 4
Last Commit: 31 Mar 16
Commits this week: 0
Total Lines of Code (LoC): 78
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
Thursday 13 Dec 2018
3:17 pm

Kurt Callebaut

Monday 10 Dec 2018
9:40 am

Tommy Hebb

9:01 am

Kurt Callebaut

[DEF736] Vhisie4 productiedagen die leverdatums bepalen moet afhankelijk zijn van klantnummer. (wordt...
[DEF736] Vhisie4 productiedagen die leverdatums bepalen moet afhankelijk zijn van klantnummer. (wordt...
Tuesday 02 Oct 2018
4:08 pm

Jo Willems

Monday 24 Sep 2018
8:32 am

Kurt Callebaut

8:08 am

Kurt Callebaut

yep, ik zal een kaartje maken om een refactor door te voeren van deze klasse.

yep, ik zal een kaartje maken om een refactor door te voeren van deze klasse.

8:06 am

Kurt Callebaut

idd

idd

8:05 am

Kurt Callebaut

altijd handig om es echte data te visualiseren en repo te testen.

altijd handig om es echte data te visualiseren en repo te testen.

Thursday 20 Sep 2018
2:42 pm

Martijn Vierstraete

2:42 pm

Martijn Vierstraete

$LL ipv $L en liefst voluit $ListLength

$LL ipv $L en liefst voluit $ListLength

2:41 pm

Martijn Vierstraete

Is deze klasse nog nodig?

Is deze klasse nog nodig?

2:37 pm

Martijn Vierstraete

Groeperen we de constructor injections?

Groeperen we de constructor injections?

2:36 pm

Martijn Vierstraete

1. Er wordt niet getest of de LeverDatumBepaler opgeroepen wordt 2. De overvloed aan constructorparameters geeft aan dat functionaliteit afgezonderd kan worden: bv. WinkelkarLijnAanOrderToevoeger. ...

1. Er wordt niet getest of de LeverDatumBepaler opgeroepen wordt
2. De overvloed aan constructorparameters geeft aan dat functionaliteit afgezonderd kan worden: bv. WinkelkarLijnAanOrderToevoeger. (Hierdoor kun je 3 constructorparameters vervangen door 1 als je de LeverDatumBepaler meerekent). Ook de inhoud van deze method lijkt geschikt om af te zonderen.

1:58 pm

Martijn Vierstraete

Interface kan weggewerkt worden

Interface kan weggewerkt worden

11:45 am

Kurt Callebaut

[UST4023] SFS-LD : Leverdatums aanbieden op vhisie4
[UST4023] SFS-LD : Leverdatums aanbieden op vhisie4
Friday 31 Aug 2018
9:20 am

Martijn Vierstraete

Thursday 30 Aug 2018
5:13 pm

Jo Willems

[UST3920] EC: app: herbestel bij probleem: DI1 en nog niet verwerken
[UST3920] EC: app: herbestel bij probleem: DI1 en nog niet verwerken
Tuesday 29 Aug 2017
4:10 pm

Kurt Callebaut

4:02 pm

Kurt Callebaut

4:02 pm

Kurt Callebaut

inderdaad.. gecopypasted http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif

inderdaad.. gecopypasted

4:00 pm

Kurt Callebaut

Monday 28 Aug 2017
2:48 pm

Martijn Vierstraete

  • More
  • CR-460
  • finished reviewing
2:48 pm

Martijn Vierstraete

In de nieuw toegevoegde tests heb je het over de gebruikte verzendwijze, je assert echter nergens op deze verzendwijze. Het al dan niet asap mogelijk zijn staat hier volgens mij los van. Gesuggeree...

In de nieuw toegevoegde tests heb je het over de gebruikte verzendwijze, je assert echter nergens op deze verzendwijze. Het al dan niet asap mogelijk zijn staat hier volgens mij los van. Gesuggereerde oplossing: tests toevoegen die doen wat de testomschrijving zegt, of mss korter: de testomschrijving aanpassen (à la "standaard verzendwijze van de winkelkar wordt opgehaald bij leeg leveradres")

2:04 pm

Martijn Vierstraete

  • More
  • CR-418
  • finished reviewing
1:49 pm

Martijn Vierstraete

Waarom niet gewoon FrequentieTabel als naam van de variable?

Waarom niet gewoon FrequentieTabel als naam van de variable?

1:25 pm

Martijn Vierstraete

  • More
  • CR-404
  • finished reviewing
1:25 pm

Martijn Vierstraete

Is het in dit geval niet meer leesbaar als je vergelijkt met "" ipv parameter GeenLeverAdresId? Deze parameter wordt slechts 1x gebruikt.

Is het in dit geval niet meer leesbaar als je vergelijkt met "" ipv parameter GeenLeverAdresId? Deze parameter wordt slechts 1x gebruikt.

12:48 pm

Kurt Callebaut

Friday 25 Aug 2017
2:49 pm

Martijn Vierstraete

  • More
  • CR-395
  • finished reviewing
2:41 pm

Martijn Vierstraete

Waarom werk je hier met een fake Winkelkar, een mock of een nieuwe lege instantie van de Winkelkar zou toch ook moeten werken?

Waarom werk je hier met een fake Winkelkar, een mock of een nieuwe lege instantie van de Winkelkar zou toch ook moeten werken?