This is a list of all comments for CR-1522. Review Summary: No summary ---------------------------------------- File: EWORDS6.mac.rou Revision Comment by Wim Vermeulen on 08 January 2019, 14:15 http://subversion02:8060/cru/CR-1522#c2114 Onze gewoonte : "open" accolade "{" schrijven we steeds op dezelfde lijn, niet de lijn eronder. Niet nodig om deze hier aan te passen, tis maar gewoon ter info. Revision Comment by Wim Vermeulen on 08 January 2019, 14:31 http://subversion02:8060/cru/CR-1522#c2116 Oude code ... zeer verwarrend, want input-parameter wordt ook gebruikt als locale variable. Beter : input aanpassen naar {code}GetLeverDagen(OrderID,GevraagdOLNr) {code} en toevoegen {code} New ORDNr {code} ---------------------------------------- File: FLOWORD5.mac.rou Revision Comment by Wim Vermeulen on 08 January 2019, 14:06 http://subversion02:8060/cru/CR-1522#c2113 "Not is leeg" --> dubbele negatie ... Liefst vervangen door: {code}$Length() of: $$$HasLength(){code} Voor die laatste moet je dan wel #include vhLib.Macro toevoegen. --- ID: CR-1522 http://subversion02:8060/cru/CR-1522 Title: [VRB591] Merged revision(s) 54518 from deploy2010: Statement of Objectives: State: Closed Summary: Author: Pieter Van Raemdonck Moderator: Pieter Van Raemdonck Reviewers: (0 active, 1 completed*) Wim Vermeulen (*)