Comment Search

Deze mag volgens mij weg, dat zal nooit meer omgedraaid worden

Op zich goed, maar de WerklijstItemsUitvoerder zou eens een refactor kunnen gebruiken, er zit nu nogal veel logica in voor de mattenlijsten (en de synchrostangen, dat is door mij )

Ik herinner me dat René van Encoway had gezegd dat "Concat()" ook werkt met meer dan 2 parameters : Concat ( str1, str2, str3 )
dan moet je geen 2 concats nesten, en dat leest ook gemakkelijker.

Zeer kleine opmerking, dus jira-kaartje mag op "review klaar" blijven staan.

Dit is net het herophalen van data waarvan we al zeker zijn dat bestaat. We hebben in een vorige stap die data opgezocht. Nu willen we die nog eens gaan uitlezen uit de DB.

Zie belangrijke comment in Jiraticket

Goeie aanpak, alleen vreemd dat je dan geen variabele gemaakt hebt voor "HoogteFrontBevestigingMidden" en "HoogteFrontBevestigingBoven". Dan staat de berekening er ook slechts 1x i.p.v. 2x (links & rechts)

Ik heb ondertussen 2 NietMee-characteristics toegevoegd aan de CustomView van "M_LBX_Model" in de (voorlopige) "ViewSection3" --> doe jij hetzelfde in je 3D-model?

Zou dit interessant zijn om achter een config setting te steken? Als dit de enige wijzigingen zijn voor L5 kan dit ook wel gewoon wachten met naar productie te gaan tot lijn 5 er is natuurlijk, up to you!

Variabele naam aanpassen naar Encoway ipv dotnet

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)

Ik dacht eerst dat dit niet nodig was, maar heb het achter aan configsettings gestoken .

De werkpost wordt hier niet ingevuld?

Ik zou deze in een config item plaatsen

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

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

ProductApi is nergens gedefinieerd als input parameter, dus deze inject is niet nodig

Naar analogie met "OpvullijstKleur" en "OpvullijstLengte", lijkt me iets beter om "OpvullijstDikte" als varnaam te gebruiken.
cfr. "C_BOX_OpvullijstDikte" , deze is (nog) niet beschikbaar in 3D, maar dat is voorlopig oké voor mij.