Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Voorlopig kan de introductie van dependency geen kwaad, commentaar toegevoegd dat bij eventuele uitbereiding hier wel rekening mee gehouden moet worden. nvdr : de term 'Offerte' slaat hier op de 'v...

Voorlopig kan de introductie van dependency geen kwaad, commentaar toegevoegd dat bij eventuele uitbereiding hier wel rekening mee gehouden moet worden.
nvdr : de term 'Offerte' slaat hier op de 'virtuele' offerte van DHL voor het transport in casu, niet de Vanhoeck offerte die bepalend is voor de goederen die getransporteerd moeten worden.
Wat dit betreft is het aantal laden te transporteren volledig irrelevant, want DHL rekent niet in 'laden', maar palletten.
De afpsraak van 10€ is ts van hoecke en Hein, DHL heeft hier eigenlijk niets in te zien.
Pragmatisch is het wel de minst kostend oplossing om het hier te bereken.

OfferteBerekenaar heeft voor zover ik me herinner geen afhankelijkheden naar data zoals offertes en winkelkarren. M.a.w. deze software zou berekeningen kunnen maken enkel a.d.h.v. berekeninput gege...

OfferteBerekenaar heeft voor zover ik me herinner geen afhankelijkheden naar data zoals offertes en winkelkarren. M.a.w. deze software zou berekeningen kunnen maken enkel a.d.h.v. berekeninput gegevens.

APPS.TRANSP.OfferteService.dto.BerekenInput heeft lijst van APPS.TRANSP.OfferteService.dto.TeverzendenItem die het aantal bevatten. Dus ik denk dat er met deze gegevens het totaal aantal laden berekend kan worden.

Maarrrr.. ik ben niet zeker of dit ook het geval is met proboxen...Er zit ergens een berekening van aantal lades naar aantal proboxen en mogelijks worden de proboxen als verzendingitems meegegeven. Als dit het geval is denk ik dat er in een hogere laag waar winkelkarren wel gebruikt worden, de berekeninput dto aangepast moet worden met een extra property waarin het aantal lades van daaruit meegegeven kan worden.

"Als" er voor AX een fase komt waarbij ze het berekenen van transport van cache gebruiken a.d.h.v. een webservice met berekeninipnut dto, dan zou dat een goede reden zijn om geen winkelkarren of offertes te gebruiken. Allé, niet om een p.i.t.a. te zijn , maar te overwegen.

Even nagezien : APPS.VKP.impl.OfferteService.VerpakkingConvertor
in geval van probox worden dus eerst de verzendingitems gemaakt met afmetingen en die worden in berekeninput meegegeven. Dus in berekeninput zit het aantal lades niet meer als het probox is, maar het aantal probox dozen.

[VRB548] EC: MP: Hein: lagere transportkosten bij weinig lades
[VRB548] EC: MP: Hein: lagere transportkosten bij weinig lades
[VRB548] EC: MP: Hein: lagere transportkosten bij weinig lades

- Ongeacht aantal palletten, minder dan 4 lades -> 10€

  1. … 1 more file in changeset.
[VRB548] EC: MP: Hein: lagere transportkosten bij weinig lades

- testjes uitgebreid

[VRB548] EC: MP: Hein: lagere transportkosten bij weinig lades

- aangepast verkoopprijs berekenen

  1. … 1 more file in changeset.
[UST2390r] Vh4: Handels: Transportmogelijkheden doorgeven

- testje met 0 palletten.

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.

  • 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

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

- Beetje refactoring internationalezending naar transport.

    • -0
    • +18
    ./LandenBepaler/GeefLandCodes/Test.cls.xml
  1. ./TransportFirmaBepaler/GeefTransportFirmas
  2. ./VerkoopprijsBerekenaar/BerekenVerkoopPrijs
  3. … 163 more files in changeset.