Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
If $$$HasLengfth(Werkost) (zie lijn 181)

If $$$HasLengfth(Werkost) (zie lijn 181)

LadeRegistratieIterator ipv LadeRegistraties, dat maakt het lezen van de code duidelijker verderop. "LadeRegistraties" kan een list, een array, een iterator, ... zijn

LadeRegistratieIterator ipv LadeRegistraties, dat maakt het lezen van de code duidelijker verderop. "LadeRegistraties" kan een list, een array, een iterator, ... zijn

de opgeroepen functies gaan dus .Zoek* moeten heten (zie comment in repo)

de opgeroepen functies gaan dus .Zoek* moeten heten (zie comment in repo)

Een Geef geeft 1 product terug, een zoek mogelijks 0 tot meerdere. Implementatie is wel aangepast, maar de methodname is nog altijd een 'geef'. De oproeper hiervan verwacht dus 1 object terug te kr...

Een Geef geeft 1 product terug, een zoek mogelijks 0 tot meerdere. Implementatie is wel aangepast, maar de methodname is nog altijd een 'geef'. De oproeper hiervan verwacht dus 1 object terug te krijgen (maar dat is niet gegarandeerd door de gebruikte 'zoek'.
Naam (en bijhorende testen) aanpassen.
Idem met wijziging hieronder

[ICT-4601] [rvTVE] Afdrukken probox etiket robuuster maken
[ICT-4601] [rvTVE] Afdrukken probox etiket robuuster maken
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-3275] [rvTVE] Oud framework voor CubeIQ opkuisen
[ICT-3275] [rvTVE] Oud framework voor CubeIQ opkuisen
[ICT-4439] [rvPVR] Ophalen BOMBOL uit Encoway:
[ICT-4439] [rvPVR] Ophalen BOMBOL uit Encoway:
[ICT-4361] MP Haefele Spanje opzetten
[ICT-4361] MP Haefele Spanje opzetten
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

Ergens zou deze method naam moeten duidelijk maken dat het originele product verwijderd wordt lijkt mij?

Ergens zou deze method naam moeten duidelijk maken dat het originele product verwijderd wordt lijkt mij?

Is dit niet 'gevaarlijk'? Moet je niet checken of er 1 product aanwezig is en anders een error gooien?

Is dit niet 'gevaarlijk'? Moet je niet checken of er 1 product aanwezig is en anders een error gooien?

Is écht muggenziften, maar deze 'if' mag op lijn 566. Ik durf het bijna niet typen :-p

Is écht muggenziften, maar deze 'if' mag op lijn 566. Ik durf het bijna niet typen :-p

Idd, is ook niet belangrijk voor het valideren of het uniek is

Idd, is ook niet belangrijk voor het valideren of het uniek is

Ik zou deze in een config item plaatsen

Ik zou deze in een config item plaatsen