Activity CacheAdminA_trunk2010/DOM/PM/Maatwerk/TBX/impl

9:38 am

Peter Vercoutter

More
Re: CR-2441
replied to Wim Vermeulen

Nagevraagd aan JB en het hoeft niet

11:41 am

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 inspan...

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.

6:23 pm

Wim Vermeulen

More
Re: CR-2441
commented on review

Deze method uitsplitsen is volgens mij echt niet nodig. Het doet vermoeden dat er een ander gedrag is t.o.v. GeefKorttekst(), maar enkel een "input"-waarde is verschillend. Het lijkt me dan ook vee...

Deze method uitsplitsen is volgens mij echt niet nodig. Het doet vermoeden dat er een ander gedrag is t.o.v. GeefKorttekst(), maar enkel een "input"-waarde is verschillend.
Het lijkt me dan ook veel eenvoudiger om de method GeefKorttekst() met een parameter DossierCode As %String uit te breiden, dan kan deze method weg.
N.B.: de method AssertPositiesInKorttekstMet5KaraktersDossierCode() is voor mij wel oké dat deze uitgesplitst is.

5:58 pm

Wim Vermeulen

More
Re: CR-2441
commented on review

"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 ...

"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"

5:36 pm

Wim Vermeulen

More
Re: CR-2441
commented on review

Bodemkleur : idem LadeHoogte : IsValid()

5:34 pm

Wim Vermeulen

More
Re: CR-2441
commented on review

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

5:32 pm

Wim Vermeulen

More
Re: CR-2441
commented on review

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

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

5:28 pm

Wim Vermeulen

More
Re: CR-2441
commented on review

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 omd...

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.

4:25 pm

Peter Vercoutter

More
CR-2344
Closed

4:09 pm

Peter Vercoutter

More
CR-2444
Closed

4:05 pm

Peter Vercoutter

More
Re: CR-2444
replied to Tom Vermeulen

Dit is de hele opzet van de test dat de korttekst wordt berekend. De boom wordt hier genewed omdat die toch niet nodig is voor de berekening van de korttekst

4:02 pm