Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
[ICT-2055] [rvTVE] Proboxinhoudetiket samenvoegen in caché

- Review opmerkingen

  1. … 2 more files in changeset.
de %OnNew bovenaan zetten (maakt functioneel niets uit, is conventie. Heeft het voordeel dat je bij openen van klasse direct ziet wat er mogelijks geïnjecteerd wordt)

de %OnNew bovenaan zetten (maakt functioneel niets uit, is conventie. Heeft het voordeel dat je bij openen van klasse direct ziet wat er mogelijks geïnjecteerd wordt)

casing in deze method en die hieronder desnoods nog gelijktrekken

casing in deze method en die hieronder desnoods nog gelijktrekken

Do ipv do

Do ipv do

waarom is deze verwijderd?

waarom is deze verwijderd?

[ICT-2055] [rvTVE] Proboxinhoudetiket samenvoegen in caché
[ICT-2055] [rvTVE] Proboxinhoudetiket samenvoegen in caché
[ICT-2055] [rvTVE] Proboxinhoudetiket samenvoegen in caché

- Methodcall verwijderen van standaardOrder

  1. … 1 more file in changeset.
no_comment
  1. … 3 more files in changeset.
Telegramstijl, omdat ze anders te lang worden en de variaties nu eenmaal soms ingewikkeld zijnhttp://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/wink.gif

Telegramstijl, omdat ze anders te lang worden en de variaties nu eenmaal soms ingewikkeld zijn

Idem reply hierboven.

Idem reply hierboven.

Zoals besproken... Dit ging over de property-namen, maar... toch liever niet. In de properties kunnen soms mocks, soms stubs gestoken worden, afhankelijk van wat de UnitTest nodig heeft.

Zoals besproken... Dit ging over de property-namen, maar... toch liever niet. In de properties kunnen soms mocks, soms stubs gestoken worden, afhankelijk van wat de UnitTest nodig heeft.

Done!

Done!

Done

Done

Done! (smartass http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/wink.gif )

Done! (smartass )

Done!

Done!

Done, maar staat momenteel al nieuwe openstaande wijziging op die klasse, die nog niet gecommit mag worden, dus komt daarna wel.

Done, maar staat momenteel al nieuwe openstaande wijziging op die klasse, die nog niet gecommit mag worden, dus komt daarna wel.

Done, maar staat momenteel al nieuwe openstaande wijziging op die klasse, die nog niet gecommit mag worden, dus komt daarna wel.

Done, maar staat momenteel al nieuwe openstaande wijziging op die klasse, die nog niet gecommit mag worden, dus komt daarna wel.

Done en done en done! http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif

Done en done en done!

Normaal binnen de scope blijven, ik weet het, maar ik dacht de while op deze manier zo proper en leesbaar mogelijk te houden. Uiteindelijk wordt elk van die dim'ekes toch in die while gebruikt en o...

Normaal binnen de scope blijven, ik weet het, maar ik dacht de while op deze manier zo proper en leesbaar mogelijk te houden. Uiteindelijk wordt elk van die dim'ekes toch in die while gebruikt en omhelst die while de volledige method.

Denk het niet. Nu krijgt ge alle OrderLijnIterators netjes op een rijtje via code completion. Eentje is dan de volledige, sommige zijn "via" dit, andere "via" dat, ...

Denk het niet. Nu krijgt ge alle OrderLijnIterators netjes op een rijtje via code completion. Eentje is dan de volledige, sommige zijn "via" dit, andere "via" dat, ...

Had ik al aangepast inmiddels, zit al in volgende review http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/wink.gif

Had ik al aangepast inmiddels, zit al in volgende review

test namen zeggen niet duidelijk wat er getest wordt.

test namen zeggen niet duidelijk wat er getest wordt.

(Is bestaande code natuurlijk) hoeft geen lokale var te zijn. mag rechtstreeks op ..ProductService(Mock)

(Is bestaande code natuurlijk) hoeft geen lokale var te zijn. mag rechtstreeks op ..ProductService(Mock)

Refact nodig zodat namen van bestaande code ook ProductServiceMock worden, etc..

Refact nodig zodat namen van bestaande code ook ProductServiceMock worden, etc..

in deze situatie gebruiken we veelal if $$asserttrue(.. hasnext){ assert.. next() } omdat de .next hiervan afhankelijk is. ..

in deze situatie gebruiken we veelal
if $$asserttrue(.. hasnext)

Unknown macro: { assert.. next() }


omdat de .next hiervan afhankelijk is. ..