Activity CacheAdminA_trunk2010/WSimpl/Halux/AAP/DnaCodeService.cls.xml

8:28 am

Joannes Lemmens

More
Re: CR-2829
replied to Tom Vermeulen

Er zijn geen kleine kartonnen pallet, dit stukje is gewijzigd naar 'Set PalletTekst = "Karton"' (ook verkort sinds die kolom niet zo breed is)

7:36 am

Tom Vermeulen

More
Re: CR-2829
commented on review

spatie voor de K? Hebben we ook kleine kartonnen palletten? Ik vraag het maar omdat ..BepaalGroottePallet "klein" en "groot" kan teruggeven, maar ik heb nog nooit een kleine kartonnen pallet gezie...

spatie voor de K?

Hebben we ook kleine kartonnen palletten? Ik vraag het maar omdat ..BepaalGroottePallet "klein" en "groot" kan teruggeven, maar ik heb nog nooit een kleine kartonnen pallet gezien hier. Ik kan hier natuurlijk ook compleet verkeerd zijn.

1:58 pm

Joannes Lemmens

More
CR-2829
started review

[ICT-4991] Implementatie voor nieuwe KlantInstelling KartonnenPallet

4:17 pm

Joannes Lemmens

More
CR-2797
Closed

4:10 pm

Joannes Lemmens

More
Re: CR-2797
replied to Tom Vermeulen

De lijn hieronder is een uitbreiding op deze en omvat deze case volledig + de A/B case.

2:27 pm

Tom Vermeulen

More
Re: CR-2797
commented on review

gaat dit nooit meer $$$True mogen zijn (geen idee hoor, just wondering)?

2:23 pm

Tom Vermeulen

More
Re: CR-2797
commented on review

dé moment om te laten zien dat ik wel naar gekeken heb: 'Do' ipv 'do' (hieronder nog ergens)

12:51 pm

Joannes Lemmens

More
Re: CR-2797
commented on review

Hierin zit dus ook het verwijderen van alles wat met LPN-code te maken heet (aan de Caché-kant). De tests zouden hopelijk duidelijk moeten maken wat de PalletIDBepaler allemaal moet doen.

12:50 pm

Joannes Lemmens

More
CR-2797
started review

[ICT-4246] Manco: Backend ProScan

2:46 pm

Joannes Lemmens

More
CR-2755
Closed

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