CacheAdminA_trunk2010/DOM/VKP/Maatwerk/impl

BestelService

Checkout Tools

Line History

Line Count Graph
Line Count Graph

Stats

Commits this week: 0
Total Committers: 2
Last Commit: 28 Feb 19
Commits this week: 0
Total Lines of Code (LoC): 237
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 01 Dec 2022
5:42 pm

Tommy Hebb

[ICT-3246] [rvWV] PM: Maatwerk: TAX: BOMBOL: Vaste producten BOM aanpassen maar prijs behouden:

- Verduidelijkingske bij iets wat niets met deze story te maken heeft => Wordt nooit gebruikt ivm LadeOnderdeel Bodem, dus dat mag ook blijken uit de naamgeving

Options
    • -1
    • +1
    ./BestelEenheidBepaler.cls.xml
  2. … 3 more files in changeset.
Thursday 04 Apr 2019
2:11 pm

Tommy Hebb

2:05 pm

Tommy Hebb

[UST4202] TAX-LC-002: Nieuwe verpakking (betere palletstapeling) - deel 1: calculatie kosten: Aanpassingen na testen en review:

- Kleine refactor na revies => Dubbele implementatie => forwarden

Options
    • -29
    • +1
    ./BestelEenheidBepaler.cls.xml
  2. … 1 more file in changeset.
1:53 pm

Tommy Hebb

12:47 pm

Wim Vermeulen

12:43 pm

Wim Vermeulen

Bouwsteen mapping : je mocht gerust de 8 tekens volledig benutten : bijvoorbeeld : VPFROSPC, VPFROINS, VPFROVDP, ...

Bouwsteen mapping : je mocht gerust de 8 tekens volledig benutten :
bijvoorbeeld : VPFROSPC, VPFROINS, VPFROVDP, ...

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
12:25 pm

Tommy Hebb

[UST4202] TAX-LC-002: Nieuwe verpakking (betere palletstapeling) - deel 1: calculatie kosten: Aanpassingen...
[UST4202] TAX-LC-002: Nieuwe verpakking (betere palletstapeling) - deel 1: calculatie kosten: Aanpassingen...
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...
Tuesday 02 Apr 2019
5:12 pm

Tommy Hebb

[UST4202] TAX-LC-002: Nieuwe verpakking (betere palletstapeling) - deel 1: calculatie kosten: Aanpassingen na testen en review:

- BestelEenheid voor BulkPerOnderdeelGegroepeerd voor Front => eigen implementatie => Hoogte N en M => 20 en Hoogte K en C => 10

Options
    • -0
    • +8
    ./BestelEenheidBepaler.cls.xml
  2. … 1 more file in changeset.
Tuesday 26 Mar 2019
1:36 pm

Tommy Hebb

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

- De DoosAfmetingenBepalerFactory geeft de bepaler

Options
    • -6
    • +12
    ./BestelEenheidBepaler/BestelEenheidLosseComponentenInBulkBepaler.cls.xml
  2. … 1 more file in changeset.
Monday 25 Mar 2019
5:49 pm

Tommy Hebb

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

- TestCase voor BestelEenheidBepaler

Options
    • -8
    • +2
    ./BestelEenheidBepaler.cls.xml
  2. … 3 more files in changeset.
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
    • -3
    • +3
    ./BestelEenheidBepaler.cls.xml
    • -0
    • +117
    ./BestelEenheidBepaler/BestelEenheidLosseComponentenInBulkBepaler.cls.xml
    • -117
    • +0
    ./BestelEenheidLosseComponentenInBulkBepaler.cls.xml
  4. … 6 more files in changeset.
4:54 pm

Tommy Hebb

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

- BestelEenheidBulkPerOnderdeelGegroepeerdBepaler => Move class naar 1 niveau dieper

Options
    • -0
    • +28
    ./BestelEenheidBepaler/BestelEenheidBulkPerOnderdeelGegroepeerdBepaler.cls.xml
    • -3
    • +3
    ./BestelEenheidBepaler.cls.xml
  3. ./BestelEenheidBepaler
  4. … 8 more files in changeset.
4:43 pm

Tommy Hebb

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

- Move class naar 1 niveau dieper

Options
    • -0
    • +84
    ./BestelEenheidBepaler.cls.xml
  2. … 7 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
    • -62
    • +35
    ./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!

11:19 am

Tommy Hebb

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

- Losse componenten => BestelEenheid => Refactor

Options
    • -24
    • +12
    ./BestelEenheidLosseComponentenInBulkBepaler.cls.xml
  2. … 1 more file in changeset.
8:43 am

Wim Vermeulen

[UST4184] TAX-LC-002: Nieuwe verpakking (betere palletstapeling) - fix voor TommyH

- BestelEenheid Bepaler via enu ProductieWijze i.p.v. via LadeOnderdeel. Dit laatste maakte een oneindige lus (FrameStack) bij het maken van de Builders / MaakLosseComponent LadeOnderdeel

- Uitzondering voor Bodem, die het LadeOnderdeel echt nodig heeft.

- UT aangepast

Options
    • -7
    • +7
    ./BestelEenheidLosseComponentenInBulkBepaler.cls.xml
  2. … 1 more file in changeset.
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:...