CacheAdminA_trunk2010/vhUnitTest/DOM/VKP/Maatwerk/impl/BestelService

ValideerBestelEenheid

Checkout Tools

Line History

Line Count Graph
Line Count Graph

Stats

Commits this week: 0
Total Committers: 2
Last Commit: 09 Sep 15
Commits this week: 0
Total Lines of Code (LoC): 270
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
Friday 02 Dec 2022
10:29 am

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

=> Pijnlijke vaststelling dat ik nog een paar gebruikerkes was vergeten mee aan te passen. Bij deze...

Options
    • -2
    • +2
    ./TAX/Test.cls.xml
  2. … 3 more files in changeset.
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...
Tuesday 02 Apr 2019
1:59 pm

Tommy Hebb

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

- Voor BulkPerOnderdeelGegroepeerd geldt de BestelEenheid voor Zijkant ook voor de andere componenten => Fix test

Options
    • -7
    • +17
    ./TAX/Test.cls.xml
Monday 25 Mar 2019
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
    • -2
    • +2
    ./TAX/Test.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
    • -25
    • +132
    ./TAX/Test.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
12:00 pm

Tommy Hebb

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

- BulkPerOnderdeelGegroepeerd => toepassing nieuwe implementatie in BestelService

Options
    • -18
    • +25
    ./TAX/Test.cls.xml
  2. … 4 more files in changeset.
11:22 am

Tommy Hebb

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

- Louter casing

Options
    • -10
    • +10
    ./TAX/Test.cls.xml
10:37 am

Tommy Hebb

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

- BestelEenheidLosseComponentenInBulkBepaler => Aangepaste implementatie voor LosseComponentenInBulk

Options
    • -19
    • +22
    ./TAX/Test.cls.xml
  2. … 1 more file in changeset.
Thursday 28 Feb 2019
3:12 pm

Tommy Hebb

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

- Meldingtekst bij ongeldige BestelEenheid vermeldt nu ook wat het minimum is en dat er ook een veelvoud van gebruikt kan worden

Options
    • -1
    • +1
    ./TAX/Test.cls.xml
  2. … 1 more file in changeset.
Monday 14 Sep 2015
3:00 pm

MartijnV

[UST1208] TAX flatpack bulk: aantal controleren:

- tests uitgebreid voor geldige en ongeldige aantallen bepaald door DoosAfmetingenBepaler

Options
    • -3
    • +43
    ./TAX/Test.cls.xml
  2. … 10 more files in changeset.
Wednesday 09 Sep 2015
11:02 am

MartijnV

[UST1208] TAX flatpack bulk: aantal controleren:

- refactor tests

Options
  1. ./TAX
    • -0
    • +31
    ./LBX/Test.cls.xml
    • -0
    • +27
    ./TestBase.cls.xml
  4. ./ValideerBestelEenheid
  5. ./LBX
    • -0
    • +45
    ./TAX/Test.cls.xml
  7. … 5 more files in changeset.