Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
[UST2860][rvJCL] Vh4: Handels: Orderbevestiging versturen naar Hein

- factories samengevoegd

    • -19
    • +0
    ./impl/TransportFirmaBepalerFactory.cls.xml
    • -0
    • +33
    ./impl/Factory.cls.xml
    • -19
    • +0
    ./impl/VerkoopprijsBerekenaarFactory.cls.xml
    • -19
    • +0
    ./impl/LandenBepalerFactory.cls.xml
  1. … 27 more files in changeset.
[ICT] - Fake - Synchronize
  1. … 1 more file in changeset.
zie hierboven

zie hierboven

Is ook gecovered door Method "Test: BereidVoorBestelling plaatst de hoofding op de offerte en bepaalt de mogelijke leverdatums - standaard zijn er geen herberekende lijnen en meldingen aan de gebru...

Is ook gecovered door Method "Test: BereidVoorBestelling plaatst de hoofding op de offerte en bepaalt de mogelijke leverdatums - standaard zijn er geen herberekende lijnen en meldingen aan de gebruiker"()
Ik heb in ieder geval een test toegevoegd die controleert dat er geen verzendingmogelijkheden berekend worden voor standaard klanten.

aangezien deploy wordt er geen aanpassing doorgevoerd.

aangezien deploy wordt er geen aanpassing doorgevoerd.

Het ging hier over een bestaande parameter zonder type. ik heb het type toegevoegd.

Het ging hier over een bestaande parameter zonder type. ik heb het type toegevoegd.

  • More
  • CR-228
  • finished reviewing
Moet eigenlijk in /vhUnitTest/.../Service/GeefMogelijkeZendingen/Test.cls

Moet eigenlijk in /vhUnitTest/.../Service/GeefMogelijkeZendingen/Test.cls

Moet eigenlijk in /vhUnitTest/.../Service/BerekenPalletGegevens/Test.cls

Moet eigenlijk in /vhUnitTest/.../Service/BerekenPalletGegevens/Test.cls

hier en daar wat casing aanpassen

hier en daar wat casing aanpassen

$$$Not ipv ' Caché kennende is het ook veiliger om if (()&&()) te doen ipv if () && () (ook hieronder)

$$$Not ipv '
Caché kennende is het ook veiliger om if (()&&()) te doen ipv if () && () (ook hieronder)

Waarom die nieuwe parameter? Er wordt op het eerste zicht niets mee gedaan.

Waarom die nieuwe parameter? Er wordt op het eerste zicht niets mee gedaan.

  • More
  • CR-227
  • finished reviewing
Wat met klanten zonder transportkeuze?

Wat met klanten zonder transportkeuze?

Misschien beter om de testen hier op te splitsen per method in de Service en dan misschien hier en daar nog een testje bijgooien om wat randgevallen of true-false dingen af te toetsen

Misschien beter om de testen hier op te splitsen per method in de Service en dan misschien hier en daar nog een testje bijgooien om wat randgevallen of true-false dingen af te toetsen

ExactAantalKeer(1) om intentie duidelijk te maken

ExactAantalKeer(1) om intentie duidelijk te maken

Wat met klanten zonder transportkeuze? Of is dat niet relevant?

Wat met klanten zonder transportkeuze? Of is dat niet relevant?

Beter een ExactAantalKeer(0) op de MailAPIMock, om duidelijk de intentie te tonen

Beter een ExactAantalKeer(0) op de MailAPIMock, om duidelijk de intentie te tonen

beter de vhTest.Fake.TECH.Mail ?

beter de vhTest.Fake.TECH.Mail ?

Aangezien dit de clou is van de test, kunt ge er misschien beter een ExactAantalKeer(1) bijzetten?

Aangezien dit de clou is van de test, kunt ge er misschien beter een ExactAantalKeer(1) bijzetten?

testje voor 0 palletten toevoegen

testje voor 0 palletten toevoegen

$$$Not

$$$Not

[UST2424] Vh4: Handels: Transportmogelijkheden doorgeven TBX
[UST2424] Vh4: Handels: Transportmogelijkheden doorgeven TBX
[UST2390] Vh4: Handels: Transportmogelijkheden doorgeven
[UST2390] Vh4: Handels: Transportmogelijkheden doorgeven
[UST2424] Vh4: Handels: Transportmogelijkheden doorgeven TBX

- Tandembox verpakking mee converteren.

- Palletten berekenen per 16 lades.

- onmiddelijk bewaren bij de stapelingresultaten ipv naar den optibox te sturen.

Tricky !!!

- De conversie van tandenbox verpakking naar CollisOpPallet maakt ook gebruik van probox. Voor taor en lbx wordt adhv volumeaandeel het aantal proboxen berekend. bv 6 lades wordt dan 1 probox die gestapeld moet worden op een pallet. Voor TBX waar we altijd 16 lades op een pallet zetten, ook in geval van probox, mogen we dus geen omrekening doen. Verpakkingtype probox moet steeds 16 verpakking blijven om het aantal palletten te bepalen... andere suggesties welkom.

  1. … 13 more files in changeset.
[UST2390] Vh4: Handels: Transportmogelijkheden doorgeven

- sessie naar tranpsort in APPS laag, wordt nu KlantIdentificatie.

    • -1
    • +1
    ./impl/VerkoopprijsBerekenaarFactory.cls.xml
    • -1
    • +1
    ./impl/TransportFirmaBepalerFactory.cls.xml
  1. … 22 more files in changeset.
[UST2390] Vh4: Handels: Transportmogelijkheden doorgeven

- Beetje refactoring dto's verplaatst naar transport.

  1. … 17 more files in changeset.
[UST2390] Vh4: Handels: Transportmogelijkheden doorgeven

- Beetje refactoring verzendingmogelijkheidrepository.

    • -0
    • +46
    ./impl/VerzendingMogelijkheidRepository.cls.xml
  1. … 18 more files in changeset.