impl

- ook melding in "maakordervanofferte" aangepast zodat er voorlopig in de logging duidelijkheid is over het niet aanmaken van een order.

- kleine prijsverschillen voor legra met kleur wit niet melden dat er prijsverschillen zijn.

- Geen Tandembox meenemen in berekeningen.

Een optie zou zijn de palletberekening van de optibox te vervangen, maar dan kunnen we het juiste gewicht niet bepalen, enkel het aantal palletten.

Indien er dan berekeningen gebeuren voor andere dan Hein, dan krijgen we verkeerde prijzen en dat willen we niet.

In dit geval hebben we ook de gewichten en afmetingen nodig van de tandemboxen, die staan momenteel overal .

We kiezen om deze tijdelijke implementatie hardcoded 16 lades per pallet te gebruiken in geval van Hein.

Voorbereiding

- lang leve de moveclass...

- Voor de properteit de unishopbesteller de hoofding via een nieuw toegevoegde method 'zethoofding' of de offerteservice de hoofding laten opslaan en en passant de externeverkoopservice ook via deze nieuwe method laten gaan.

- Aanpassingskes gedaan aan de RegioInstellingImporteur --> Em kan enkel maar de interne en de externe verantwoordelijke updaten van bestaande regiod's adhv den regio excel.

- review opmerkingen verwerkt.

- Gebruiken van de door DOM.EC.Winkelkar.impl.LeverDatumBepaler datum array om de vroegste leverdatum te bepalen en de vroegste leverdatum per lijn terug te geven.

- DOM.EC.Winkelkar.impl.LeverDatumBepaler : Een datum array opvullen met voor elke lijn een winkelkar de vroegste leverdatum

- Method is geen classmethod meer en gebruikte klasse laten injecten

- Method op offerte IsVhisie4 en toevoegen tekst aan E-mail naar OV.

- Aanpassing om groepnaam ook op productlijnen te krijgen

- Refactoring winkelkar VoegToeMaatwerkLijn geeft object weer ipv ID.

- Raiseevent voor OfferteGemaakt is verhuisd naar VerkoopService.

Bedenking : GemaaktofferteEvent zou zo dicht mogelijk bij het dataobject moeten zitten zodat een event geraiset wordt bij het aanmaken van een offerte, aangemaakt vanuit eender welk stuk code. (bv op repository) Indien er enkel geraiset wordt vanuit verkoopservice, dan heeft de event waarschijnlijk een andere bedoeling/betekenis.

- Authorisatie controles voor offerte, winkelkar + testje

- selectie criteria voor offertes aangepast in GeefOfferteSamenvattingenVoorKlant

- BewaarMaatwerkLijn met ExterneId + test

- te bekijken of dit via verkoopservice moet verlopen. (afspraak voor lijnen is niet via verkoopservice.)

- Melding veralgemenen

- RaiseOfferteEvent hernoemt naar RaiseOfferteGemaaktEvent.

- foutmelding aangepast. (geen details in melding opnemen die missschien niet waar zijn.)

- %id is geefid() geworden

- raiseevent voor offerte naar offerteservice verplaatst om test op maakofferte te kunnen uitvoeren. (test controle op externeid.) hoort daar ook thuis

- Daar ik gezien had dat een nieuwe unittest een Save gebruikte ( dus persistent en multiprocess aware is ).

Heb ik wat basis aanpassingen gedaan aan de code zodoende de dependencies al wat duidelijker zichtbaar zijn in de constructor van de klasse.

Dit is een eerste stap in de goede richting.. Verdere stappen zullen in de toekomst wel genomen worden.

[UST1931] Hein: V1 LBX tussenetiket: wegwerppallet vermelden

- ColliDragerType wordt bepaald en opgeslagen bij zendingdetails.

(beter zou zijn als de volledige flow doorlopen wordt om dit te bepalen. Dwz bij request om palletten te bepalen reeds meegeven welke palletten / dragers er ondersteund worden voor een bepaalde klant (bv Hein). Deze waarden meegeven naar de optibox en het resultaat van de optibox bepaalt dan ColliDragerType dat via verzendingmogelijkheden, offerte,... op het etiket komt. Nog niet doorgevoerd omdat er voor andere klanten nog geen nood is voor extra info op etiket.)

........

[UST1931] Hein: V1 LBX tussenetiket: wegwerppallet vermelden

- Overbodige code is verwijderd uit test.

........