WinkelkarUtils.cls.xml

Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
commentaar mag weg

commentaar mag weg

[ICT-1767] [rvTVE] [rvJWI] EC: MP: ORGALUX: productlijnen mee kopiëren van MP-winkelkar naar VH-winkelkar...
[ICT-1767] [rvTVE] [rvJWI] EC: MP: ORGALUX: productlijnen mee kopiëren van MP-winkelkar naar VH-winkelkar...
[ICT-1767] [rvTVE] [rvJWI] EC: MP: ORGALUX: productlijnen mee kopiëren van MP-winkelkar naar VH-winkelkar

-GeefLijnenIterator toegevoegd voor het veralgemenen van de methodes zo dat zowel productlijnen als maatwerklijnen worden geiterate ;)

  1. … 2 more files in changeset.
[DEF764] EC: MP: Volgorde winkelkarlijnen anders in doorbestelde winkelkar

- review opm

Toch wel, maar impliciet. De asserts gebeuren per lijn van de input. Er wordt niet getest op de aanwezigheid, maar de input lijst wordt overlopen, en lijn per lijn gematched met de kopie.

Toch wel, maar impliciet. De asserts gebeuren per lijn van de input.
Er wordt niet getest op de aanwezigheid, maar de input lijst wordt overlopen, en lijn per lijn gematched met de kopie.

Dee volgorde van de maatwerklijnen wordt niet getest

Dee volgorde van de maatwerklijnen wordt niet getest

naamgeving: Key (of Sleutel) ipv Subscript?

naamgeving: Key (of Sleutel) ipv Subscript?

[DEF764] EC: MP: Volgorde winkelkarlijnen anders in doorbestelde winkelkar
[DEF764] EC: MP: Volgorde winkelkarlijnen anders in doorbestelde winkelkar
[DEF764] EC: MP: Volgorde winkelkarlijnen anders in doorbestelde winkelkar

- Array gebruiken ipv list voor de maatwerklijnen om volgorde volgens externeid te behouden.

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

- WinkelkarUtils uitmockbaar maken

  1. … 2 more files in changeset.
  • More
  • CR-532
  • finished reviewing
#dimmekes nakijken zie GeefMaatwerkLijnen()

#dimmekes nakijken zie GeefMaatwerkLijnen()

Ofwel test je dat MaatwerkLijn1 en MaatwerkLijn2 in de lijst zitten als je dat niet doet is het totaal niet noodzakelijk om daar variabelen aan te verspillen

Ofwel test je dat MaatwerkLijn1 en MaatwerkLijn2 in de lijst zitten als je dat niet doet is het totaal niet noodzakelijk om daar variabelen aan te verspillen

Lijkt niet echt te kloppen As WS.vhisie4.Winkelkar.dto.MaatwerkLijn ; Misschien beter om AssertEquals(WinkelkarUtils.GeefMaatwerkLijnen(Winkelkar).Count(),0) te schrijven dan kan de dim niet verke...

Lijkt niet echt te kloppen As WS.vhisie4.Winkelkar.dto.MaatwerkLijn ;

Misschien beter om AssertEquals(WinkelkarUtils.GeefMaatwerkLijnen(Winkelkar).Count(),0) te schrijven dan kan de dim niet verkeerd zijn

[UST2039] Vh4_SVC: Sync web naar caché
[UST2039] Vh4_SVC: Sync web naar caché
[UST2039] Vh4_SVC: Volwaardige sync web naar caché

- Herwerkt zodat ook productlijnen gesynchroniseerd kunnen worden.

- Voor herbereken blijft het enkel maatwerklijen.

- nem. :P

  1. … 20 more files in changeset.
  • More
  • CR-47
  • finished reviewing
[UST2175] : Vh4_SVC : Test services klaar hebben voor intracto

- Verpakking bepaling tandembox neemt de verpakking van de legacy kenmerken als de verpakking niet ingevuld is op de gewone kenmerken

- ExterneId wordt ook overgenomen bij de ProductLijnen

- WinkelkarUtils biedt vanaf heden ook de ProductLijnen aan

  1. … 3 more files in changeset.
[UST2038] Vh4_SVC: HerberekenMaatwerk

- Een iets andere werking geimplementeerd

  1. … 1 more file in changeset.
  • More
  • CR-47
  • started review
[UST2038] Vh4_SVC: HerberekenMaatwerk
[UST2038] Vh4_SVC: HerberekenMaatwerk