CacheAdminA_trunk2010/vhTest/Fake/DOM/VKP/Maatwerk/impl/BestelService

BestelEenheidLosseComponentenInBulkBepaler.cls.xml

Checkout Tools

Line History

Line Count Graph
Line Count Graph

Stats

Commits this week: 0
Total Committers: 1
Last Commit: 25 Mar 19
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
Thursday 04 Apr 2019
1:53 pm

Tommy Hebb

12:35 pm

Wim Vermeulen

12:29 pm

Wim Vermeulen

Ter info : 1) Alles in één testklasse is oké; maar je kan ook meerdere testklasses maken, met naam .TestLosseComponentInBulk.cls en .TestBOG.cls 2) Testklasse bevat meestal ook de naam van de publi...

Ter info :
1) Alles in één testklasse is oké; maar je kan ook meerdere testklasses maken, met naam .TestLosseComponentInBulk.cls en .TestBOG.cls
2) Testklasse bevat meestal ook de naam van de public method die je aan 't testen bent. Maar misschien werd de naam dan te lang.
Je hoeft dit hier niet te veranderen, maar is taak van reviewer om dit op te merken

12:11 pm

Wim Vermeulen

Deze method doet exact hetzelfde als ..BestelEenheidLosseComponentenInBulkBepaler.GeefBestelEenheidZRFLosseComponentenInBulk() Uiteraard klopt dat ook qua resultaat, maar waarom staat de code dan d...

Deze method doet exact hetzelfde als ..BestelEenheidLosseComponentenInBulkBepaler.GeefBestelEenheidZRFLosseComponentenInBulk()
Uiteraard klopt dat ook qua resultaat, maar waarom staat de code dan dubbel? --> dubbel onderhoud.
Misschien kan je hier (BOG) de implementatie van LosCompInBulkk laten oproepen.

Wednesday 03 Apr 2019
11:26 am

Tommy Hebb

[UST4187] TAX-LC-002: Nieuwe verpakking (betere palletstapeling) - deel 1: calculatie kosten: KlantEtiket...
[UST4187] TAX-LC-002: Nieuwe verpakking (betere palletstapeling) - deel 1: calculatie kosten: KlantEtiket...
Monday 25 Mar 2019
4:59 pm

Tommy Hebb

[UST4187] TAX-LC-002: Nieuwe verpakking (betere palletstapeling) - deel 1: calculatie kosten: KlantEtiket ZRF:

- BestelEenheidLosseComponentenInBulkBepaler => Move class naar 1 niveau dieper

Options
    • -47
    • +0
    ./BestelEenheidLosseComponentenInBulkBepaler.cls.xml
  2. … 8 more files in changeset.
4:33 pm

Tommy Hebb

[UST4187] TAX-LC-002: Nieuwe verpakking (betere palletstapeling) - deel 1: calculatie kosten: KlantEtiket ZRF:

- Grondige refactor BestelService => Ts BestelService en haar dependencies die BestelEenheid teruggaven zit nu een BestelEenheidBepaler die BestelEenheid teruggeeft volgens LadeOnderdeel.

Dit was nodig opdat de builders die tussenlaag direct kunnen aanspreken en het van daaruit belangrijk is volgens LadeOnderdeel de BestelEenheid te krijgen en niet volgens Kenmerken,

want igv BulkPerOnderdeelGegroepeerd zijn de Kenmerken niet specifiek genoeg om te weten over welk LadeOnderdeel het gaat.

Options
    • -0
    • +28
    ./BestelEenheidLosseComponentenInBulkBepaler.cls.xml
  2. … 8 more files in changeset.
Monday 18 Mar 2019
4:22 pm

Tommy Hebb

Friday 15 Mar 2019
3:02 pm

Wim Vermeulen

11:22 am

Tommy Hebb

Done!

Done!

Thursday 14 Mar 2019
5:33 pm

Tommy Hebb

Is overgenomen van de defaulting van de (énige) implementatieklasse. De mock wordt van de interface gemaakt. Dat maakte dat die ander gedrag vertoonde dan de echte.

Is overgenomen van de defaulting van de (énige) implementatieklasse. De mock wordt van de interface gemaakt. Dat maakte dat die ander gedrag vertoonde dan de echte.

5:29 pm

Tommy Hebb

Die worden allebei in de caller van deze method al uitgesloten

Die worden allebei in de caller van deze method al uitgesloten

Friday 08 Mar 2019
4:02 pm

Wim Vermeulen

object LadeOnderdeel doorgeven, i.p.v. Rol As enu dan moet je voor Bodem niet opnieuw de "..MaakLosseComponent(Kenmerken) " opnieuw oproepen --> vermijd dubbel werk.

object LadeOnderdeel doorgeven, i.p.v. Rol As enu
dan moet je voor Bodem niet opnieuw de "..MaakLosseComponent(Kenmerken) " opnieuw oproepen --> vermijd dubbel werk.

3:04 pm

Wim Vermeulen

Dit is de algemene MaatwerkCalcAPI --> straf dat je hier de parameter gaat defaulten naar klant "Halux". Ik zou verwachten dat je ergens een specifieke oproep hebt, waar je dan gecontroleerd de kla...

Dit is de algemene MaatwerkCalcAPI --> straf dat je hier de parameter gaat defaulten naar klant "Halux".
Ik zou verwachten dat je ergens een specifieke oproep hebt, waar je dan gecontroleerd de klant meegeeft.

Laat voorlopig maar staan, maar hou er wel rekening mee, dat dit soort defaultings "gedurfd" is

2:37 pm

Wim Vermeulen

De bijhorende UT geeft aan dat *ID = "" geeft "" *ID = 0 geeft "" Mijn vraag : is dit wenselijk, of zou het beter zijn om een exceptie te smijten? Want voor deze waarden is het eigenlijk zinloos ...

De bijhorende UT geeft aan dat

  • ID = "" geeft ""
  • ID = 0 geeft ""
    Mijn vraag : is dit wenselijk, of zou het beter zijn om een exceptie te smijten?
    Want voor deze waarden is het eigenlijk zinloos om de bepaler op te roepen.
2:34 pm

Wim Vermeulen

Deze lijn zou ik binnen de Else zetten. Nu wordt ze uitgevoerd, zelfs bij ID > 1332 en nadien wordt dan toch exception gesmeten. M.a.w. de AflfaKey() uitvoeren was niet nodig.

Deze lijn zou ik binnen de Else zetten.
Nu wordt ze uitgevoerd, zelfs bij ID > 1332 en nadien wordt dan toch exception gesmeten. M.a.w. de AflfaKey() uitvoeren was niet nodig.

Thursday 07 Mar 2019
5:30 pm

Tommy Hebb

[UST4166] TAX-LC-002: Nieuwe verpakking (betere palletstapeling) - deel 1: calculatie kosten:...
[UST4166] TAX-LC-002: Nieuwe verpakking (betere palletstapeling) - deel 1: calculatie kosten:...
Monday 04 Mar 2019
10:30 am

Tommy Hebb

[UST4166] TAX-LC-002: Nieuwe verpakking (betere palletstapeling) - deel 1: calculatie kosten: MinimumBestelhoeveelheid:

- BestelEenheidLosseComponentenInBulkBepaler => Mockable gemaakt

Options
    • -0
    • +19
    ./BestelEenheidLosseComponentenInBulkBepaler.cls.xml
  2. … 5 more files in changeset.