Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
[DEF736] Vhisie4 productiedagen die leverdatums bepalen moet afhankelijk zijn van klantnummer. (wordt...
[DEF736] Vhisie4 productiedagen die leverdatums bepalen moet afhankelijk zijn van klantnummer. (wordt...
[DEF736] Vhisie4 productiedagen die leverdatums bepalen moet afhankelijk zijn van klantnummer. (wordt verkeerdelijk klantid meegegeven.

- aanmaken van verzendwijze mag ook naar verzendwijzebepaler.

- de if-kes nog te refactoren, want het wordt al een lange method vol ifkes. :)

    • -14
    • +2
    ./MogelijkeLeverdatumsBepaler.cls.xml
  1. … 4 more files in changeset.
[UST4023] SFS-LD : Leverdatums aanbieden op vhisie4

- review opm/ verwerken.

  1. … 1 more file in changeset.
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

- Voor SFS personality

[UST4023] SFS-LD : Leverdatums aanbieden op vhisie4

- refact, voorbereiding op aanleveren van leverdatums voor SFS

[UST4023] SFS-LD : Leverdatums aanbieden op vhisie4

- refact, voorbereiding op aanleveren van leverdatums voor SFS

[UST4023] SFS-LD : Leverdatums aanbieden op vhisie4

- refact, voorbereiding op aanleveren van leverdatums

    • -2
    • +11
    ./MogelijkeLeverdatumsBepaler.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

- Maken dat de verzendwijzebepaler zeker opgeroepen wordt indien HerbestellingBijProbleem op winkelkar

  1. … 1 more file in changeset.
inderdaad.. gecopypasted http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif

inderdaad.. gecopypasted

  • More
  • CR-460
  • finished reviewing
In de nieuw toegevoegde tests heb je het over de gebruikte verzendwijze, je assert echter nergens op deze verzendwijze. Het al dan niet asap mogelijk zijn staat hier volgens mij los van. Gesuggeree...

In de nieuw toegevoegde tests heb je het over de gebruikte verzendwijze, je assert echter nergens op deze verzendwijze. Het al dan niet asap mogelijk zijn staat hier volgens mij los van. Gesuggereerde oplossing: tests toevoegen die doen wat de testomschrijving zegt, of mss korter: de testomschrijving aanpassen (à la "standaard verzendwijze van de winkelkar wordt opgehaald bij leeg leveradres")