CacheAdminA_trunk/BL/MB/UGLYPicking

OrderReservatie.cls.xml

Checkout Tools

Line History

Line Count Graph
Line Count Graph

Stats

Commits this week: 0
Total Committers: 4
Last Commit: 05 Jul 22
Commits this week: 0
Total Lines of Code (LoC): 906
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
Monday 01 Aug 2022
8:32 am

Jo Willems

Tuesday 05 Jul 2022
10:33 am

Jo Willems

[ICT-2483]Kvik-workaround voor andere klanten

- Toevoegen van OL Beslag aan het lijstje : zij verkopen de KVIK spacetowers in Denemarken aan KVIK Denemarken.

Options
    • -5
    • +6
    ./OrderReservatie.cls.xml
Wednesday 22 Dec 2021
11:26 am

Jo Willems

[ICT-2483]Kvik-workaround voor andere klanten

- Code werd niet uitgevoerd omdat de Do ontbrak :-( Gelukkig heb ik de fout zelf gevonden

Options
    • -3
    • +3
    ./OrderReservatie.cls.xml
Friday 10 Dec 2021
4:20 pm

Wim Vermeulen

4:17 pm

Wim Vermeulen

De lijst van de klantnummers is nu duplicate code. Kunnen we afspreken dat bij een volgende uitbreiding, dit zal ge-refactored worden. Dan krijg je gratis een method met een leesbare naam, wat de b...

De lijst van de klantnummers is nu duplicate code.
Kunnen we afspreken dat bij een volgende uitbreiding, dit zal ge-refactored worden. Dan krijg je gratis een method met een leesbare naam, wat de begrijpbaarheid van de code verhoogd.
Bijkomende optionele uitbreiding is om dit in Data te steken, dan is het tevens gedeeld met cache05, wat dan weer enkele aanpassingen uitspaart (misschien :-P )

Op dit moment is de code-change aanvaard.

3:56 pm

Wim Vermeulen

Lijst van klantnummers : zie opmerking hieronder

Lijst van klantnummers : zie opmerking hieronder

Thursday 09 Dec 2021
3:34 pm

Jo Willems

[ICT-2483]Kvik-workaround voor andere klanten
[ICT-2483]Kvik-workaround voor andere klanten
3:31 pm

Jo Willems

11:54 am

Jo Willems

[ICT-2483]Kvik-workaround voor andere klanten

- onderscheid specifieker maken, HAL of TBX laat bvb ook kader deuren door.

- wijziging in PRODGRP^PRODUKT op aanraden van PV en zou vergeen impact mogen hebben.

Options
    • -1
    • +1
    ./OrderReservatie.cls.xml
  2. … 1 more file in changeset.
Wednesday 08 Dec 2021
11:22 am

Jo Willems

[ICT-2483]Kvik-workaround voor andere klanten

- Er was nog een probleem met niet maatwerk op bons met ook maatwerk die gepicked moesten worden. Er komt een lijst palletten voor het maatwerk, maar niet voor het beslag, en dat geraakt dus niet gepicked.

- dit gebeurde niet bij KVIK, omdat die alleen maatwerk bestellen

- maken dat er wel een fallback is voor niet maatwerk.

Options
    • -2
    • +4
    ./OrderReservatie.cls.xml
Friday 08 Oct 2021
3:10 pm

Jo Willems

[ICT-2483]Kvik-workaround voor andere klanten

- Equals ipv Contains gebruiken.(bis)

Options
    • -1
    • +1
    ./OrderReservatie.cls.xml
10:47 am

Jo Willems

[ICT-2483]Kvik-workaround voor andere klanten

- Equals ipv Contains gebruiken.

Options
    • -1
    • +1
    ./OrderReservatie.cls.xml
Wednesday 06 Oct 2021
5:49 pm

Jo Willems

[ICT-2483]Kvik-workaround voor andere klanten

- klantnummers toevoegen.

Options
    • -8
    • +6
    ./OrderReservatie.cls.xml
Monday 26 Jul 2021
10:10 am

Jo Willems

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

- na te veel problemen : geen fallback meer voor KVIK, dus alleen op basis van EWPAL (paletten voor order) en niet meer paletten met product van mogelijk verkeerd order.

Options
    • -3
    • +6
    ./OrderReservatie.cls.xml
Monday 06 Jan 2020
10:33 am

Jo Willems

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

- log msg duidelijker

Options
    • -1
    • +1
    ./OrderReservatie.cls.xml
10:24 am

Jo Willems

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

- Indien gelukte fallback: indien nodig mail naar ict_meldingen om NRA en RDL te verwittigen

- indien nodig wil zeggen : reservatie voor maatwerk, en meer dan één toelevering voor desbetreffend product de voorbije maand

Options
    • -0
    • +30
    ./OrderReservatie.cls.xml
Monday 25 Nov 2019
10:53 am

Jo Willems

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

- Er zijn 2 kviks...

Options
    • -1
    • +1
    ./OrderReservatie.cls.xml
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