Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
[UST2865] - EPIC50 - Seefelder Personality aanmaken

- Cleanup overige methods -> ongedaan gemaakt blijkbaar zat daar een specialeke in -> ipv het probleem aan de bron aan te pakken -> property ID dat geen ID is ofwel property renamen ofwel van de property effectief een ID maken

    • -1
    • +25
    ./VerzendingMogelijkheidRepository.cls.xml
[UST2865] - EPIC50 - Seefelder Personality aanmaken

- VerzendingMogelijkheidRepository - ZoekViaOfferteId toegevoegd

- Cleanup overige methods

    • -22
    • +5
    ./VerzendingMogelijkheidRepository.cls.xml
[UST2860][rvJCL] Vh4: Handels: Orderbevestiging versturen naar Hein

- factories samengevoegd

[UST2860][rvJCL] Vh4: Handels: Orderbevestiging versturen naar Hein

- factories samengevoegd

    • -36
    • +0
    ./TransportFirmaBepalerFactory.cls.xml
    • -36
    • +0
    ./VerkoopprijsBerekenaarFactory.cls.xml
  1. … 27 more files in changeset.
[ICT] Dubbell code, super class bevat dezelfde code.
    • -20
    • +0
    ./BerekeningFoutenRepository.cls.xml
zie hierboven

zie hierboven

Is ook gecovered door Method "Test: BereidVoorBestelling plaatst de hoofding op de offerte en bepaalt de mogelijke leverdatums - standaard zijn er geen herberekende lijnen en meldingen aan de gebru...

Is ook gecovered door Method "Test: BereidVoorBestelling plaatst de hoofding op de offerte en bepaalt de mogelijke leverdatums - standaard zijn er geen herberekende lijnen en meldingen aan de gebruiker"()
Ik heb in ieder geval een test toegevoegd die controleert dat er geen verzendingmogelijkheden berekend worden voor standaard klanten.

aangezien deploy wordt er geen aanpassing doorgevoerd.

aangezien deploy wordt er geen aanpassing doorgevoerd.

  • More
  • CR-231
  • finished reviewing
  • More
  • CR-227
  • finished reviewing
Wat met klanten zonder transportkeuze?

Wat met klanten zonder transportkeuze?

Misschien beter om de testen hier op te splitsen per method in de Service en dan misschien hier en daar nog een testje bijgooien om wat randgevallen of true-false dingen af te toetsen

Misschien beter om de testen hier op te splitsen per method in de Service en dan misschien hier en daar nog een testje bijgooien om wat randgevallen of true-false dingen af te toetsen

ExactAantalKeer(1) om intentie duidelijk te maken

ExactAantalKeer(1) om intentie duidelijk te maken

Wat met klanten zonder transportkeuze? Of is dat niet relevant?

Wat met klanten zonder transportkeuze? Of is dat niet relevant?

Beter een ExactAantalKeer(0) op de MailAPIMock, om duidelijk de intentie te tonen

Beter een ExactAantalKeer(0) op de MailAPIMock, om duidelijk de intentie te tonen

beter de vhTest.Fake.TECH.Mail ?

beter de vhTest.Fake.TECH.Mail ?

Aangezien dit de clou is van de test, kunt ge er misschien beter een ExactAantalKeer(1) bijzetten?

Aangezien dit de clou is van de test, kunt ge er misschien beter een ExactAantalKeer(1) bijzetten?

testje voor 0 palletten toevoegen

testje voor 0 palletten toevoegen

$$$Not

$$$Not

[VRB430] Refactoring TransportBerekeningen.
[VRB430] Refactoring TransportBerekeningen.
[VRB430] Refactoring TransportBerekeningen.

- Paar kleine methods.

[UST2390] Vh4: Handels: Transportmogelijkheden doorgeven
[UST2390] Vh4: Handels: Transportmogelijkheden doorgeven
[UST2390] Vh4: Handels: Transportmogelijkheden doorgeven

- sessie naar tranpsort in APPS laag, wordt nu KlantIdentificatie.

    • -1
    • +1
    ./TransportFirmaBepalerFactory.cls.xml
    • -1
    • +1
    ./VerkoopprijsBerekenaarFactory.cls.xml
  1. … 23 more files in changeset.
[UST2390] Vh4: Handels: Transportmogelijkheden doorgeven

- hernoemt naar data.

    • -1
    • +1
    ./VerzendingMogelijkheidRepository.cls.xml
  1. … 12 more files in changeset.
[UST2390] Vh4: Handels: Transportmogelijkheden doorgeven

- tussentijdseccommit om casing weg te werken van Data naar data

  1. … 10 more files in changeset.
[UST2390] Vh4: Handels: Transportmogelijkheden doorgeven

- VerzendingMogelijkheden naar APPS laag bij transport.

    • -1
    • +1
    ./VerzendingMogelijkheidRepository.cls.xml
  1. … 6 more files in changeset.
[UST2390] Vh4: Handels: Transportmogelijkheden doorgeven

- BerekeningFouten naar APPS laag bij transport.

  1. … 6 more files in changeset.
[UST2390] Vh4: Handels: Transportmogelijkheden doorgeven

- Beetje refactoring dto's verplaatst naar transport.

  1. … 1 more file in changeset.