OrderIngaveService

Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
yep, ik zal een kaartje maken om een refactor door te voeren van deze klasse.

yep, ik zal een kaartje maken om een refactor door te voeren van deze klasse.

idd

idd

altijd handig om es echte data te visualiseren en repo te testen.

altijd handig om es echte data te visualiseren en repo te testen.

$LL ipv $L en liefst voluit $ListLength

$LL ipv $L en liefst voluit $ListLength

Is deze klasse nog nodig?

Is deze klasse nog nodig?

Groeperen we de constructor injections?

Groeperen we de constructor injections?

1. Er wordt niet getest of de LeverDatumBepaler opgeroepen wordt 2. De overvloed aan constructorparameters geeft aan dat functionaliteit afgezonderd kan worden: bv. WinkelkarLijnAanOrderToevoeger. ...

1. Er wordt niet getest of de LeverDatumBepaler opgeroepen wordt
2. De overvloed aan constructorparameters geeft aan dat functionaliteit afgezonderd kan worden: bv. WinkelkarLijnAanOrderToevoeger. (Hierdoor kun je 3 constructorparameters vervangen door 1 als je de LeverDatumBepaler meerekent). Ook de inhoud van deze method lijkt geschikt om af te zonderen.

Interface kan weggewerkt worden

Interface kan weggewerkt worden

[UST4023] SFS-LD : Leverdatums aanbieden op vhisie4
[UST4023] SFS-LD : Leverdatums aanbieden op vhisie4
[UST4023] SFS-LD : Leverdatums aanbieden op vhisie4

- config item implementatie

  1. … 1 more file in changeset.
[UST4023] SFS-LD : Leverdatums aanbieden op vhisie4

- Voorbereiding config item

[UST4023] SFS-LD : Leverdatums aanbieden op vhisie4

- Datumformaat fixed

[UST4023] SFS-LD : Leverdatums aanbieden op vhisie4

- Test + fix datum /offerte melding.

  1. … 1 more file in changeset.
[UST4023] SFS-LD : Leverdatums aanbieden op vhisie4

- Test + fix

  1. … 3 more files in changeset.
[UST4023] SFS-LD : Leverdatums aanbieden op vhisie4

- Info van SFS. Zij sturen bij orderconfirmation een pickupdate terug, zo moeten wij aan cachekant bij maakordervanofferte, geen conversie doen voor KlantSFS.

- Voor klant van SFS, dus personality SFS is de wijziging nog niet weg. Normaal gezien wordt er nooit een order gemaakt van een offerte met personlaity SFS, maar in geval van, dan heeft de klant steeds te maken met leverdatum en niet met pickup datums.

[UST4023] SFS-LD : Leverdatums aanbieden op vhisie4

- Voor SFS personality

[UST4023] SFS-LD : Leverdatums aanbieden op vhisie4

- Foutmelding indien een ongeldige leverdatum ingevoerd is voor SFS.

[UST4023] SFS-LD : Leverdatums aanbieden op vhisie4

- nog geen functionele wijzigingen.

[UST4023] SFS-LD : Leverdatums aanbieden op vhisie4

- Offerte meegeven want we hebben ook klant nodig.

  1. … 1 more file in changeset.
[UST4023] SFS-LD : Leverdatums aanbieden op vhisie4

- voorbereiding omzetten van leverdatum SFS naar pickup datum.

    • -0
    • +15
    ./LeverDatumBepaler.cls.xml
[UST3920] EC: app: herbestel bij probleem: DI1 en nog niet verwerken
[UST3920] EC: app: herbestel bij probleem: DI1 en nog niet verwerken
[UST3920] EC: app: herbestel bij probleem: DI1 en nog niet verwerken

- Afzonderen en test van code die leveradres aanpast bij maak order van offerte obv winkelkar met herbestellingbijprobleem

    • -0
    • +41
    ./LeverAdresAanpasserBijHerbestellingBijProbleem.cls.xml
  1. … 4 more files in changeset.
[UST2711r] Vh4: Geen leverdatum-tekstlijn indien deelleveringen gekozen

- Dus niet meer tonen als er geen datum ingevuld is.

  • More
  • CR-431
  • finished reviewing
Ik zou hier opteren voor enkel de If $Length(Offerte.GeefLeverDatum()) dan ..

Ik zou hier opteren voor enkel de If $Length(Offerte.GeefLeverDatum()) dan ..