Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Deze code was dus niet correct...http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif is intussen gewijzigd. Later misschien nog te refactoren.

Deze code was dus niet correct... is intussen gewijzigd. Later misschien nog te refactoren.

ik ga ondanks de beslissing om hier niet aan verder te werken, toch een kaartje aanmaken.. ge weet maar nooit.

ik ga ondanks de beslissing om hier niet aan verder te werken, toch een kaartje aanmaken.. ge weet maar nooit.

  • More
  • CR-580
  • finished reviewing
LogActie lijkt mij toch niet echt correct te zijn, het hoort allemaal in de detail te staan. Actie lijkt mij eerder ControleKenmerken - Kwestie dat de logging beetje ordentelijk is

LogActie lijkt mij toch niet echt correct te zijn, het hoort allemaal in de detail te staan.

Actie lijkt mij eerder ControleKenmerken - Kwestie dat de logging beetje ordentelijk is

DOM logica

DOM logica

DOM logica

DOM logica

APPS logica

APPS logica

APPS logica

APPS logica

APPS logica

APPS logica

APPS logica

APPS logica

DOM Logica

DOM Logica

DOM Logica

DOM Logica

DOM Logica

DOM Logica

[UST2867] Vh4_SVC: Snelheid sync winkelkar naar Caché controleren en tot aanvaardbaar punt brengen
[UST2867] Vh4_SVC: Snelheid sync winkelkar naar Caché controleren en tot aanvaardbaar punt brengen
[UST2867] Vh4_SVC: Snelheid sync winkelkar naar Caché controleren en tot aanvaardbaar punt brengen

- Methods naar Dom WineklkarService.

- Testen aangepast. (zijn niet verbeterd of aangevuld.)

  1. … 7 more files in changeset.
[UST2867] Vh4_SVC: Snelheid sync winkelkar naar Caché controleren en tot aanvaardbaar punt brengen

- refactoring nodig. APPS en DOM scheiden. -> Move class

  1. … 9 more files in changeset.
[UST2175] : Vh4_SVC : Test services klaar hebben voor intracto

- Aanpassing om groepnaam ook op productlijnen te krijgen

  1. … 3 more files in changeset.
[UST2175] : Vh4_SVC : Test services klaar hebben voor intracto

- ProductLijnen hebben nu ook notie van de groeplijn

  1. … 9 more files in changeset.
[UST2079] BewaarProductLijn moet ExterneId laten bewaren

- ExterneId via ProductLijn.Maak()

  1. … 5 more files in changeset.
[UST1942] EDI bestellingen met enkel voorgedefinieerde producten

- ervoor zorgen dat de productlijnen in een aparte groep komen te staan (gebeurt nu in de ProductLijnBuilder.MaakLijn...)

  1. … 9 more files in changeset.
[UST1862R] DKG beslag: pakketreferentie in EDI omzetten in de nodige productlijnen

-APPS.VKP.impl.OfferteService : VoegToeProduct -> method lijkt mij niet echt unittestbaar door de %OpenId .. beter hiervoor de repository gebruiken - DOM.EC.Winkelkar.LijnRepository.GeefViaID of DOM.EC.Winkelkar.WinkelkarRepository GeefLijnViaID - Eerste mogelijkheid lijkt mij toch net iets beter - kwestie van de ene repo is voor de winkelkarren en dan de andere specifiek voor de lijnen

- fake en mock ProductLijnRepository ook ineens weggehaald

- moet nog eens denken over een schone testbenaming in OfferteService/VoegToeProduct/Test

  1. … 10 more files in changeset.
[UST1862R] DKG beslag: pakketreferentie in EDI omzetten in de nodige productlijnen

-vhUnitTest.APPS.EC.Winkelkar.impl.WinkelkarService.VoegToeProduct.Test - waarom de combinatie van een indien en een verwacht op een mock .. ?? Nu wekken we de indruk dat het uiteindelijk niet uitmaakt welke id er daar wordt doorgegeven maar eigenlijk is het belangrijk

-vhUnitTest.APPS.EC.Winkelkar.impl.WinkelkarService.VoegToeProduct.Test :

"Test: niets toevoegen: nog altijd 2 elementen op fake winkelkar (root en tekstlijn)"() --> nu wek je de indruk dat fake iets speciaal is . wat met een echte winkelkar / of is het wel degelijk speciaal ?

AssertEersteTweeElementenInWinkelkar() --> misschien kan je die hernoemen naar AssertVerplichteElementen en als er dan meerdere worden kunnen we die method hergebruiken , want misschien zal hij dan op termijn op een basis klasse komen etc... btw ik zou die ook privatiseren

-vhUnitTest.APPS.EC.Winkelkar.impl.WinkelkarService.VoegToeProduct.Test :

Test: 2 producten proberen toevoegen met gekend (gelijk) ID, geen pakketreferentie ->; moeten beide als nieuwe lijn op winkelkar -> lijkt mij iets beter dat je spreekt over een Fake1 en Fake2 ipv Fake en Fake2

[UST1862] DKG beslag: pakketreferentie in EDI omzetten in de nodige productlijnen

- De WinkelkarService gebruikt nu de winkelkarRepo en (nieuwe) ProductLijnRepo om gegevens te bewaren.

  1. … 4 more files in changeset.
[UST1862] DKG beslag: pakketreferentie in EDI omzetten in de nodige productlijnen

- Extra test voor twee gelijke producten

[UST1862] DKG beslag: pakketreferentie in EDI omzetten in de nodige productlijnen

- Assert van de lege winkelkar beetje aangepast

[UST1862] DKG beslag: pakketreferentie in EDI omzetten in de nodige productlijnen

- WinkelkarService.VoegToeProductLijn -> WinkelkarService.VoegToeProduct: move van de bijhorende test

  1. … 2 more files in changeset.