CacheAdminA_trunk2010/vhUnitTest/APPS/MAG/UGLYPicking

TestUglyPickingService.cls.xml

Checkout Tools

Line History

Line Count Graph
Line Count Graph

Stats

Commits this week: 0
Total Committers: 3
Last Commit: 25 Aug 20
Commits this week: 0
Total Lines of Code (LoC): 652
Net change in LoC this week: 0

Commit Activity

Commits By Day In Week
52 week commits volume
Commits By Day In Week
Commits by day
Commits By Day In Week
Commits by hour
Commits By Hour In Day
Commit calendar
 

Most active committers (90 days)

loading
loading
  • last updated a few seconds ago
All Commits Reviews
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Friday 22 Nov 2019
11:21 am

Jo Willems

Thursday 21 Nov 2019
10:06 am

Wim Vermeulen

10:01 am

Wim Vermeulen

Idem voor de cleanup van de UTDataVoorKvik --> aparte method Hier zelfs geen parameters nodig, denk.

Idem voor de cleanup van de UTDataVoorKvik --> aparte method
Hier zelfs geen parameters nodig, denk.

9:56 am

Wim Vermeulen

Ik heb lang gezocht naar het verschil in setup tussen deze method en de method erboven. Door de setup (de lijnen hierboven) in een aparte private method te zetten wordt het onderscheid meteen duide...

Ik heb lang gezocht naar het verschil in setup tussen deze method en de method erboven.
Door de setup (de lijnen hierboven) in een aparte private method te zetten wordt het onderscheid meteen duidelijk:

bvb. Do ..SetUpUTDataVoorKvik ( PalletIdVoorEWMS, PalletIdVoorEWPAL )

En hierdoor worden de 2 tests meteen een pak overzichtelijker

9:42 am

Wim Vermeulen

MethodNaam volgens vh-conventies : ZetOpEWPAL() en KuisOpEWPAL() en ZetOpEWMSData()

MethodNaam volgens vh-conventies : ZetOpEWPAL() en KuisOpEWPAL() en ZetOpEWMSData()

Monday 18 Nov 2019
5:04 pm

Kurt Callebaut

Thursday 10 Oct 2019
3:40 pm

Jo Willems

Als er palletten meegeven zijn, en we vinden niet wat we zoeken, dan stoppen we. Als er geen palletten meegegeven zijn moeten we verder. (fallback) de test op Count is dus essentieel. en dient niet...

Als er palletten meegeven zijn, en we vinden niet wat we zoeken, dan stoppen we.
Als er geen palletten meegegeven zijn moeten we verder. (fallback)
de test op Count is dus essentieel. en dient niet om de 0 te detecteren, maar om niet 0 te detecteren.

1:33 pm

Wim Vermeulen

OPGELET: Tracht zoveel mogelijk de code van deze klasse tussen caché v5 en cache2010 in sync te houden. Net voor uw wijzigingen waren deze methods redelijk goed in sync (behalve method FPalletsPerP...

OPGELET:
Tracht zoveel mogelijk de code van deze klasse tussen caché v5 en cache2010 in sync te houden.
Net voor uw wijzigingen waren deze methods redelijk goed in sync (behalve method FPalletsPerProdNr() die is ge-refactored).
Na uw wijzigingen dreigt de code uiteen te lopen en wordt het steeds lastiger voor iemand anders om nog toe te voegen, of erger nog: om te troubleshooten!

Maak er desnoods een apart kaartje van. Of overleg met Pieter en/of met mij

12:59 pm

Wim Vermeulen

Early quit :-O Gelukkig gaat ie binnenkort wel weg.

Early quit :-O
Gelukkig gaat ie binnenkort wel weg.

12:56 pm

Wim Vermeulen

Als reviewer ben ik verplicht om dit aan te geven : embedded sql aanpassen : beter : wegwerken en vervangen door TECH.Resultset reden: gedrag is beter onder controle bij errors, early quits, onvoo...

Als reviewer ben ik verplicht om dit aan te geven :

embedded sql aanpassen :
beter : wegwerken en vervangen door TECH.Resultset
reden: gedrag is beter onder controle bij errors, early quits, onvoorspelbaar gedrag indien 2 sql's gemixed worden (lees op andere stacklevel een andere &sql doorlopen), compile-issues.

In dit specifiek geval zou ik misschien toch ook de &sql behouden.

12:53 pm

Wim Vermeulen

Zeer goed dat je deze lijn erbij zet! http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif

Zeer goed dat je deze lijn erbij zet!

11:56 am

Wim Vermeulen

Count() is niet nodig, want als Count=0 , dan zal Find() ook niks vinden.

Count() is niet nodig, want als Count=0 , dan zal Find() ook niks vinden.

Wednesday 09 Oct 2019
4:37 pm

Jo Willems

4:36 pm

Jo Willems

[ICT778] MAG: Picking: juiste exemplaren picken (voor het juiste order)
[ICT778] MAG: Picking: juiste exemplaren picken (voor het juiste order)