HalffabItemsBoom.cls.xml

Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
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

- Wijziging ter voorbereiding van eenvoudigere halffabitemsboom voor encowayproducten. Proberen de wortel uit te schakelen. Uit de taor testen blijkt dat door deze conditie niet wordt gelopen

  1. … 2 more files in changeset.
[ICT-3982] Productcreatie in cache voor encowayconfiguraties

- Wijziging ter voorbereiding van eenvoudigere halffabitemsboom voor encowayproducten. Proberen de wortel uit te schakelen

  1. … 2 more files in changeset.
[ICT-2097] Niet-meeleveren voor Langtekst berekening
[ICT-2097] Niet-meeleveren voor Langtekst berekening
[ICT-2097] Niet-meeleveren voor Langtekst berekening

- Property VerwijderdeHalffabItems toegevoegd aan de HalffabItemsBoom

- Property VerwijderdeHalffabItems toegevoegd aan MaatwerkDetailsParsedFromJson

  1. … 1 more file in changeset.
Van mij mogen die eigenlijk terug weg. Worden niet gebruikt...

Van mij mogen die eigenlijk terug weg. Worden niet gebruikt...

Zet jij er dan de besproken check bij die controleert op de lengte aub? Misschien een goede om wat kennis over te verspreiden (demo / mailtje)?

Zet jij er dan de besproken check bij die controleert op de lengte aub? Misschien een goede om wat kennis over te verspreiden (demo / mailtje)?

Lijkt mij zeker ok om in die zin te verbeteren hoor. Was overblijfsel van omschakelen naar %-variabele.

Lijkt mij zeker ok om in die zin te verbeteren hoor. Was overblijfsel van omschakelen naar %-variabele.

Alvast voor het archief: % heb ik ingevoerd omdat er op verschillende plaatsen de berekenaar genewed wordt, en ik daar in eerste instantie niet op wou focussen. Geen idee hoeveel werk het is om die...

Alvast voor het archief: % heb ik ingevoerd omdat er op verschillende plaatsen de berekenaar genewed wordt, en ik daar in eerste instantie niet op wou focussen. Geen idee hoeveel werk het is om die verschillende plaatsen te reduceren.
Verschillende gebruikers verschillende versie van de boom: aan welk scenario denk je juist?

  • More
  • CR-299
  • finished reviewing
% variabelen ?? Is er dan geen gevaar dat er verschillende gebruikers een verschillende versie van de boom kunnen vasthebben ? PrijsBerekeningen vs ProductieAansturing

% variabelen ?? Is er dan geen gevaar dat er verschillende gebruikers een verschillende versie van de boom kunnen vasthebben ?

PrijsBerekeningen vs ProductieAansturing

Waarom % variabelen ???? Private properties toch beter

Waarom % variabelen ???? Private properties toch beter

Door de canonische waarde te gebruiken bestaat er de kans dat er een subscriptfout kan ontstaan wegens de Key te lang

Door de canonische waarde te gebruiken bestaat er de kans dat er een subscriptfout kan ontstaan wegens de Key te lang