Activity CacheAdminA_trunk2010/vhUnitTest/DOM

2:35 pm

Joannes Lemmens

More
72841
changed 2 files
root:

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

- Reviewopmerkingen opgepikt

- -0
- +8
./VKP/impl/EindPersonalityBepaler/GeefEindPersonality/Test.cls.xml
- History
- Source
- Diff
./VKP/impl/.../Test.cls.xml
… 1 more file in changeset.

9:47 am

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

10:47 am

Joannes Lemmens

More
72827
changed 2 files
root: CR-2731

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

- Opsplitsing gemaakt voor Blum Polen, waarbij BlumPolen altijd de eindklant is onafhankelijk van de diepte

- unit test aangepast

- -2
- +10
./VKP/impl/EindPersonalityBepaler/GeefEindPersonality/Test.cls.xml
- History
- Source
- Diff
./VKP/impl/.../Test.cls.xml
… 1 more file in changeset.

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

Jelle Baele

More
CR-2706
finished reviewing

11:20 am

Jelle Baele

More
Re: CR-2706
commented on review

Zijn deze lijnen in commentaar nog relevant?

11:15 am

Jelle Baele

More
Re: CR-2706
commented on review

Samengesteld ipv SamenGesteld

11:14 am

Jelle Baele

More
Re: CR-2706
commented on review

'Dotnet' -> Encoway

10:23 am

Peter Vercoutter

More
CR-2725
started review

[ICT-4303] RVG: Aanpassingen Blum release 1/2/2024

9:50 am

Tommy Hebb

More
CR-2724
Closed

9:40 am

Peter Vercoutter

More
CR-2724
finished reviewing

9:38 am

Peter Vercoutter

More
Re: CR-2724
commented on review

Gebruik mss ##class(DOM.common.enu.Personality).HaefeleIT()

9:30 am

Tommy Hebb

More
CR-2724
started review

[ICT-4457] [rvTVE] Aanpassing importfiles Ardis voor klant Snaidero (Häefele IT):

5:07 pm

Tommy Hebb

More
72770
changed 3 files
root: CR-2724

[ICT-4457] [rvTVE] Aanpassing importfiles Ardis voor klant Snaidero (Häefele IT):

- In DataMToelevering blijkt de property EindPersonalityBepaler transient te zijn => dus OpenId i.p.v. new wegens gepersisteerd => dus de EindPersonalityBepaler op juist plek en in juiste geval alsnog newen

- In de global staan er dubbele quotes in de oorsprongketting => die weghalen bij het ophalen van de EindKlantId

- -1
- +1
./PARTIJ/impl/TestLegacyPartijAPI.cls.xml
- History
- Source
- Diff
./PARTIJ/impl/TestLegacyPartijAPI.cls.xml
… 2 more files in changeset.

12:58 pm

Tommy Hebb

More
72768
changed 5 files
root: CR-2724

[ICT-4457] [rvTVE] Aanpassing importfiles Ardis voor klant Snaidero (Häefele IT):

- LegacyPartijAPI uitgebreid met GeefEindklantIdVolgensOorsprongKetting

- UnitTestje voor bovenstaande method

- Voor IsEindklantSnaideroOnderPersonalityHaefeleItalie wordt voor het Front de Rol op "Monster" gezet => Opdat de afdeelzaag gebruikt kan worden i.p.v. de afkortzaag.