Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
[ICT-5447] [rvJBA] Revego V3
    • -0
    • +14
    ./Kenmerk/impl/DataDefinitieRepository.cls.xml
  1. … 8 more files in changeset.
[ICT-5447] [rvJBA] Revego V3
    • -0
    • +114
    ./Kenmerk/impl/DataDefinitieRepository.cls.xml
  1. … 5 more files in changeset.
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)

Zijn deze lijnen in commentaar nog relevant?

Zijn deze lijnen in commentaar nog relevant?

Samengesteld ipv SamenGesteld

Samengesteld ipv SamenGesteld

'Dotnet' -> Encoway

'Dotnet' -> Encoway

Rol doet niets, mag weg

Rol doet niets, mag weg

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

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

Ik zou deze in een config item plaatsen

Ik zou deze in een config item plaatsen

In enu steken en hier ook naar verwijzen?

In enu steken en hier ook naar verwijzen?

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)

Talen kunnen eventueel in een aparte enu?

Talen kunnen eventueel in een aparte enu?

Variabele naam aanpassen naar Encoway ipv dotnet

Variabele naam aanpassen naar Encoway ipv dotnet

Variabele naam aanpassen naar Encoway ipv dotnet

Variabele naam aanpassen naar Encoway ipv dotnet

[ICT-3982] Productcreatie in cache voor encowayconfiguraties
[ICT-3982] Productcreatie in cache voor encowayconfiguraties
[ICT-3982] Productcreatie in cache voor encowayconfiguraties
  1. … 28 more files in changeset.
[ICT-2124] Move Prod.Kenmerk.impl.DataDefinitieRepository klasse
[ICT-2124] Move Prod.Kenmerk.impl.DataDefinitieRepository klasse
[ICT-2124] Move Prod.Kenmerk.impl.DataDefinitieRepository klasse

- Nieuwe fake en mock

    • -0
    • +33
    ./Kenmerk/impl/DataDefinitieTAORCubicRepository.cls.xml
    • -0
    • +0
    ./Kenmerk/impl/DataDefinitieRepository.cls.xml
  1. … 3 more files in changeset.
[ICT-2124] Move Prod.Kenmerk.impl.DataDefinitieRepository klasse

- Move Class uitgevoerd

    • -1
    • +1
    ./Kenmerk/impl/DataDefinitieRepository.cls.xml
  1. … 5 more files in changeset.
Reverse merged revision(s) 56420 from trunk2010:

Reverse merged revision(s) 56414 from trunk2010:

[ICT] Aanpassingen als gevolg van een shshsh die ik heb uitgevoerd lokaal...

........

revert omdat mijn xml export blijkbaar nogal verschillend is van de rest van de ventjes hier

revert van de revert omdat mijn wijzigingen wel goed waren, maar de rest van de ventjes niet :)

  1. … 64 more files in changeset.
Reverse merged revision(s) 56414 from trunk2010:

[ICT] Aanpassingen als gevolg van een shshsh die ik heb uitgevoerd lokaal...

........

revert omdat mijn xml export blijkbaar nogal verschillend is van de rest van de ventjes hier

  1. … 64 more files in changeset.
[ICT] Aanpassingen als gevolg van een shshsh die ik heb uitgevoerd lokaal...
  1. … 64 more files in changeset.
  • More
  • CR-692
  • finished reviewing
Over de weetniet velden heb ik nog niet teveel opmerkingen gegeven -> zullen hopelijk in de nabije toekomst WeetWelVeld1 ,WeetWelVeld2,WeetWelVeld3 worden

Over de weetniet velden heb ik nog niet teveel opmerkingen gegeven -> zullen hopelijk in de nabije toekomst WeetWelVeld1 ,WeetWelVeld2,WeetWelVeld3 worden