HandelOfferteNaarVanHoeckeOfferteConverteerderVhisie4.cls.xml

Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
  • More
  • CR-454
  • finished reviewing
[UST2488] Vh4: Handels: Koppeling verleggen en wis oude Unishop-winkelkar
[UST2488] Vh4: Handels: Koppeling verleggen en wis oude Unishop-winkelkar
[UST2488] Vh4: Handels: Koppeling verleggen en wis oude Unishop-winkelkar

- MoveClass("APPS.VKP.Offerte.impl.HandelOfferteNaarVanHoeckeOfferteConverteerderVhisie4","APPS.VKP.Offerte.impl.KlantBestellingNaarLeverancierOfferteConverterVhisie4")

    • -138
    • +0
    ./HandelOfferteNaarVanHoeckeOfferteConverteerderVhisie4.cls.xml
  1. … 3 more files in changeset.
  • More
  • CR-282
  • finished reviewing
[UST2486R] Vh4: Handels: Migratie na bestelling

- leveradres als APPS.common.dto.LeverAdres toevoegen

- nog wat testjes die moeten herwerkt worden

    • -35
    • +26
    ./HandelOfferteNaarVanHoeckeOfferteConverteerderVhisie4.cls.xml
  1. … 5 more files in changeset.
[UST2486R] Vh4: Handels: Migratie na bestelling

- hoera, MoveClass heeft mij te pakken gekregen

    • -3
    • +3
    ./HandelOfferteNaarVanHoeckeOfferteConverteerderVhisie4.cls.xml
  1. … 1 more file in changeset.
[UST2486R] Vh4: Handels: Migratie na bestelling

- klaske verplaatst

    • -7
    • +8
    ./HandelOfferteNaarVanHoeckeOfferteConverteerderVhisie4.cls.xml
  1. … 4 more files in changeset.
  • More
  • CR-283
  • finished reviewing
Best hierover toch een klapke doen

Best hierover toch een klapke doen

Waarom zouden we dit doen ???

Waarom zouden we dit doen ???

Der staat wel redelijk wat rommel in deze klasse .. Gelieve al hetgeen momenteel nog niet op de radar staat op te kuisen.

Der staat wel redelijk wat rommel in deze klasse .. Gelieve al hetgeen momenteel nog niet op de radar staat op te kuisen.

Deze klasse mocht eigenlijk blijven bestaan en afhankelijk van welke offerte hij binnen kreeg ging door delegeren naar vhisie3 of vhisie4 versie, dan had je vorige opmerkingen qua fabriekje niet aa...

Deze klasse mocht eigenlijk blijven bestaan en afhankelijk van welke offerte hij binnen kreeg ging door delegeren naar vhisie3 of vhisie4 versie, dan had je vorige opmerkingen qua fabriekje niet aan uw been

Factory method hoort eigenlijk niet thuis op een service klasse

Factory method hoort eigenlijk niet thuis op een service klasse

Opkuisen

Opkuisen

Eigenlijk was het de moment om dit ServiceLocator patroon te vervangen door een fabriekje.

Eigenlijk was het de moment om dit ServiceLocator patroon te vervangen door een fabriekje.

Naamgeving van de klasse lijkt mij beter dat het gewoon OfferteConverter is en dat daar dan methods kunnen opkomen alla VanEcommerce en NaarEcommerce. Zodoende we maar één converter klasse hebben ...

Naamgeving van de klasse lijkt mij beter dat het gewoon OfferteConverter is en dat daar dan methods kunnen opkomen alla VanEcommerce en NaarEcommerce.

Zodoende we maar één converter klasse hebben die converteert in beide richtingen.

Ik heb dit kaartje al afgesloten, maar ik weet wel dat er nog een hoop opkuiswerk volgt, plus nog wat extra functionaliteit (maar die staat in volgende kaartjes) Ook net opgevallen dat ik nog niet...

Ik heb dit kaartje al afgesloten, maar ik weet wel dat er nog een hoop opkuiswerk volgt, plus nog wat extra functionaliteit (maar die staat in volgende kaartjes)

Ook net opgevallen dat ik nog niet met Override-objectje werk.

[UST2488] Vh4: Handels: Koppeling verleggen en wis oude Unishop-winkelkar
[UST2488] Vh4: Handels: Koppeling verleggen en wis oude Unishop-winkelkar
[UST2486] Vh4: Handels: Migratie na bestelling
[UST2486] Vh4: Handels: Migratie na bestelling
[UST2486] Vh4: Handels: Migratie na bestelling

- WIP

    • -12
    • +4
    ./HandelOfferteNaarVanHoeckeOfferteConverteerderVhisie4.cls.xml
[UST2486] Vh4: Handels: Migratie na bestelling

- OfferteNaarDtoConverter op basis van offerte, ook voor testjes

    • -4
    • +4
    ./HandelOfferteNaarVanHoeckeOfferteConverteerderVhisie4.cls.xml
  1. … 2 more files in changeset.
[UST2486] Vh4: Handels: Migratie na bestelling

- Niet te veel in 1 kaartje: 't een en 't ander in commentaar

    • -2
    • +1
    ./HandelOfferteNaarVanHoeckeOfferteConverteerderVhisie4.cls.xml
[UST2486] Vh4: Handels: Migratie na bestelling

- OfferteNaarDtoConverter op basis van offerte ipv offerteId

- Renames

    • -10
    • +9
    ./HandelOfferteNaarVanHoeckeOfferteConverteerderVhisie4.cls.xml
  1. … 2 more files in changeset.
[UST2486] Vh4: Handels: Migratie na bestelling

- WIP! APPS.VKP.Offerte.impl.HandelOfferteNaarVanHoeckeOfferteConverteerderVhisie4 gecommit om op terug te vallen

- er staat nog veel te veel in, ik weet het

    • -1
    • +142
    ./HandelOfferteNaarVanHoeckeOfferteConverteerderVhisie4.cls.xml
[UST2486] Vh4: Handels: Migratie na bestelling

- offerteservice laten kiezen tss 3 en 4 (HandelOfferteNaarVanHoeckeOfferteConverteerderVhisie*)

    • -0
    • +15
    ./HandelOfferteNaarVanHoeckeOfferteConverteerderVhisie4.cls.xml
  1. … 2 more files in changeset.