Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Done!

Done!

Is overgenomen van de defaulting van de (énige) implementatieklasse. De mock wordt van de interface gemaakt. Dat maakte dat die ander gedrag vertoonde dan de echte.

Is overgenomen van de defaulting van de (énige) implementatieklasse. De mock wordt van de interface gemaakt. Dat maakte dat die ander gedrag vertoonde dan de echte.

Die worden allebei in de caller van deze method al uitgesloten

Die worden allebei in de caller van deze method al uitgesloten

object LadeOnderdeel doorgeven, i.p.v. Rol As enu dan moet je voor Bodem niet opnieuw de "..MaakLosseComponent(Kenmerken) " opnieuw oproepen --> vermijd dubbel werk.

object LadeOnderdeel doorgeven, i.p.v. Rol As enu
dan moet je voor Bodem niet opnieuw de "..MaakLosseComponent(Kenmerken) " opnieuw oproepen --> vermijd dubbel werk.

Dit is de algemene MaatwerkCalcAPI --> straf dat je hier de parameter gaat defaulten naar klant "Halux". Ik zou verwachten dat je ergens een specifieke oproep hebt, waar je dan gecontroleerd de kla...

Dit is de algemene MaatwerkCalcAPI --> straf dat je hier de parameter gaat defaulten naar klant "Halux".
Ik zou verwachten dat je ergens een specifieke oproep hebt, waar je dan gecontroleerd de klant meegeeft.

Laat voorlopig maar staan, maar hou er wel rekening mee, dat dit soort defaultings "gedurfd" is

De bijhorende UT geeft aan dat *ID = "" geeft "" *ID = 0 geeft "" Mijn vraag : is dit wenselijk, of zou het beter zijn om een exceptie te smijten? Want voor deze waarden is het eigenlijk zinloos ...

De bijhorende UT geeft aan dat

  • ID = "" geeft ""
  • ID = 0 geeft ""
    Mijn vraag : is dit wenselijk, of zou het beter zijn om een exceptie te smijten?
    Want voor deze waarden is het eigenlijk zinloos om de bepaler op te roepen.
Deze lijn zou ik binnen de Else zetten. Nu wordt ze uitgevoerd, zelfs bij ID > 1332 en nadien wordt dan toch exception gesmeten. M.a.w. de AflfaKey() uitvoeren was niet nodig.

Deze lijn zou ik binnen de Else zetten.
Nu wordt ze uitgevoerd, zelfs bij ID > 1332 en nadien wordt dan toch exception gesmeten. M.a.w. de AflfaKey() uitvoeren was niet nodig.

[UST4166] TAX-LC-002: Nieuwe verpakking (betere palletstapeling) - deel 1: calculatie kosten:...
[UST4166] TAX-LC-002: Nieuwe verpakking (betere palletstapeling) - deel 1: calculatie kosten:...
[UST4166] TAX-LC-002: Nieuwe verpakking (betere palletstapeling) - deel 1: calculatie kosten: MinimumBestelhoeveelheid:

- BestelEenheidLosseComponentenInBulkBepaler => Terughalen van oude implementatie voor BestelEenheidVoorBodem

    • -0
    • +40
    ./HFVerpakkingv001/LosseComponenten/DoosAfmetingenBodemBepaler.cls.xml
  1. … 5 more files in changeset.
[UST4166] TAX-LC-002: Nieuwe verpakking (betere palletstapeling) - deel 1: calculatie kosten: MinimumBestelhoeveelheid:

- Aanpassing dependencies na Move package implementatie vanuit zTryout => DOM

  1. … 13 more files in changeset.
[UST4166] TAX-LC-002: Nieuwe verpakking (betere palletstapeling) - deel 1: calculatie kosten: MinimumBestelhoeveelheid:

- Move class Mock vanuit zTryout => DOM

  1. … 1 more file in changeset.
[UST3970] TAX diepteverstelling op pickinglijst losse onderdelen:

- Niet onbelangrijk => is DiepteVerstelling wel aanwezig

    • -2
    • +2
    ./HFSetSchroeven/ConditieDiepteVerstellingSchroefBepaler.cls.xml
  1. … 5 more files in changeset.
idem als in de bodemondersteuning

idem als in de bodemondersteuning

als ge toch '> 0' doet, kunt ge evengoed de boolean overslaan http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif

als ge toch '> 0' doet, kunt ge evengoed de boolean overslaan

[UST3970] TAX diepteverstelling op pickinglijst losse onderdelen:

- ConditieBodemOndersteuningSchroef en ConditieKoppelingSchroef en ConditieDiepteVerstellingSchroef afgezonderd naar hun respectievelijke Bepalers met elk een eigen testcase

- Bovengenoemde Bepalers geinjecteerd in constructor van TAOR.Builder.HFSetSchroevenv000 voor testcase BuilderMoetWerken

- Nieuwe testcase BuilderMoetWerken => check op resultaat bovengenoemde condities => via mocks

- Oude en foute testcase-opzet voor BuilderMoetWerken verwijderd => nogal onleesbare opzet en bovendien niet SOLID => verweven verantwoordelijkheden

    • -0
    • +20
    ./HFSetSchroeven/ConditieKoppelingSchroefBepaler.cls.xml
    • -0
    • +20
    ./HFSetSchroeven/ConditieDiepteVerstellingSchroefBepaler.cls.xml
    • -0
    • +20
    ./HFSetSchroeven/ConditieBodemOndersteuningSchroefBepaler.cls.xml
  1. … 18 more files in changeset.
Best hier een paar puntjes voorzetten, ander gaat ge wat vonken krijgen op het bord http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif

Best hier een paar puntjes voorzetten, ander gaat ge wat vonken krijgen op het bord

in apart klaske met zijn eigen testjes steken. veiliger voor de toekomst.

in apart klaske met zijn eigen testjes steken. veiliger voor de toekomst.

[UST3970] TAX diepteverstelling op pickinglijst losse onderdelen:
[UST3970] TAX diepteverstelling op pickinglijst losse onderdelen: