This is a list of all comments for CR-2731.

Review Summary: No summary


----------------------------------------
File: DOM/VKP/impl/EindPersonalityBepaler.cls.xml
Revision Comment by Tommy Hebb on 10 April 2024, 14:24 defect
http://subversion02:8060/cru/CR-2731#c4334

  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.

----------------------------------------
File: vhUnitTest/DOM/VKP/impl/EindPersonalityBepaler/GeefEindPersonality/Test.cls.xml
Revision Comment by Tommy Hebb on 10 April 2024, 14:37 defect
http://subversion02:8060/cru/CR-2731#c4335

  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.


---

ID: CR-2731 http://subversion02:8060/cru/CR-2731

Title: [ICT-4516] EindPersonalityBepaler klopt niet voor diepte groter dan twee
Statement of Objectives:
ICT-4516

State: Closed

Summary: 

Author: Joannes Lemmens
Moderator: Joannes Lemmens
Reviewers: (0 active, 1 completed*)
     Tommy Hebb   (*)