Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
3 keer is scheepsrecht http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/wink.gif

3 keer is scheepsrecht

twijfelgevalleke: zondert ge de gemeenschappelijke code af in private method of niet... Ja: want het is een beperkte exacte copy Nee: 't staat er maar twee keer... your call

twijfelgevalleke: zondert ge de gemeenschappelijke code af in private method of niet...
Ja: want het is een beperkte exacte copy
Nee: 't staat er maar twee keer...

your call

best geen vervolglijnen op dezelfde indent laten beginnen als de beginlijn (hier de if). volgens mij is het ook properder om zo'n kloeft in een private methodje af te zonderen. dat bevordert de lee...

best geen vervolglijnen op dezelfde indent laten beginnen als de beginlijn (hier de if).
volgens mij is het ook properder om zo'n kloeft in een private methodje af te zonderen. dat bevordert de leesbaarheid enorm

[UST3682] VHIP903: PickingCompleet: dubbels teveel verwijderd:
[UST3682] VHIP903: PickingCompleet: dubbels teveel verwijderd:
  • More
  • CR-953
  • finished reviewing
[UST3606] VHIP903 InventSync: Uitfilteren ongewenste producten:
[UST3606] VHIP903 InventSync: Uitfilteren ongewenste producten:
  • More
  • CR-951
  • finished reviewing
[UST3594] VHIP903: InventSync: error "Geen InventSyncs gemaakt" wegwerken:
[UST3594] VHIP903: InventSync: error "Geen InventSyncs gemaakt" wegwerken:
  • More
  • CR-937
  • finished reviewing
[UST3576] VHIP903: ProdRAF en PickingCompleet: niet dubbel sturen van componenten:
[UST3576] VHIP903: ProdRAF en PickingCompleet: niet dubbel sturen van componenten:
  • More
  • CR-914
  • finished reviewing
[UST3553] VHIP903 VoorraadSync - Converter: Aanpassing logging: geen ID's in actie:
[UST3553] VHIP903 VoorraadSync - Converter: Aanpassing logging: geen ID's in actie:
  • More
  • CR-897
  • finished reviewing
Code aangepast => Verantwoordelijkheid verhuisd naar gepaste klasse (InventSync => MaatwerkStockVerschilAfhandelaar) en gedrag passender gemaakt (Indien geen stockverschillen => standaard mailtje d...

Code aangepast => Verantwoordelijkheid verhuisd naar gepaste klasse (InventSync => MaatwerkStockVerschilAfhandelaar) en gedrag passender gemaakt (Indien geen stockverschillen => standaard mailtje dat dit meldt, ipv een lege lijst te mailen met dezelfde tekst als mochten er wel stockverschillen zijn). UnitTests uitgebreid.

Dan is dat nog steeds standaard gedrag, want dan zijn er geen AxConfigIds en blijft die variabele ook netjes leeg. Uiteindelijk doet die klasse, buiten het mailen, niets anders dan de data gemaakt ...

Dan is dat nog steeds standaard gedrag, want dan zijn er geen AxConfigIds en blijft die variabele ook netjes leeg. Uiteindelijk doet die klasse, buiten het mailen, niets anders dan de data gemaakt door andere klassen binnenhalen en verzamelen, er wordt verder niets mee gedaan qua logica.

Heet nu "AxConfigIds"

Heet nu "AxConfigIds"

"Ts" is de universele afkorting voor "Tussen" http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/wink.gif

"Ts" is de universele afkorting voor "Tussen"

Testje geschreven.

Testje geschreven.

Done! http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif

Done!

Nope. Zie uitleg bij lijn 60 http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif

Nope. Zie uitleg bij lijn 60

Heb de naamgeving van die parameter wat aangepast, zodat het duidelijk is voor iedereen.

Heb de naamgeving van die parameter wat aangepast, zodat het duidelijk is voor iedereen.

Nope! http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif 'Deze' QtyVH is die zoals die berekent (som) en doorgegeven is door de SyncAllMaatwerk, en hangt d...

Nope! 'Deze' QtyVH is die zoals die berekent (som) en doorgegeven is door de SyncAllMaatwerk, en hangt dus volledig af van wat er op de OrderLijnen staat (in specifieke gevallen, gedefinieerd in de IteratorFilter van de dynamische Iterator gebruikt door de SyncAllMaatwerk). Het niet leeg zijn van deze parameter zorgt er tevens voor dat de InventSyncCreator weet dat het over een maatwerkproduct gaat dat via de SyncAllMaatwerk gesynct wordt.

standaardgedrag is leuk, maar wat met het afwijkend gedrag: ConfigIdIterator leeg,

standaardgedrag is leuk, maar wat met het afwijkend gedrag: ConfigIdIterator leeg,