Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
[ICT-4676] VHLOG MES M1 - TBX Aantal smalle lades wordt niet correct ingevuld:
[ICT-4676] VHLOG MES M1 - TBX Aantal smalle lades wordt niet correct ingevuld:
Is dit nu exact dezelfde method als die erboven? Alleen WandDikte 25 toegevoegd? Kan je dan niet beter een extra parameter toevoegen aan die method? Optioneel WandDikte As %Decimal = 0 Tip 2 : dez...

Is dit nu exact dezelfde method als die erboven? Alleen WandDikte 25 toegevoegd?
Kan je dan niet beter een extra parameter toevoegen aan die method? Optioneel WandDikte As %Decimal = 0

Tip 2 : deze 2 properties moeten best altijd samengaan :
Set LBXKenmerken.GUIKenmerken.IsBinnenbreedte = $$$False
Set LBXKenmerken.GUIKenmerken.WandDikte = 18 of 25, whatever

Ik vind het zelfs een beetje vreemd dat uw tests slagen, ondanks de afwezigheid van .IsBinnenBreedte = false

Zoals steeds de gewoonte is : %New() van de CBWaardenBepaler verplaatsen naar de constructor. Idem bij de method hierboven GeefCorpusBreedte()

Zoals steeds de gewoonte is :
%New() van de CBWaardenBepaler verplaatsen naar de constructor. Idem bij de method hierboven GeefCorpusBreedte()

[ICT-2102] [rvWV] Verkeerde logica in SmalleLadeBepaler
[ICT-2102] [rvWV] Verkeerde logica in SmalleLadeBepaler
zal inderdaad duidelijker zijn.

zal inderdaad duidelijker zijn.

Wat denk je van een Private method AssertSmalleLade met params (Breedte As %Numeric, VerwachteWaarde As %Boolean) ? Alle code kan daarheen verplaatst worden

Wat denk je van een Private method AssertSmalleLade met params (Breedte As %Numeric, VerwachteWaarde As %Boolean) ?
Alle code kan daarheen verplaatst worden

APPS.PM.Maatwerk - was toegevoegd, maar onnodig, enkel DOM laag heeft deze nodig. Dus wijzigingen in APPS laag zijn ongedaan gemaakt. Staan dus wel in deze review.

APPS.PM.Maatwerk - was toegevoegd, maar onnodig, enkel DOM laag heeft deze nodig.
Dus wijzigingen in APPS laag zijn ongedaan gemaakt. Staan dus wel in deze review.

ICT-816: Vhintra Configurator - gebruiker attent maken op ingave van een smalle lade. (met mogelijkse meerkost)
ICT-816: Vhintra Configurator - gebruiker attent maken op ingave van een smalle lade. (met mogelijkse meerkost)