CacheAdminA_trunk2010/vhTest/Mock/DOM/PM/Maatwerk/Syncs/DotNet/MaatwerkCalculator

impl

Checkout Tools

Line History

Line Count Graph
Line Count Graph

Stats

Commits this week: 0
Total Committers: 1
Last Commit: 18 Dec 20
Commits this week: 0
Total Lines of Code (LoC): 80
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 30 May 2022
11:59 am

Peter Vercoutter

11:08 am

Jelle Baele

Monday 23 May 2022
10:34 am

Peter Vercoutter

[ICT-2902] [rvJBA] BOMBOL .NET met schaduwprijzen mogelijk maken
[ICT-2902] [rvJBA] BOMBOL .NET met schaduwprijzen mogelijk maken
10:22 am

Peter Vercoutter

[ICT-2902] [rvJBA] BOMBOL .NET met schaduwprijzen mogelijk maken

- Request uitbreiden met 'IsSchaduwBerekening' parameter

- Kostenplaatsen en productInfo tabellen + sync uitbreiden met schaduwprijzen

Options
    • -2
    • +2
    ./ProductInfoInvuller.cls.xml
    • -2
    • +2
    ./KostenPlaatsWaardeInfoInvuller.cls.xml
  3. … 14 more files in changeset.
Monday 20 Dec 2021
12:57 pm

Peter Vercoutter

12:57 pm

Peter Vercoutter

Ik heb er ook niet echt een idee van. Deze klasse werd 100% gegenereerd en in de DB is dit nochtans een datetime datatype

Ik heb er ook niet echt een idee van. Deze klasse werd 100% gegenereerd en in de DB is dit nochtans een datetime datatype

Wednesday 15 Dec 2021
4:12 pm

Jelle Baele

4:12 pm

Jelle Baele

Waarom is deze plots een string ipv timestamp? In de 'ProductInfoInvuller.cls' wordt deze info ook nog meegegeven als een timestamp zie ik

Waarom is deze plots een string ipv timestamp? In de 'ProductInfoInvuller.cls' wordt deze info ook nog meegegeven als een timestamp zie ik

Thursday 09 Dec 2021
11:15 am

Peter Vercoutter

[ICT-2625] [rvJBA] Sync ProductInfo van cache naar .NET uitbreiden met Identnummer
[ICT-2625] [rvJBA] Sync ProductInfo van cache naar .NET uitbreiden met Identnummer
10:32 am

Peter Vercoutter

[ICT-2625] [rvJBA] Sync ProductInfo van cache naar .NET uitbreiden met Identnummer

Options
    • -2
    • +2
    ./ProductInfoInvuller.cls.xml
  2. … 3 more files in changeset.
Wednesday 20 Jan 2021
2:11 pm

Peter Vercoutter

Monday 18 Jan 2021
10:18 am

Peter Vercoutter

[ICT-1843] [+FMA] [rvWV] PM: Maatwerk: Selectiekenmerken (en prijs+gewicht) sync van Admin naar MvxBomBolCalculatorService

- Reviewopmerking

Options
    • -2
    • +2
    ./ProductInfoInvuller.cls.xml
  2. … 6 more files in changeset.
10:13 am

Peter Vercoutter

De implementatie is volgens mij wel correct, enkel de naamgeving is wat ongelukkig gekozen voor die klasse waarnaar ik verwijs

De implementatie is volgens mij wel correct, enkel de naamgeving is wat ongelukkig gekozen voor die klasse waarnaar ik verwijs

Friday 08 Jan 2021
11:28 am

Tom Vermeulen

11:26 am

Tom Vermeulen

Hier gaan toch nog wat testen bij moeten komen, want er is een hele hoop gedrag dat nu niet expliciet getest wordt. De SUT roept één van twee (nog niet) private methods op, maar die hebben op hun b...

Hier gaan toch nog wat testen bij moeten komen, want er is een hele hoop gedrag dat nu niet expliciet getest wordt. De SUT roept één van twee (nog niet) private methods op, maar die hebben op hun beurt weer wat while's en if's zitten. Testen moeten weergeven wat er gebeurt als de while leeg is, wat er in de if en else gebeurt enz enz

11:24 am

Tom Vermeulen

BestaatViaFilter en UpdateProductViaFilter zijn dingen voor een repo en horen hier eigenlijk niet te staan (dan kunt ge ze ook proper uitmocken voor de testen)

BestaatViaFilter en UpdateProductViaFilter zijn dingen voor een repo en horen hier eigenlijk niet te staan (dan kunt ge ze ook proper uitmocken voor de testen)

11:22 am

Tom Vermeulen

Deze twee lijnen kunt ge beter vervangen door Quit:ResulSet.HasNext()

Deze twee lijnen kunt ge beter vervangen door Quit:ResulSet.HasNext()

11:21 am

Tom Vermeulen

kunt ge hier dan niet de ProductInfoInvuller gebruiken? (2x)

kunt ge hier dan niet de ProductInfoInvuller gebruiken? (2x)

11:18 am

Tom Vermeulen

naming: een Geef is eigenlijk voor op een repo, maak hier eerder VulInProductInfo van

naming: een Geef is eigenlijk voor op een repo, maak hier eerder VulInProductInfo van

11:16 am

Tom Vermeulen

TECH.Mockable is al een TECH.RegisteredObject, dus die TECH.RegisteredObject mag weg

TECH.Mockable is al een TECH.RegisteredObject, dus die TECH.RegisteredObject mag weg

11:12 am

Tom Vermeulen

De standaardnaam voor een testklasse is vhUnitTest.<klassanaam van SUT>.<tetestenmethod>.Test.cls. Dat maakt het veel simpeler voor het geval er meer dan 1 publieke method in de klasse zit.

De standaardnaam voor een testklasse is vhUnitTest.<klassanaam van SUT>.<tetestenmethod>.Test.cls. Dat maakt het veel simpeler voor het geval er meer dan 1 publieke method in de klasse zit.

11:09 am

Tom Vermeulen

aan de testen te zien moet deze (en die hierboven) private zijn (want van de andere methods zijn geen testen)

aan de testen te zien moet deze (en die hierboven) private zijn (want van de andere methods zijn geen testen)

11:07 am

Tom Vermeulen

Dus KenmerkGroepenIterator is een Iterator van Repositories als ik het juist volg. Dat komt niet echt overeen met wat in de SUT staat, want daar wordt dat als string beschouwd. Hier zit dus nog 't ...

Dus KenmerkGroepenIterator is een Iterator van Repositories als ik het juist volg. Dat komt niet echt overeen met wat in de SUT staat, want daar wordt dat als string beschouwd. Hier zit dus nog 't een en 't ander niet juist

10:59 am

Tom Vermeulen

deze GevondenSelectieKenmerken worden nergens anders gebruikt, mag dus weg

deze GevondenSelectieKenmerken worden nergens anders gebruikt, mag dus weg

10:57 am

Tom Vermeulen

KenmerkGroep hernoemen naar KenmerkGroepRepository (want het is een repo, maakt het verderop duidelijker als er kenmerkgroep.kenmerkgroep staat. en eigenlijk kan je de lijn beter vervangen door ee...

KenmerkGroep hernoemen naar KenmerkGroepRepository (want het is een repo, maakt het verderop duidelijker als er kenmerkgroep.kenmerkgroep staat.

en eigenlijk kan je de lijn beter vervangen door een Set ipv een #dim

10:53 am

Tom Vermeulen

SelectieKenmerken hernoemen naar SelectieKenmerkenIterator

SelectieKenmerken hernoemen naar SelectieKenmerkenIterator

10:50 am

Tom Vermeulen

Naming convention: een Geef geeft 1 object terug (of een exception), een Zoek geeft een Iterator met al niet items erop terug.

Naming convention: een Geef geeft 1 object terug (of een exception), een Zoek geeft een Iterator met al niet items erop terug.

10:50 am

Tom Vermeulen

Naming convention: een Geef geeft 1 object terug (of een exception), een Zoek geeft een Iterator met al niet items erop terug. Dit moet dus "ZoekAlleSelectiekenmerken" zijn

Naming convention: een Geef geeft 1 object terug (of een exception), een Zoek geeft een Iterator met al niet items erop terug.
Dit moet dus "ZoekAlleSelectiekenmerken" zijn

10:46 am

Tom Vermeulen

Inject ipv Implementation

Inject ipv Implementation

10:45 am

Tom Vermeulen

LegeIterator -> ..EmptyIterator() (en opkuisen hierboven)

LegeIterator -> ..EmptyIterator() (en opkuisen hierboven)