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 => deze test is overbodig geworden

[ICT851] TAXFP1: Activering intern:
[ICT851] TAXFP1: Activering intern:
[ICT851] TAXFP1: Activering intern:

- WinkelkarControleur => Melding voor MinimaleBestelhoeveelheid is nu LadeVariant-specifiek met gebruik van placeholder in vertalingen

- Deployklaskes met nieuwe vertalingen => eentje voor wanneer het niet uitmaakt of VerpakkingTypes worden gecombineerd en eentje voor wanneer wel

- OPGELET DEPLOYKLASKES => Voorlopig zijn enkel N en NB al OK => andere talen in afwachting van correcte vertaling => oude overgenomen

- Verdere refactor Testcase en uitbreiding met TestControleerWinkelkarTweeLadeVariantenTweeVerpakkingTypes en testjes voor FlatpackV1

    • -6
    • +5
    ./TestTaorboxEnLegraboxZelfdeVerpakking.cls.xml
  1. … 5 more files in changeset.
[ICT851] TAXFP1: Activering intern:

- Voorafgaandelijke refactor => GeefTBXKenmerken op Base gezet

- Voorafgaandelijke refactor => GeefKenmerken op Base gezet

- Voorafgaandelijke refactor => callers van VoegToeLadenInWinkelkar aangepast => inclusief LadeVariant

    • -49
    • +10
    ./TestTaorboxEnLegraboxZelfdeVerpakking.cls.xml
  1. … 2 more files in changeset.
[ICT851] TAXFP1: Activering intern:

- Voorafgaandelijke refactor => GeefTAORKenmerken op Base gezet

- Voorafgaandelijke refactor => Consistent maken van callers van VoegToeLadenInWinkelkar

    • -5
    • +1
    ./TestTaorboxEnLegraboxZelfdeVerpakking.cls.xml
  1. … 2 more files in changeset.
[ICT851] TAXFP1: Activering intern:

- Voorafgaandelijke refactor => GeefLBXKenmerken op Base gezet

  1. … 1 more file in changeset.
[UW718] VH4: bestelling met te weinig proboxen (bij Sfs)

- UT uitgebreid met mi van TBX (V1) en LBX Probox (quick-and-dirty copy-paste gedaan)

    • -0
    • +74
    ./TestTaorboxEnLegraboxZelfdeVerpakking.cls.xml
  • More
  • CR-239
  • finished reviewing
[BUS618] Probox vanaf 4 lades
[BUS618] Probox vanaf 4 lades
[BUS618] Probox vanaf 4 lades

- 5->4 en testjes

    • -4
    • +4
    ./TestTaorboxEnLegraboxZelfdeVerpakking.cls.xml
  1. … 4 more files in changeset.
  • More
  • CR-95
  • finished reviewing
  • 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