Activity CacheAdminA_trunk2010/DOM

2:59 pm

Tom Vermeulen

More
Re: CR-2753
commented on review

geen testen voor geschreven?

11:14 am

Tom Vermeulen

More
Re: CR-2756
commented on review

zelfde opmerking als bij andere review: check zeker performantie van deze query. Btw: in Caché kunt ge ook de "[" operator gebruiker (='contains'), dan moet ge niet met die '%' werken (idem bij de...

zelfde opmerking als bij andere review: check zeker performantie van deze query. Btw: in Caché kunt ge ook de "[" operator gebruiker (='contains'), dan moet ge niet met die '%' werken

(idem bij de andere queries natuurlijk)

10:55 am

Tom Vermeulen

More
CR-2755
finished reviewing

10:54 am

Tom Vermeulen

More
Re: CR-2755
commented on review

gaat deze zoek nog even snel gaan? een "=" gaat indexed zoeken, maar een "like" gaat mogelijks natural door de data. Het gaat hier wel om een tabel met enkele miljoenen records. Zeker dubbelchecken...

gaat deze zoek nog even snel gaan? een "=" gaat indexed zoeken, maar een "like" gaat mogelijks natural door de data. Het gaat hier wel om een tabel met enkele miljoenen records. Zeker dubbelchecken! (idem voor de andere queries natuurlijk)

10:52 am

Tom Vermeulen

More
Re: CR-2755
commented on review

Gisteren hier nog even met WV over gebabbeld en we zouden hier toch 'adviseren' om de grotere private methods in (een) aparte helperklasse(s) te steken, elk met hun eigen testen. Op die manier zijn...

Gisteren hier nog even met WV over gebabbeld en we zouden hier toch 'adviseren' om de grotere private methods in (een) aparte helperklasse(s) te steken, elk met hun eigen testen. Op die manier zijn de verschillende codepaden beter te testen (ipv sommige testen in de UT van deze klasse)
Dit valt onder het "Single Responsibility" principe. Ik ga dat hier niet allemaal verder uittypen, we praten of bellen wel

10:49 am

Tom Vermeulen

More
Re: CR-2755
commented on review

opeetcode!!! zeker aanpassen

3:11 pm

Joannes Lemmens

More
CR-2759
Abandoned

3:10 pm

Joannes Lemmens

More
CR-2758
Abandoned

3:10 pm

Joannes Lemmens

More
CR-2757
Abandoned

3:08 pm

Joannes Lemmens

More
CR-2756
started review

[ICT-4237] Manco: DnaCodeRepository uitbreiden voor A/B

3:07 pm

Joannes Lemmens

More
CR-2755
started review

[ICT-4242] Manco: DnaCodeService uitbreiding voor A/B

3:04 pm

Joannes Lemmens

More
CR-2753
started review

[ICT-4657] Manco: A/B tonen op documenten levbon en pakbon

12:56 pm

Peter Vercoutter

More
Re: CR-2749
commented on review

Dit is niet oke. ProductData is iets anders dan de opgeslagen vhConfigs. ProductData gaat over KPR, PRBS, ... en al die bazaar. Die moet wel degelijk nog worden verwijderd na iedere test.

11:42 am

Jelle Baele

11:42 am

Jelle Baele

More
CR-2749
started review

[ICT-4441] [rvPVR] AansturingActies via BOMBOL Encoway:

11:35 am

Jelle Baele

More
CR-2743
finished reviewing

11:34 am

Jelle Baele

More
Re: CR-2743
commented on review

$J zou ik vervangen door '$Job'?

10:03 am

Jelle Baele

More
Re: CR-2743
commented on review

Ergens geeft dit nu wel een verkeerd beeld van deze method; ik zou ervan uitgaan dat het productid enkel wordt bepaald, maar als je dan in de method gaat kijken zie je dat hier ook de vhConfig word...

Ergens geeft dit nu wel een verkeerd beeld van deze method; ik zou ervan uitgaan dat het productid enkel wordt bepaald, maar als je dan in de method gaat kijken zie je dat hier ook de vhConfig wordt opgeslagen

9:43 am