CacheAdminA_trunk2010

Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
[UST4278] PM: Maatwerk: BOMBOL VHIP481: Ook TAX updaten:
[UST4278] PM: Maatwerk: BOMBOL VHIP481: Ook TAX updaten:
Die volgen het default gedrag (geen exceptie smijten). De LegacyConverter wordt o.a.aangesproken door de Synchroniseer en HerberekenMaatwerk, dus die mag ni crashen.

Die volgen het default gedrag (geen exceptie smijten).
De LegacyConverter wordt o.a.aangesproken door de Synchroniseer en HerberekenMaatwerk, dus die mag ni crashen.

[UST4160] VHIP2A+: Econ: TBX: caché-kant kenmerknamen consistent maken.
[UST4160] VHIP2A+: Econ: TBX: caché-kant kenmerknamen consistent maken.
Wat met APPS.EDI.common.impl.LegacyConverter en bij uitbereiding de klasses die hier van gebruik maken? ECONimpl.PM.Maatwerk.KenmerkenConverter.AppsNaarEcon.Tbx ECONimpl.PM.Maatwerk.KenmerkenConve...

Wat met
APPS.EDI.common.impl.LegacyConverter

en bij uitbereiding de klasses die hier van gebruik maken?
ECONimpl.PM.Maatwerk.KenmerkenConverter.AppsNaarEcon.Tbx
ECONimpl.PM.Maatwerk.KenmerkenConverter.EconNaarApps.Tbx

Default gedrag : de TBXLadeCodeBepaler smijt geen exceptie; Via constructor-flag = True zal deze wel exceptie smijten, indien geen geldige ladecode gevonden.

Default gedrag : de TBXLadeCodeBepaler smijt geen exceptie;
Via constructor-flag = True zal deze wel exceptie smijten, indien geen geldige ladecode gevonden.

[BUS792] TBX: Uitschakelen LadeCodes (Sifon type1+2 en burolade)
[BUS792] TBX: Uitschakelen LadeCodes (Sifon type1+2 en burolade)
[ICT742] [Kanban UST4277] TAXFP1: Flatpack Probox => Flatpack V1:
[ICT742] [Kanban UST4277] TAXFP1: Flatpack Probox => Flatpack V1:
[ICT218] [Kanban ICT490] - TAX: FPV1: Opkuis code flatpack V1:
[ICT218] [Kanban ICT490] - TAX: FPV1: Opkuis code flatpack V1:
Maar wel belangrijker !!! Ik ben absoluut voorstander van SIMPELE uitdrukkingen : (Var '= "") ----> beter : $L(Var) of nog beter : $$$HasLength(Var) '($L(Var)) ----> VEEL BETER : (Var = "") ...

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...

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 http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/sad.gif http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/sad....

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

Is eigenlijk niet nodig om 4x hetzelfde te testen, namelijk dat bij DubbeleSifonlade de kosten "x 3" i.p.v. "x 2" is. Graag effe samen bespreken.

Is eigenlijk niet nodig om 4x hetzelfde te testen, namelijk dat bij DubbeleSifonlade de kosten "x 3" i.p.v. "x 2" is.
Graag effe samen bespreken.

Beter :  Quit ##super(FormatterInput,IngegevenKenmerken) _ ..GeefSpecialeLadeVoorHaluxTekst(IngegevenKenmerken) en de method dan laten staan op de basisklasse, en dan moet ge de method in de LBX-...

Beter :

 Quit ##super(FormatterInput,IngegevenKenmerken) _ ..GeefSpecialeLadeVoorHaluxTekst(IngegevenKenmerken)


en de method dan laten staan op de basisklasse,
en dan moet ge de method in de LBX-klasse niet overriden.

Beter : Do ##super(IngegevenKenmerkenTypeAPI)

Beter : Do ##super(IngegevenKenmerkenTypeAPI)

deze lijn wordt dan  Set C("KLREFOMS")=$G(KlRefOms)

deze lijn wordt dan

 Set C("KLREFOMS")=$G(KlRefOms) 
Deze lijn mag dan integraal weg.

Deze lijn mag dan integraal weg.

Steek ^PAKKET("D",PAKNr) in ne variabele, zeg maar Rec en gebruik $$$IfLength( ) vanuit de include vhLib.Macro Dan wordt de code  . Set:(C("PAKKETREFERENTIE") '= $P(Rec,D,2)) KlRef = KlRef_$$$IfLe...

Steek ^PAKKET("D",PAKNr) in ne variabele, zeg maar Rec en gebruik $$$IfLength( ) vanuit de include vhLib.Macro
Dan wordt de code

 . Set:(C("PAKKETREFERENTIE") '= $P(Rec,D,2)) KlRef = KlRef_$$$IfLength($P(Rec,D,2)," & ","")   // Nog een overeenkomstig pakket gevonden, maar indien zelfde als op orderlijn, dan niet dubbel zetten.

 . Set:$L(Rec,D,4)) KlRefOms = KlRefOms_$$$IfLength($P(Rec,D,4)," & ","")    // Sommige pakketten hebben niet enkel een naam, maar ook een omschrijving.


Dan moet je geen $E( ... , 4, 80) meer doen, want de " & " wordt nu conditioneel toegevoegd als prefix.
Leve de macro's :-D

[UW778] HX: Klantetiket: pakketref zelfs niet tekstueel bij Aventos Servo Drive set (blijkt omdat...
[UW778] HX: Klantetiket: pakketref zelfs niet tekstueel bij Aventos Servo Drive set (blijkt omdat...
[BUS793] Config item + unit test voor DKG pakketref als barcode.
[BUS793] Config item + unit test voor DKG pakketref als barcode.