Deploy20190723xDkgPakketRefAlsBarcodeOpKlantEtiket.cls.xml

Maar wel belangrijker !!!
Ik ben absoluut voorstander van SIMPELE uitdrukkingen :

 
(Var '= "") ----> beter : $L(Var)   of nog beter : $$$HasLength(Var)

'($L(Var))  ----> VEEL BETER :  (Var = "")     dus hier : $G(C("DOSNR")) = ""    Dat zijn 2 redeneringsstappen minder !!

Tis een beetje muggeziften over convenience ... maar ik zou deze expressie in 2 aparte IFen steken.
Dan heb je
a) duidelijker afscheiding tussen "project-setting" en de echte business-rule conditie
b) minder haakjes, en dus minder kans op verwarring/fouten. Is ook gemakkelijker voor de reviewer

En als we dan toch aan t muggeziften zijn :-P dan kunt ge beter eerst de voorwaarde zetten van (KLNr = DKG etc.) en dan pas de IsActief-setting controleren, want nu wordt die setting van DKG voor alle klanten gecheckt.
Sebiet is mijn zift helemaal versleten ;-D

OLDSCHOOL Unittest

MaakData en VerwijderData zouden in de OnBefore/OnAfterOneTest moeten staan.
In dit geval moet je dan de hele UT gaan refactoren vrees ik. m.a.w. is t nogal moeilijk om boyscout hier toe te passen.
Daarom zal ik het door de vingers zien.
Maar het blijft verschrikkelijk om te weten dat dit soort testen in ons systeem zitten :-O