CacheAdminA_trunk2010/APPS/EC/Winkelkar/impl

VerpakkingControleurWinkelkarVisitor.cls.xml

Checkout Tools

Line History

Line Count Graph
Line Count Graph

Stats

Commits this week: 0
Total Committers: 3
Last Commit: 23 Jul 18
Commits this week: 0
Total Lines of Code (LoC): -7
Net change in LoC this week: 0

Commit Activity

Commits By Day In Week
52 week commits volume
Commits By Day In Week
Commits by day
Commits By Day In Week
Commits by hour
Commits By Hour In Day
Commit calendar
 

Most active committers (90 days)

loading
loading
  • last updated a few seconds ago
All Commits Reviews
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Tuesday 31 Jul 2018
12:13 pm

Tommy Hebb

12:04 pm

Kurt Callebaut

10:14 am

Tommy Hebb

Yup :-s => refactored!

Yup :-s => refactored!

9:36 am

Kurt Callebaut

Duplicaat met VerpakkingTaor ?

Duplicaat met VerpakkingTaor ?

Monday 30 Jul 2018
12:01 pm

Tommy Hebb

[VRB532] Vh4: Verpakkingtypes gemengd toelaten in een bestelling:
[VRB532] Vh4: Verpakkingtypes gemengd toelaten in een bestelling:
Monday 23 Jul 2018
12:04 pm

Tommy Hebb

[VRB532] Vh4: Verpakkingtypes gemengd toelaten in een bestelling:

- Opkuis: private method die nergens gebruikt wordt en tevens een call doet van een onbestaande method

Options
    • -7
    • +0
    ./VerpakkingControleurWinkelkarVisitor.cls.xml
Friday 28 Apr 2017
10:20 am

KurtC

[DEF577] Controle op bestelde winkelkar geeft onterechte foutmelding ivm verschillende verpakkingen. Ondanks alle configuraties V1 zijn ingegeven.

- Verpakking en verpakkingtype op tandembox zijn nog niet op 1 lijn gebracht met legra en taor.

In de winkelkarcontroleur visitor werd verpakking "S" bijgehouden als type en voor legra V1 en taor V1 waardoor een melding werd gegeven dat verschillende verpakkingen niet te combineren zijn.

-> Nu wordt de conversie uitgevoerd voordat de aantallen en verpakkingen bijgehouden worden.

Options
    • -3
    • +1
    ./VerpakkingControleurWinkelkarVisitor.cls.xml
  2. … 1 more file in changeset.
Wednesday 01 Mar 2017
8:04 am

Kurt Callebaut

Monday 27 Feb 2017
1:07 pm

Jo Claes

  • More
  • CR-99
  • finished reviewing
Thursday 23 Feb 2017
2:34 pm

Jo Claes

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

2:31 pm

Jo Claes

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

geen Initialen in deploy stories

2:30 pm

Jo Claes

Zelfde opmerking zoals bij de MagKlantVerpakkingTypeBestellen

Zelfde opmerking zoals bij de MagKlantVerpakkingTypeBestellen

2:30 pm

Jo Claes

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

2:21 pm

Jo Claes

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.

2:15 pm

Jo Claes

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

2:12 pm

Jo Claes

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

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

2:12 pm

Jo Claes

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.

2:10 pm

Jo Claes

#dim naar het juiste type

#dim naar het juiste type

2:10 pm

Jo Claes

  • More
  • CR-99
  • started review
[UST2170] Vh4_SVC: TBX Probox (e.a.) min. bestelhoeveelheid
[UST2170] Vh4_SVC: TBX Probox (e.a.) min. bestelhoeveelheid
Tuesday 07 Feb 2017
1:33 pm

KurtC

[UST2170] Vh4_SVC: TBX Probox (e.a.) min. bestelhoeveelheid

- zelfde gedrag als LBX. In Visitor wordt conversie doorgevoerd van opgegeven verpakkingtype naar verpakking.

Options
    • -1
    • +3
    ./VerpakkingControleurWinkelkarVisitor.cls.xml
  2. … 1 more file in changeset.