MVXLangtekstCreator

Nagevraagd aan JB en het hoeft niet

Sifonlade D met M-rughoogte : dit blijft ons nog steeds achtervolgen. Voorlopig moet je hier niks aan veranderen.
Eenmaal op het nieuwe platform van Encoway CPQ-studio, gaan we wel een extra inspanning doen om dit principe in al onze lade-systemen te uniformiseren.
De term "Afwijkende" hoogte moet dan verdwijnen, want is te verwarrend.

"Vhoss zonder onderdelen" was vroeger code "A", maar sinds eCon bestaat deze niet meer.
Wim Van Daele had een verzoek, namelijk dat als Verpakking B en alle Nietmeeleverens = "true", dan toch code "A" in korttekst zetten.
Zo kunnen ze meteen aan de korttekst zien dat het om "Vhoss zonder onderdelen" gaat. Eventueel te verifieren bij JohanB.
TIP: indien nodig, kijk eens rond in caché, er bestaat misschien al iets in de aard van "TBX alle nietmeeleverens = true"

Bodemkleur : idem LadeHoogte : IsValid()

LadeKleur "OG" nog toevoegen. Heb ik ondertussen via mail gestuurd :-p

Waarom niet gewoonweg : if (enum.LadeHoogte.IsValidDT()) then ladehoogte else " "
Dan kan heel de If-elseif-else weg.

Draagkracht 50kg zou ik eventueel nog toevoegen --> GeleiderCode = "9"
En de fallback naar "I" was van heel lang geleden, zijnde "Intrabox", lijkt me niet opportuun. Misschien "0" of "?" zetten omdat dit iets beter zal opvallen, denk.

Deze set kunt ge ineens in uw dimmeke doen hé, weer een lijn minder

't zelfste
Als ge 't doet omdat het meermaals gebruikt wordt, dan is een lokale variabele voldoende, toch?

't zelfste

't zelfste

't Zelfde

Waarom een simple stringske afzonderen naar een private method als ge bovendien 2 lijnen verder wel ineens de string meegeeft?

Vervolgend op mijn opmerking bij de bijhorende logica in de klasse zelf... dit is een false positive en klopt dus niet!!
De énige reden waarom deze test slaagt is omdat de volgende positie in de korttekst (positie 21) ALTIJD een spatie is (zie volgende UnitTest). Je test hier dus niet de lege spatie voor het VerpakkingType, want volgens de logica in de klasse zelf zal die "" zijn i.g.v. V1. De gevolgen zijn dat de korttekst 1 karakter korter is, dat deze specifieke assert onterecht slaagt voor positie 20, omdat positie 21 1e is opgeschoven naar voren en je die eigenlijk hier test en dat alle posities daarna volgens de testen wel slagen, maar in werkelijkheid allemaal op de verkeerde positie zullen staan in gegenereerde kortteksten!

Weet niet of het effectief zo is dat de kleuren buiten en binnen verschillen, maar het viel mij op dat er hierboven bij BuitenKleurCode SW (Zijdewit) bijgekomen is, terwijl hier WI en S bijgekomen is. Misschien klopt het wel hé, maar just in case vermeld ik het maar.

Ben je zeker dat het dan leeg moet zijn? M.a.w. GEEN karakter op die plek in de korttekst? Default = " " (lege spatie), die ga je dus i.g.v. V1 overschrijven met leeg, wat de korttekst dus 1 karakter korter maakt!

JB wil het op deze manier