EWORDS6.mac.rou

Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Indien kritisch review, dan ontbreken nog enkele UT's. Maar gezien de korte tijd dat deze code nog zal bestaan, is het huidige niveau van code-coverage voldoende. Review O.K.

Indien kritisch review, dan ontbreken nog enkele UT's. Maar gezien de korte tijd dat deze code nog zal bestaan, is het huidige niveau van code-coverage voldoende.
Review O.K.

[ICT-2539] melding bij doorsturen picking orderlijn op leverdatum later dan vandaag
[ICT-2539] melding bij doorsturen picking orderlijn op leverdatum later dan vandaag
Is manueel gebeurd http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif

Is manueel gebeurd

moet er niet nog een deployklaske bij die de code van de zTryout gaat uitvoeren? Of gaat ge dat manueel doen?

moet er niet nog een deployklaske bij die de code van de zTryout gaat uitvoeren? Of gaat ge dat manueel doen?

leuk gevonden als reminder http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif

leuk gevonden als reminder

[ICT-1932] BIDI: KlantSync: (VHIP900): doorsturen picking automatisch of manueel:
[ICT-1932] BIDI: KlantSync: (VHIP900): doorsturen picking automatisch of manueel:
[DEF695] MAG: Doorsturen picking: Cubics en ORGALUX als 1 groep beschouwen:
[DEF695] MAG: Doorsturen picking: Cubics en ORGALUX als 1 groep beschouwen:
[VRB602] BIDI: doorsturen picking: rembours: automatisch doorsturen mag toch niet splitsen:
[VRB602] BIDI: doorsturen picking: rembours: automatisch doorsturen mag toch niet splitsen:
Oude code ... zeer verwarrend, want input-parameter wordt ook gebruikt als locale variable. Beter : input aanpassen naar GetLeverDagen(OrderID,GevraagdOLNr) en toevoegen New ORDNr

Oude code ... zeer verwarrend, want input-parameter wordt ook gebruikt als locale variable.
Beter : input aanpassen naar

GetLeverDagen(OrderID,GevraagdOLNr) 

en toevoegen

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

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.

"Not is leeg" --> dubbele negatie ... Liefst vervangen door: $Length() of: $$$HasLength() Voor die laatste moet je dan wel #include vhLib.Macro toevoegen.

"Not is leeg" --> dubbele negatie ...
Liefst vervangen door:

$Length()   of:   $$$HasLength()


Voor die laatste moet je dan wel #include vhLib.Macro toevoegen.

[VRB591] Merged revision(s) 54518 from deploy2010:
[VRB591] Merged revision(s) 54518 from deploy2010:
[UST3587] VHIP-MAG: Doorsturen picking: splits ORGALUX: case van meerdere orders tegelijk doorsturen...
[UST3587] VHIP-MAG: Doorsturen picking: splits ORGALUX: case van meerdere orders tegelijk doorsturen...
[UST3470] VHIP-MAG: splits orderpicking ORGALUX en niet-ORGALUX
[UST3470] VHIP-MAG: splits orderpicking ORGALUX en niet-ORGALUX
[UST3188] Commentaar bij vooronderzoek.
[UST3188] Commentaar bij vooronderzoek.