CacheAdminA_trunk2010

Prod

Checkout Tools

Line History

Line Count Graph
Line Count Graph

Stats

Commits this week: 0
Total Committers: 12
Last Commit: 19 Jun 15
Commits this week: 0
Total Lines of Code (LoC): 40.636
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 13 May 2024
10:32 am

Peter Vercoutter

Tuesday 02 Apr 2024
11:23 am

Jelle Baele

Ik had deze in de klassenaam zelf misschien ook "converter" bijgezet. Nu is dit HalffabItemsboom, maar is op die manier niet echt duidelijk dat het om een converter gaat zonder naar de folder te ki...

Ik had deze in de klassenaam zelf misschien ook "converter" bijgezet. Nu is dit HalffabItemsboom, maar is op die manier niet echt duidelijk dat het om een converter gaat zonder naar de folder te kijken (zelfde voor LBXKenmerken)

11:21 am

Jelle Baele

11:20 am

Jelle Baele

Zijn deze lijnen in commentaar nog relevant?

Zijn deze lijnen in commentaar nog relevant?

11:15 am

Jelle Baele

Samengesteld ipv SamenGesteld

Samengesteld ipv SamenGesteld

11:14 am

Jelle Baele

'Dotnet' -> Encoway

'Dotnet' -> Encoway

Monday 11 Mar 2024
11:29 am

Jelle Baele

Rol doet niets, mag weg

Rol doet niets, mag weg

11:27 am

Jelle Baele

ProductApi is nergens gedefinieerd als input parameter, dus deze inject is niet nodig

ProductApi is nergens gedefinieerd als input parameter, dus deze inject is niet nodig

11:21 am

Jelle Baele

Ik zou dit niet als constructor porperty meegeven, want nu moet je overal waar je de VHConfigHelper wilt gebruiken, eerst hem initialiseren in de code zelf. Op die manier kan je moeilijker de vhcon...

Ik zou dit niet als constructor porperty meegeven, want nu moet je overal waar je de VHConfigHelper wilt gebruiken, eerst hem initialiseren in de code zelf. Op die manier kan je moeilijker de vhconfighelper injecteren.

Er is wel de method 'ZetVhConfig', maar die moet je dan eerst aanroepen vooraleer je de andere methods kan gebruiken. Ergens lijkt mij dat wat zot, omdat je dan snel fouten kan maken

Thursday 07 Mar 2024
1:47 pm

Jelle Baele

Ik zou deze in een config item plaatsen

Ik zou deze in een config item plaatsen

Friday 01 Mar 2024
4:26 pm

Jelle Baele

In enu steken en hier ook naar verwijzen?

In enu steken en hier ook naar verwijzen?

4:23 pm

Jelle Baele

Mijn gevoel is om dit ook in een aparte klasse te plaatsen en onder test te steken (ook al zal deze klasse dan vrij klein zijn)

Mijn gevoel is om dit ook in een aparte klasse te plaatsen en onder test te steken (ook al zal deze klasse dan vrij klein zijn)

4:19 pm

Jelle Baele

Talen kunnen eventueel in een aparte enu?

Talen kunnen eventueel in een aparte enu?

4:12 pm

Jelle Baele

Variabele naam aanpassen naar Encoway ipv dotnet

Variabele naam aanpassen naar Encoway ipv dotnet

4:12 pm

Jelle Baele

Variabele naam aanpassen naar Encoway ipv dotnet

Variabele naam aanpassen naar Encoway ipv dotnet

9:48 am

Peter Vercoutter

[ICT-3982] Productcreatie in cache voor encowayconfiguraties
[ICT-3982] Productcreatie in cache voor encowayconfiguraties
Tuesday 13 Jun 2023
1:37 pm

Wim Vermeulen

11:57 am

Wim Vermeulen

No big deal. Codebase in t Nederlands, maar StringUtils.Replace() is Engels, dus wellicht geen taal-reset van mijn brains http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/em...

No big deal.
Codebase in t Nederlands, maar StringUtils.Replace() is Engels, dus wellicht geen taal-reset van mijn brains

Maar in feite zetten we geen commentaar in onze code (= clean code principle!)
Hier, in dit specifiek geval is het echter om te benadrukken dat ALLE spaties moeten vervangen worden.
Uitzondering bevestigt de regel.

8:12 am

Matthias Claessens

8:12 am

Matthias Claessens

Algemene vraag: Ik heb als PM tester enkele opmerkingen in het Jira ticket geplaatst. Moet ik hier al reviewer rekening mee houden tijdens de review? De uitleg van Tom was dat de reviewer enkel d...

Algemene vraag:

Ik heb als PM tester enkele opmerkingen in het Jira ticket geplaatst.
Moet ik hier al reviewer rekening mee houden tijdens de review?

De uitleg van Tom was dat de reviewer enkel de kwaliteit van de code moet beoordelen.
Maar er is een optie 'factually incorrect' in de tool, en mogelijks vallen enkele van mijn opmerkingen onder deze noemer?

Graag advies.

7:36 am

Matthias Claessens

Style: *Codebase is volledig in het Nederlands, waarom deze comment dan in het Engels?

Style:

  • Codebase is volledig in het Nederlands, waarom deze comment dan in het Engels?
Tuesday 06 Jun 2023
3:25 pm

Wim Vermeulen

[ICT-3685][rvMCL] PRJ029 - Uitfaseren WWW03 : ProductInfo via Vhisie4 url i.p.v. product.aspx
[ICT-3685][rvMCL] PRJ029 - Uitfaseren WWW03 : ProductInfo via Vhisie4 url i.p.v. product.aspx
Monday 01 Aug 2022
8:32 am

Jo Willems

Wednesday 13 Apr 2022
11:50 am

Wim Vermeulen

Monday 04 Apr 2022
12:47 pm

Peter Vercoutter

Friday 01 Apr 2022
5:02 pm

Wim Vermeulen

[ICT-2813] PM: Kostenplaatsen invullen : Schaduw voor grote prijswijziging op 01/05/2022
[ICT-2813] PM: Kostenplaatsen invullen : Schaduw voor grote prijswijziging op 01/05/2022
Monday 07 Feb 2022
11:00 am

Wim Vermeulen

8:30 am

Peter Vercoutter

Thursday 03 Feb 2022
11:59 am

Wim Vermeulen

Ik stel voor dat we de private method MaakProductVanNietTbxKenmerken() hernoemen naar iets wat duidt op ons maarwerkframework, of alleszins dat ook voor TbxKenmerken werkt. Feel free om een naam te...

Ik stel voor dat we de private method MaakProductVanNietTbxKenmerken() hernoemen naar iets wat duidt op ons maarwerkframework, of alleszins dat ook voor TbxKenmerken werkt.
Feel free om een naam te bedenken, alleen of samen met mij

11:56 am

Wim Vermeulen

[ICT-2694][rvPVR] TBX MaakProductVanKenmerken : KenmerkDataDefinities toevoegen aan nieuwe calculatie...
[ICT-2694][rvPVR] TBX MaakProductVanKenmerken : KenmerkDataDefinities toevoegen aan nieuwe calculatie...