HandelOfferteNaarVanHoeckeOfferteConverteerderVhisie3

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

- Vhisie4 de nieuwe repository laten gebruiken

- Standaardtestje

  1. … 3 more files in changeset.
[UST2488] Vh4: Handels: Koppeling verleggen en wis oude Unishop-winkelkar

- Testen verwijderd uit Vhisie3 (en 4)/converteer/test (zijn al verplaatst naar de kopieerder)

  1. … 1 more file in changeset.
[UST2488] Vh4: Handels: Koppeling verleggen en wis oude Unishop-winkelkar

- Logica uit de Converter3 gepeuterd en in Kopieerder gestoken, opdat het gebruikt zal kunnen worden door Converter4

  1. … 5 more files in changeset.
[UST2488] Vh4: Handels: Koppeling verleggen en wis oude Unishop-winkelkar

- MoveClass("APPS.VKP.Offerte.impl.HandelOfferteNaarVanHoeckeOfferteConverteerderVhisie3","APPS.VKP.Offerte.impl.KlantBestellingNaarLeverancierOfferteConverterVhisie3")

  1. … 3 more files in changeset.
  • More
  • CR-282
  • finished reviewing
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.

[UST2486] Vh4: Handels: Migratie na bestelling
[UST2486] Vh4: Handels: Migratie na bestelling
[UST2486] Vh4: Handels: Migratie na bestelling

- MoveClass("vhUnitTest.APPS.VKP.Offerte.impl.HandelOfferteNaarVanHoeckeOfferteConverteerder.Converteer.Test","vhUnitTest.APPS.VKP.Offerte.impl.HandelOfferteNaarVanHoeckeOfferteConverteerderVhisie3.Converteer.Test")

  1. ./HandelOfferteNaarVanHoeckeOfferteConverteerderVhisie3
  2. … 3 more files in changeset.