Activity CacheAdminA_trunk2010/DOM

2:17 pm

Tommy Hebb

More
Re: CR-2731
commented on review

Deze 2 testen zien er goed uit, maar ik mis nog een test waarbij de ketting langer dan 2 is, maar de voorlaatste NIET BlumPolen is.

2:35 pm

Tommy Hebb

More
Re: CR-2731
commented on review

Hey Joannes. Volgens mij kan dit allemaal veel duidelijk, korter en leesbaarder... Als het voorlaatste item in de list BlumPolen is, dan maakt het eigenlijk totaal niet meer uit hoelang de list is...

Hey Joannes. Volgens mij kan dit allemaal veel duidelijk, korter en leesbaarder...

Als het voorlaatste item in de list BlumPolen is, dan maakt het eigenlijk totaal niet meer uit hoelang de list is, dan gaat het sowieso over Personality BlumPolen. BlumPolen zal ook altijd als voorlaatste in de ketting staan, want wij gaan nooit BlumPolen bedienen via iemand anders. Dus:

Als de Oorspronketting niet leeg is, dan de boolean IsPersonalityBlumPolen zetten met ($Piece($List(OorsprongKetting, $ListLength(OorsprongKetting) - 1),",", 1) = PersonalityIdBlumPolen)
Dan: If (IsPersonalityBlumPolen) { Set PersonalityID = PersonalityIdBlumPolen (er is geen nood aan om die ID nog eens terug op te halen uit die list, je weet het hier eigenlijk al wat de ID is, dus gelijk de enum gebruiken is ok).
Else: Die mag blijven zoals hij is, want in alle andere gevallen is het de personality die vooraan in de ketting zit, ongeacht hoe lang die is.

10:48 am

Joannes Lemmens

More
CR-2731
started review

[ICT-4516] EindPersonalityBepaler klopt niet voor diepte groter dan twee

1:21 pm

Peter Vercoutter

More
CR-2730
started review

[ICT-4503] [rvJBA] Detailpapier afdekkappen zonder logo opdruk "zonder logo"

11:23 am

Jelle Baele

More
Re: CR-2706
commented on review

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)

11:21 am