CacheAdminA_trunk2010

Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
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.
[ICT739] VKP: Regios: nieuwe regio 53 en aanpassing omschrijving 52 (architecten NL opgesplitst).
[ICT739] VKP: Regios: nieuwe regio 53 en aanpassing omschrijving 52 (architecten NL opgesplitst).
[ICT362] [UST4255] TAXSY - extra mogelijkheden om Talsee binnen te halen:
[ICT362] [UST4255] TAXSY - extra mogelijkheden om Talsee binnen te halen:
[UW782] LBX: BevestigingBodemFront bij Binnenlade ALTIJD aanwezig
[UW782] LBX: BevestigingBodemFront bij Binnenlade ALTIJD aanwezig
hmmm, tis uitgeschakeld (in commentaar) en tis wel handig wanneer er een nieuwe structuur bijkomt die nog niet verwerkt/getoond wordt. en tis in oude code :-D

hmmm,
tis uitgeschakeld (in commentaar) en tis wel handig wanneer er een nieuwe structuur bijkomt die nog niet verwerkt/getoond wordt.
en tis in oude code :-D

Moet dat wlipke erin blijven staan? http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif

Moet dat wlipke erin blijven staan?

Om te reviewen kan je best alleen de tweede commit bekijken, want de eerste svn-commit is alleen herschrijven van de bestaande logica, dus geen functionele wijzigingen.

Om te reviewen kan je best alleen de tweede commit bekijken, want de eerste svn-commit is alleen herschrijven van de bestaande logica, dus geen functionele wijzigingen.

[UW781] ICT: Kenmerken UitsparingData tonen in Scherm E in Admin
[UW781] ICT: Kenmerken UitsparingData tonen in Scherm E in Admin
[UW780] VH4: Alpnach SFS : Korting 35.7 -> 36.0 %
[UW780] VH4: Alpnach SFS : Korting 35.7 -> 36.0 %
#dim ProductServiceMock As vhTest.Mock.WSimpl.AX.CalculatedProduct.ProductService = ##class(vhTest.Mock.WSimpl.AX.CalculatedProduct.ProductService).%New() Do ProductServiceMock.VerwachtMethodCall("...

#dim ProductServiceMock As vhTest.Mock.WSimpl.AX.CalculatedProduct.ProductService = ##class(vhTest.Mock.WSimpl.AX.CalculatedProduct.ProductService).%New()
Do ProductServiceMock.VerwachtMethodCall("GeefProduct", ..#RandomProductId, Request.ItemId).ExactAantalKeer(0)
Set ..ProductService = ProductServiceMock

Dit toevoegen, dan kan je de Do ..Verifieer() oproepen

Deze property wordt niet meer gebruikt in deze klasse en mag dus weg (is verplaatst naar productUpdater)

Deze property wordt niet meer gebruikt in deze klasse en mag dus weg (is verplaatst naar productUpdater)