Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
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

- constructors bij gegeneerd door compilatie de mock parentclasses

  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

- Verkeerd returntype in WSimpl.Vhisie4.Winkelkar.WinkelkarExport.UnishopWinkelkarConverter.Converter gecorrigeerd

  1. … 2 more files in changeset.
[UST2486] Vh4: Handels: Migratie na bestelling

- beginsel van OfferteNaarDtoConverter

  1. … 4 more files in changeset.
  • More
  • CR-242
  • finished reviewing
[UST2082] - Migratie openstaande Unishop-winkelkarren naar Vhisie4 (nog zonder de kenmerken van de...
[UST2082] - Migratie openstaande Unishop-winkelkarren naar Vhisie4 (nog zonder de kenmerken van de...
  • More
  • CR-233
  • finished reviewing
casing trekt op niets en methodname/testname is niet altijd correct voor de rest dik in orde

casing trekt op niets en methodname/testname is niet altijd correct
voor de rest dik in orde

[UST2082] - Migratie openstaande Unishop-winkelkarren naar Vhisie4 (nog zonder de kenmerken van de...
[UST2082] - Migratie openstaande Unishop-winkelkarren naar Vhisie4 (nog zonder de kenmerken van de...
[UST2082] - Migratie openstaande Unishop-winkelkarren naar Vhisie4 (nog zonder de kenmerken van de lades)

- Aanmaken van de Converter en Retriever

- WinkelkarLijnConverter kan nu ook naar Vhisie4 converteren

- FlowWinkelkarConverter -> rename naar Converter -> deze zal zowel het converteren van flow offertes , de tbx en unishops bundelen , kwestie van wat flexibiliteit te hebben

  1. … 14 more files in changeset.