LBX productie_006_LBX_OnderdelenPickingLijst.txt

Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
[ICT - 1341] Aansturingsactie bestelbon verwijderen voor TBX TAX en LBX

- vhTestFiles aangepast voor LBX

    • --6
    • +0
    ./LBX productie_006_LBX_OnderdelenPickingLijst.txt
  1. … 225 more files in changeset.
[ICT-853] fix - sgeleider (falende test na wijzigingen in kenmerkeneditor.)

- terug activeren.

    • -4
    • +2
    ./LBX productie_006_LBX_OnderdelenPickingLijst.txt
  1. … 10 more files in changeset.
ICT-853: Opkuis sgeleider wijzigingen, na falende test
ICT-853: Opkuis sgeleider wijzigingen, na falende test
[ICT-853] fix - sgeleider (falende test na wijzigingen in kenmerkeneditor.)
    • -2
    • +4
    ./LBX productie_006_LBX_OnderdelenPickingLijst.txt
  1. … 9 more files in changeset.
Het gebruik van #Delimiter is min of meer oké. Voor den "\" is het algemeen aanvaard om deze letterlijk in de code te zetten, dus hierop had ik als reviewer niet geprottesteerd als die zou blijven ...

Het gebruik van #Delimiter is min of meer oké.
Voor den "\" is het algemeen aanvaard om deze letterlijk in de code te zetten, dus hierop had ik als reviewer niet geprottesteerd als die zou blijven staan.

Wel te vervangen bij refactoring is de variabele D door "\" (hardcoded, één van de weinige uitzonderingen waar dit oké is )

$$$Not( Var = "" ) graag leesbaarder maken door $$$HasLength( Var ) te gebruiken Liefst zoveel mogelijk de rechtstreekse expressies gebruiken. Zo ook : (Var = "") i.p.v. $Length( Var ) = 0 deze la...

$$$Not( Var = "" )
graag leesbaarder maken door $$$HasLength( Var ) te gebruiken

Liefst zoveel mogelijk de rechtstreekse expressies gebruiken.
Zo ook :
(Var = "") i.p.v. $Length( Var ) = 0
deze laatste is hier wel in orde

[VRB633] HLX: (1) LBX Probox pickinglijst - kolom Control weg

- Dit is al een correcte output.

- nog te bekijken voor refactor.

    • -6
    • +0
    ./LBX productie_006_LBX_OnderdelenPickingLijst.txt
  1. … 5 more files in changeset.
[VRB633] HLX: (1) LBX Probox pickinglijst - kolom Control weg
[VRB633] HLX: (1) LBX Probox pickinglijst - kolom Control weg
[VRB633] HLX: (1) LBX Probox pickinglijst - kolom Control weg

Testresultaten

    • -19
    • +19
    ./LBX productie_006_LBX_OnderdelenPickingLijst.txt
  1. … 10 more files in changeset.
Is exact dezeflde implementatie als de vorige method, behalve ..#ProductId1. Ik heb al een zeer geslaagde poging gedaan om de volledige UT te herschrijven. Misschien een goede gelegenheid om er in...

Is exact dezeflde implementatie als de vorige method, behalve ..#ProductId1.

Ik heb al een zeer geslaagde poging gedaan om de volledige UT te herschrijven.
Misschien een goede gelegenheid om er in pair-programming door te lopen.
Ik zal iets inboeken in uw agenda.

#dim ... As DOM.PM.enu...

#dim ... As DOM.PM.enu...

beter: Array.Count() i.p.v. Object.Count() idem Verwacht...

beter: Array.Count() i.p.v. Object.Count()

idem Verwacht...

Gewoon ter info: Deze test is een mooi voorbeeld, want hier zie beide gevallen naast (onder) elkaar : TOB 270mm is nog steeds de "oude" geleider TOB 300mm is de nieuwe S-geleider, dus de technolog...

Gewoon ter info:

Deze test is een mooi voorbeeld, want hier zie beide gevallen naast (onder) elkaar :
TOB 270mm is nog steeds de "oude" geleider
TOB 300mm is de nieuwe S-geleider, dus de technologie is nu weggelaten.

Daarom is het nice als een test in één oogopslag beide scenario's verifieert

Bij nader inzien staat er nu een dubbele negatie, want de laatste lijn van de functie is ook Quit $$$Not( ... ) Dat is vermoeiend om te begrijpen http://subversion02.vanhoecke.be/static/ogdo0b/2sta...

Bij nader inzien staat er nu een dubbele negatie, want de laatste lijn van de functie is ook Quit $$$Not( ... )
Dat is vermoeiend om te begrijpen

Dus als we dit omdraaien naar ..IsHalffabEenSGeleiderVoorBlumotionEnTOB()
dan kunnen we 2x "$$$Not" weglaten.
en bovendien geeft de methodnaam dan veel specifieker aan wat er getest wordt.

Dus als je akkoord bent, deze naam wijzigen in ..IsHalffabEenSGeleiderVoorBlumotionEnTOB() en onderaan de $$$Not() wegdoen.

Dus als je akkoord bent, deze naam wijzigen in ..IsHalffabEenSGeleiderVoorBlumotionEnTOB()
en onderaan de $$$Not() wegdoen.

hier dus http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif

hier dus

[VRB474] LBX S-geleider: probox onderdelenpickingblad: samentellen; geleidertechnologie niet opnemen...
[VRB474] LBX S-geleider: probox onderdelenpickingblad: samentellen; geleidertechnologie niet opnemen...
[VRB474] LBX S-geleider: probox onderdelenpickingblad: samentellen; geleidertechnologie niet opnemen in omschrijving:

- Nieuwe bestanden integratietests => omschrijvingen => geen BLUMOTION en TIP-ON BLUMOTION meer

    • -2
    • +2
    ./LBX productie_006_LBX_OnderdelenPickingLijst.txt
  1. … 8 more files in changeset.
[UW443] PM heeft nieuwe LBX-geleiders toegevoegd - Testresultaten aanpassen
[UW443] PM heeft nieuwe LBX-geleiders toegevoegd - Testresultaten aanpassen
[UW443] PM heeft nieuwe LBX-geleiders toegevoegd - Testresultaten aanpassen

- data aangepast door PM : 753.5001S K LI/RE en andere varianten 750 en LD:400 etc.

    • -2
    • +2
    ./LBX productie_006_LBX_OnderdelenPickingLijst.txt
  1. … 29 more files in changeset.
DocumentAfdrukker.cls : aantal tekens van 70 naar 68 (openstaande change op Cache01 sinds maanden) - TestFiles LBX
    • -3
    • +3
    ./LBX productie_006_LBX_OnderdelenPickingLijst.txt
  1. … 97 more files in changeset.