Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
[ICT-3275] [rvTVE] Oud framework voor CubeIQ opkuisen
[ICT-2829] Haefele NO multipersonality:

- Initiële commit (Personality toegevoegd en alles errond dat het doet werken)

    • -1
    • +1
    ./VanHoecke/impl/LandenBepaler.cls.xml
  1. … 28 more files in changeset.
[ICT] GeefMogelijkeZendingen() in Transport.Service

- ExtraInfo toegeveoegd bij Foutafhandeling "verzendingmogelijkheden"

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.

[VRB548] EC: MP: Hein: lagere transportkosten bij wein

- TransportService : InnerException toegevoegd aan APPS.EC.Transport.exc.KanNietBerekenen

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
    • +2
    ./Hein/impl/VerkoopprijsBerekenaar.cls.xml
  1. … 1 more file in changeset.
[VRB548] EC: MP: Hein: lagere transportkosten bij weinig lades

- aangepast verkoopprijs berekenen

    • -0
    • +1
    ./Hein/impl/VerkoopprijsBerekenaar.cls.xml
  1. … 1 more file in changeset.
[UST3024] [rvJCL] Vh4: Hein: alle wijzigingen via trunk en deploy goed zetten.

- Bepaling originele offerte aangepast om Hein al te doen werken.

[UST2865] - EPIC50 - Seefelder Personality aanmaken

- Cleanup overige methods -> ongedaan gemaakt blijkbaar zat daar een specialeke in -> ipv het probleem aan de bron aan te pakken -> property ID dat geen ID is ofwel property renamen ofwel van de property effectief een ID maken

    • -1
    • +25
    ./impl/VerzendingMogelijkheidRepository.cls.xml
[UST2865] - EPIC50 - Seefelder Personality aanmaken

- VerzendingMogelijkheidRepository - ZoekViaOfferteId toegevoegd

- Cleanup overige methods

    • -22
    • +5
    ./impl/VerzendingMogelijkheidRepository.cls.xml
[UST3024] [rvJCL] Vh4: Hein: alle wijzigingen via trunk en deploy goed zetten.

- Method ipv property gebruiken

[UST3024] [rvJCL] Vh4: Hein: alle wijzigingen via trunk en deploy goed zetten.
  1. … 45 more files in changeset.
[UST2860][rvJCL] Vh4: Handels: Orderbevestiging versturen naar Hein

- factories samengevoegd

[UST2860][rvJCL] Vh4: Handels: Orderbevestiging versturen naar Hein

- factories samengevoegd

    • -36
    • +0
    ./impl/LandenBepalerFactory.cls.xml
    • -0
    • +79
    ./impl/Factory.cls.xml
    • -36
    • +0
    ./impl/TransportFirmaBepalerFactory.cls.xml
    • -36
    • +0
    ./impl/VerkoopprijsBerekenaarFactory.cls.xml
  1. … 27 more files in changeset.
[UST2865] - EPIC50 - Seefelder Personality aanmaken

- verzendingmogelijkheid van offertevan handel overnemen... kan meerdere iteraties zijn. _> klant, seefelder, hein...

  1. … 1 more file in changeset.
[UST2860] Vh4: Handels: Orderbevestiging versturen naar Hein

- Kopieren van Offerte verzendingitems.

  1. … 2 more files in changeset.
[ICT] Dubbell code, super class bevat dezelfde code.
    • -20
    • +0
    ./impl/BerekeningFoutenRepository.cls.xml
[VRB430] Refactoring TransportBerekeningen.

- ' => Not :-)

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-231
  • finished reviewing