CacheAdminA_trunk2010/vhTest/Mock/APPS/OV

KlantInstellingenService.cls.xml

Checkout Tools

Line History

Line Count Graph
Line Count Graph

Stats

Commits this week: 0
Total Committers: 2
Last Commit: 07 Feb 17
Commits this week: 0
Total Lines of Code (LoC): 0
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
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
3:03 pm

KurtC

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

- hernoemen method GeefMinimimAantalProductLijnenVoorProbox -> GeefMinimaleBestelhoeveelheid

Options
    • -2
    • +2
    ./KlantInstellingenService.cls.xml
  2. … 5 more files in changeset.