Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
[ICT-3076][rvPVR]: Revego: BOM uitbreiden voor plaatmateriaal:

- Dotnet sync uitbreiden voor RVG

    • -0
    • +6
    ./MaatwerkCalculator/Data/impl/SelectieKenmerkenRepository.cls.xml
  1. … 4 more files in changeset.
[ICT-2902] [rvJBA] BOMBOL .NET met schaduwprijzen mogelijk maken
[ICT-2902] [rvJBA] BOMBOL .NET met schaduwprijzen mogelijk maken
[ICT-2902] [rvJBA] BOMBOL .NET met schaduwprijzen mogelijk maken

- Request uitbreiden met 'IsSchaduwBerekening' parameter

- Kostenplaatsen en productInfo tabellen + sync uitbreiden met schaduwprijzen

    • -2
    • +2
    ./MaatwerkCalculator/impl/ProductInfoInvuller.cls.xml
    • -2
    • +2
    ./MaatwerkCalculator/impl/KostenPlaatsWaardeInfoInvuller.cls.xml
  1. … 14 more files in changeset.
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

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

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

    • -2
    • +2
    ./MaatwerkCalculator/impl/ProductInfoInvuller.cls.xml
  1. … 3 more files in changeset.
[ICT-1922] Incrementele sync van cache naar DotNet voor selectiekenmerken

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

    • -4
    • +16
    ./MaatwerkCalculator/Data/impl/SelectieKenmerkenRepository.cls.xml
  1. … 3 more files in changeset.
[ICT-2474] LBX SST BOM-builders porten naar .Net

- Uitbreiden SelectieKenmerkenRepository voor LBX en SST

    • -0
    • +12
    ./MaatwerkCalculator/Data/impl/SelectieKenmerkenRepository.cls.xml
  1. … 2 more files in changeset.
[ICT-1843] [+FMA] [rvWV] PM: Maatwerk: Selectiekenmerken (en prijs+gewicht) sync van Admin naar MvxBomBolCalculatorService

- reviewopmerkingen: gebruik maken van Dynamic repository

    • -79
    • +0
    ./MaatwerkCalculator/Data/impl/ProductPerSelectieKenmerkRepository.cls.xml
  1. … 4 more files in changeset.
[ICT-2123] PM: Maatwerk: PPSKenmerken sync naar .NET

- Refactoring zodat alles via Repository framework gebeurt

    • -0
    • +33
    ./MaatwerkCalculator/Data/PPSKenmerkRepository.cls.xml
  1. … 5 more files in changeset.
[ICT-1843] [+FMA] [rvWV] PM: Maatwerk: Selectiekenmerken (en prijs+gewicht) sync van Admin naar MvxBomBolCalculatorService

- Reviewopmerking

    • -2
    • +2
    ./MaatwerkCalculator/impl/ProductInfoInvuller.cls.xml
  1. … 6 more files in changeset.
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

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

- Reviewopmerking: Zoek ipv Geef

    • -31
    • +13
    ./MaatwerkCalculator/Data/SelectieKenmerkenRepository.cls.xml
    • -89
    • +89
    ./MaatwerkCalculator/Data/impl/SelectieKenmerkenRepository.cls.xml
  1. … 5 more files in changeset.
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

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)

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

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

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

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

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

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

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.

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)

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