KlantInstellingenServiceSFS.cls.xml

Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
[ICT-1790] TAX e-con Flatpack V1 : min.best.hoev. op 1st zetten ipv 4st:
[ICT-1790] TAX e-con Flatpack V1 : min.best.hoev. op 1st zetten ipv 4st:
[ICT-1790] TAX e-con Flatpack V1 : min.best.hoev. op 1st zetten ipv 4st:

- FlatpackV1Verpakking mag voortaan per 1 stuk besteld worden door iedereen

  1. … 3 more files in changeset.
  • More
  • CR-99
  • finished reviewing
Testen zouden eigenlijk het aantal moeten kunnen uitmocken zodat de logica van de code kan getest worden onafhankelijk van het aantal. Anders zal er bij iedere configuratie wijzigingen test-resulta...

Testen zouden eigenlijk het aantal moeten kunnen uitmocken zodat de logica van de code kan getest worden onafhankelijk van het aantal. Anders zal er bij iedere configuratie wijzigingen test-resultaten wijzigen

geen Initialen in deploy stories http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/biggrin.gif

geen Initialen in deploy stories

Zelfde opmerking zoals bij de MagKlantVerpakkingTypeBestellen

Zelfde opmerking zoals bij de MagKlantVerpakkingTypeBestellen

Beter om op de KlantInstellingenService een extra method aan te maken die Lade_Familie binnenkrijgt , de klantid en het verpakkingtype zodoende we niet per Lade_Familie extra code moeten toevoegen

Beter om op de KlantInstellingenService een extra method aan te maken die Lade_Familie binnenkrijgt , de klantid en het verpakkingtype zodoende we niet per Lade_Familie extra code moeten toevoegen

Het lijkt mij beter om deze code te herwerken zodat er met een MultiDim / Array kan gewerkt worden waarbij de Lade_Familie_Variant als key gebruikt worden.

Het lijkt mij beter om deze code te herwerken zodat er met een MultiDim / Array kan gewerkt worden waarbij de Lade_Familie_Variant als key gebruikt worden.

Waarom is er voor legrabox een uitzondering door niet te controleren op MagKlantVerpakkingTypeBestellen ??? ps : ik zie ook wel duidelijk dat die boolean hardcoded op $$$true wordt gezet

Waarom is er voor legrabox een uitzondering door niet te controleren op MagKlantVerpakkingTypeBestellen ???
ps : ik zie ook wel duidelijk dat die boolean hardcoded op $$$true wordt gezet

Niet nodig om een onliner af te zonderen naar een aparte method

Niet nodig om een onliner af te zonderen naar een aparte method

Beter deze logica afzonderen naar een submethode kwestie om het toch wat leesbaarder te krijgen.

Beter deze logica afzonderen naar een submethode kwestie om het toch wat leesbaarder te krijgen.

#dim naar het juiste type

#dim naar het juiste type

  • More
  • CR-99
  • started review
[UST2170] Vh4_SVC: TBX Probox (e.a.) min. bestelhoeveelheid
[UST2170] Vh4_SVC: TBX Probox (e.a.) min. bestelhoeveelheid
[UST2170] Vh4_SVC: TBX Probox (e.a.) min. bestelhoeveelheid

- Controle op procom toegevoegd. (lange method nog te refactoren.)

  1. … 2 more files in changeset.
[UST2170] Vh4_SVC: TBX Probox (e.a.) min. bestelhoeveelheid

- VerpakkingType meegeven bij bepalen minimumhoeveelheid

  1. … 4 more files in changeset.
[UST2170] Vh4_SVC: TBX Probox (e.a.) min. bestelhoeveelheid

- hernoemen method GeefMinimimAantalProductLijnenVoorProbox -> GeefMinimaleBestelhoeveelheid

  1. … 5 more files in changeset.
[BUS551] SFS Probox vanaf 4 laden (TBX+LBX+TAX) - KlantInstellingenService +UT

!!! OPGELET !!! op 01-03-2016 in productie brengen!

  1. … 4 more files in changeset.