FLOWORD5.mac.rou

Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
[VRB591] Automatisch doorsturen picking: enkel de lijnen die nu van toepassing zijn.

Accolade-conventies; meer commentaar; geen dubbel gebruik van naam voor variabele+inputargument; niet-leeg-check.

  1. … 1 more file in changeset.
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:
Merged revision(s) 54518 from deploy2010:

[PaulV] [PieterVR] [VRB591] Automatisch doorsturen voor orderpicking: enkel de orderlijnen die voor de huidige of volgende arbeidsdag zijn.

Dit om orders met over tijd gesplitste leveringen, correct te behandelen.

We waren een NOT vergeten...

........

  1. … 2 more files in changeset.
[ICT196] Falende testen - openstaande variabelen

- New'en van variabele

  • More
  • CR-168
  • finished reviewing
[DEF567] Mails terugname maatwerk komen niet meer toe
[DEF567] Mails terugname maatwerk komen niet meer toe
[DEF567] Mails terugname maatwerk komen niet meer toe

- waarde configitem wordt geinterpreteerd als admin userid's, niet als email adres. -> configitem moet nu door ; gescheiden e-mail adressen bevatten

[UW318] Bedrag opnemen in mailtjes terugname maatwerk.

Manueel getest.