CacheAdminA_trunk2010/vhTest/Fake/DOM/PM/Maatwerk/Syncs

DotNet

Checkout Tools

Line History

Line Count Graph
Line Count Graph

Stats

Commits this week: 0
Total Committers: 2
Last Commit: 16 Oct 20
Commits this week: 0
Total Lines of Code (LoC): 645
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 01 Sep 2022
7:55 pm

Jelle Baele

[ICT-3076][rvPVR]: Revego: BOM uitbreiden voor plaatmateriaal:

- Dotnet sync uitbreiden voor RVG

Options
    • -0
    • +6
    ./MaatwerkCalculator/Data/impl/SelectieKenmerkenRepository.cls.xml
  2. … 4 more files in changeset.
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
    • -1
    • +1
    ./MaatwerkCalculator/impl/ProductInfoInvuller.cls.xml
    • -1
    • +1
    ./MaatwerkCalculator/impl/KostenPlaatsWaardeInfoInvuller.cls.xml
  3. … 14 more files in changeset.
Thursday 04 Nov 2021
11:27 am

Peter Vercoutter

[ICT-1922] Incrementele sync van cache naar DotNet voor selectiekenmerken

- Refactoring code: code proper onderbrengen zodat logica niet langer in deployklasse zit

Options
    • -2
    • +14
    ./MaatwerkCalculator/Data/impl/SelectieKenmerkenRepository.cls.xml
  2. … 3 more files in changeset.
Tuesday 26 Oct 2021
4:25 pm

Jelle Baele

[ICT-2474] LBX SST BOM-builders porten naar .Net

- Uitbreiden SelectieKenmerkenRepository voor LBX en SST

Options
    • -5
    • +14
    ./MaatwerkCalculator/Data/impl/SelectieKenmerkenRepository.cls.xml
  2. … 2 more files in changeset.
Thursday 21 Jan 2021
10:01 am

Peter Vercoutter

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

- reviewopmerkingen: gebruik maken van Dynamic repository

Options
    • -75
    • +0
    ./MaatwerkCalculator/Data/impl/ProductPerSelectieKenmerkRepository.cls.xml
  2. … 4 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
    • -0
    • +7
    ./MaatwerkCalculator/impl/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

9:51 am

Peter Vercoutter

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

- Reviewopmerking: Zoek ipv Geef

Options
    • -93
    • +81
    ./MaatwerkCalculator/Data/impl/SelectieKenmerkenRepository.cls.xml
  2. … 6 more files in changeset.
9:33 am

Peter Vercoutter

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)