Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Dit is niet oke. ProductData is iets anders dan de opgeslagen vhConfigs. ProductData gaat over KPR, PRBS, ... en al die bazaar. Die moet wel degelijk nog worden verwijderd na iedere test.

Dit is niet oke. ProductData is iets anders dan de opgeslagen vhConfigs. ProductData gaat over KPR, PRBS, ... en al die bazaar. Die moet wel degelijk nog worden verwijderd na iedere test.

[ICT-4441] [rvPVR] AansturingActies via BOMBOL Encoway:
[ICT-4441] [rvPVR] AansturingActies via BOMBOL Encoway:
$J zou ik vervangen door '$Job'?

$J zou ik vervangen door '$Job'?

Ergens geeft dit nu wel een verkeerd beeld van deze method; ik zou ervan uitgaan dat het productid enkel wordt bepaald, maar als je dan in de method gaat kijken zie je dat hier ook de vhConfig word...

Ergens geeft dit nu wel een verkeerd beeld van deze method; ik zou ervan uitgaan dat het productid enkel wordt bepaald, maar als je dan in de method gaat kijken zie je dat hier ook de vhConfig wordt opgeslagen

Converter in constructor initialiseren

Converter in constructor initialiseren

Ik zie dit nu pas, maar misschien kan dit in een 'constante' variabale om wat meer context te geven? Iets als 'DefaultCidAX' of iets dergelijks?

Ik zie dit nu pas, maar misschien kan dit in een 'constante' variabale om wat meer context te geven? Iets als 'DefaultCidAX' of iets dergelijks?

[ICT-4440] [rvJBA] VhConfig persisteren in caché na ordersync AX
[ICT-4440] [rvJBA] VhConfig persisteren in caché na ordersync AX
[ICT-4567] [rvJBA] RVG Wijzigingen april release
[ICT-4567] [rvJBA] RVG Wijzigingen april release
[ICT-4248] RVG Toelaten voor MP SFS en OPO op Test
[ICT-4248] RVG Toelaten voor MP SFS en OPO op Test
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.

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.

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.
[ICT-4516] EindPersonalityBepaler klopt niet voor diepte groter dan twee
[ICT-4516] EindPersonalityBepaler klopt niet voor diepte groter dan twee
[ICT-4503] [rvJBA] Detailpapier afdekkappen zonder logo opdruk "zonder logo"
[ICT-4503] [rvJBA] Detailpapier afdekkappen zonder logo opdruk "zonder logo"
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)

Zijn deze lijnen in commentaar nog relevant?

Zijn deze lijnen in commentaar nog relevant?