Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
[ICT-5909] Tecon: LBX Amperos wijzigen
  1. … 20 more files in changeset.
[ICT-5643] CPConfig TBX ondersteunen in cache
  1. … 8 more files in changeset.
[ICT-5644] CPConfig MVX ondersteunen in cache
  1. … 18 more files in changeset.
[ICT-4623] [rvPVR] Integratietesten aansturingacties via Encoway afwerken:

- wlips weg

- extra spatie verwijderen

  1. … 1 more file in changeset.
Vrij veel code in 1 method, ik zou die wat meer opsplitsen http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif. Ik zie ook geen UT's, dat kan op die manier ...

Vrij veel code in 1 method, ik zou die wat meer opsplitsen . Ik zie ook geen UT's, dat kan op die manier wel opgelost worden

Commentaar mag weg

Commentaar mag weg

Is deze implementatie niet heel gelijkaardig aan die voor MVX? Lijkt mij dat deze in de common klasse kan http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif

Is deze implementatie niet heel gelijkaardig aan die voor MVX? Lijkt mij dat deze in de common klasse kan

[PRJ051-26] [rvJBA] Papier voor aansturing BWC freesmachine (Homag) vervangen door barcode
[PRJ051-26] [rvJBA] Papier voor aansturing BWC freesmachine (Homag) vervangen door barcode
[PRJ051-26] [rvJBA] Papier voor aansturing BWC freesmachine (Homag) vervangen door barcode
  1. … 7 more files in changeset.
[ICT-4749] [rvJBA] RVG: Nieuwe kenmerken voor tip-on kenmerken
[ICT-4749] [rvJBA] RVG: Nieuwe kenmerken voor tip-on kenmerken
[ICT-4749] [rvJBA] RVG: Nieuwe kenmerken voor tip-on kenmerken
  1. … 19 more files in changeset.
[ICT-3982] Productcreatie in cache voor encowayconfiguraties

- Reviewopmerkingen verwerken

  1. … 14 more files in changeset.
[ICT-4567] [rvJBA] RVG Wijzigingen april release

- 2 Nieuwe kenmerken toevoegen in verschillende lagen/convertoren

  1. … 12 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

[ICT-4303] RVG: Aanpassingen Blum release 1/2/2024
[ICT-4303] RVG: Aanpassingen Blum release 1/2/2024
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