Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
  • 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
  1. … 3 more files in changeset.
[UST1915] Hein doorbestelde offerte adres van klant

- In test ook leveradres controleren.

  1. … 1 more file in changeset.
[UST1915] Hein doorbestelde offerte: adres van klant

- Method hernoemt

  1. … 4 more files in changeset.
[UST1915] Hein doorbestelde offerte: adres van klant

- Refactoring testklasse

[UST1915] Hein doorbestelde offerte: adres van klant

- adres klanten van hein worden niet aangepast bij het kopieren van offerte. Bij standaard klanten wordt het standaard leveradres ingevuld.

- Ge wilt nie weten wat ik heb moeten veranderen heb om 1 lijn code te testen in HandelOfferteNaarVanHoeckeOfferteConverteerder

- De testen zelf moeten nog nagekeken worden op vereenvoudiging en variabele namen.

  1. … 12 more files in changeset.