CacheAdminA_trunk2010

EWPAL2.mac.rou

Checkout Tools

Line History

Line Count Graph
Line Count Graph

Stats

Commits this week: 0
Total Committers: 4
Last Commit: 29 May 24
Commits this week: 0
Total Lines of Code (LoC): 15
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
Wednesday 29 May 2024
4:54 pm

Tom Vermeulen

[ICT] backports openstaande wijzigingen Cache01

(changelist: Backport indien veel tijd)

Options
    • -0
    • +2
    /EWPAL2.mac.rou
  2. … 5 more files in changeset.
Friday 12 Jan 2018
9:25 am

Tommy Hebb

9:18 am

Tom Vermeulen

  • More
  • CR-897
  • finished reviewing
Thursday 11 Jan 2018
3:34 pm

Tommy Hebb

Code aangepast => Verantwoordelijkheid verhuisd naar gepaste klasse (InventSync => MaatwerkStockVerschilAfhandelaar) en gedrag passender gemaakt (Indien geen stockverschillen => standaard mailtje d...

Code aangepast => Verantwoordelijkheid verhuisd naar gepaste klasse (InventSync => MaatwerkStockVerschilAfhandelaar) en gedrag passender gemaakt (Indien geen stockverschillen => standaard mailtje dat dit meldt, ipv een lege lijst te mailen met dezelfde tekst als mochten er wel stockverschillen zijn). UnitTests uitgebreid.

1:37 pm

Tommy Hebb

Dan is dat nog steeds standaard gedrag, want dan zijn er geen AxConfigIds en blijft die variabele ook netjes leeg. Uiteindelijk doet die klasse, buiten het mailen, niets anders dan de data gemaakt ...

Dan is dat nog steeds standaard gedrag, want dan zijn er geen AxConfigIds en blijft die variabele ook netjes leeg. Uiteindelijk doet die klasse, buiten het mailen, niets anders dan de data gemaakt door andere klassen binnenhalen en verzamelen, er wordt verder niets mee gedaan qua logica.

11:55 am

Tommy Hebb

Heet nu "AxConfigIds"

Heet nu "AxConfigIds"

11:53 am

Tommy Hebb

"Ts" is de universele afkorting voor "Tussen" http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/wink.gif

"Ts" is de universele afkorting voor "Tussen"

11:52 am

Tommy Hebb

Testje geschreven.

Testje geschreven.

10:22 am

Tommy Hebb

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

Done!

10:18 am

Tommy Hebb

Nope. Zie uitleg bij lijn 60 http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif

Nope. Zie uitleg bij lijn 60

10:17 am

Tommy Hebb

Heb de naamgeving van die parameter wat aangepast, zodat het duidelijk is voor iedereen.

Heb de naamgeving van die parameter wat aangepast, zodat het duidelijk is voor iedereen.

10:03 am

Tommy Hebb

Nope! http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif 'Deze' QtyVH is die zoals die berekent (som) en doorgegeven is door de SyncAllMaatwerk, en hangt d...

Nope! 'Deze' QtyVH is die zoals die berekent (som) en doorgegeven is door de SyncAllMaatwerk, en hangt dus volledig af van wat er op de OrderLijnen staat (in specifieke gevallen, gedefinieerd in de IteratorFilter van de dynamische Iterator gebruikt door de SyncAllMaatwerk). Het niet leeg zijn van deze parameter zorgt er tevens voor dat de InventSyncCreator weet dat het over een maatwerkproduct gaat dat via de SyncAllMaatwerk gesynct wordt.

Wednesday 10 Jan 2018
3:05 pm

Tom Vermeulen

standaardgedrag is leuk, maar wat met het afwijkend gedrag: ConfigIdIterator leeg,

standaardgedrag is leuk, maar wat met het afwijkend gedrag: ConfigIdIterator leeg,

3:04 pm

Tom Vermeulen

Noem deze niet *Lijst, want het is geen %List. Zorgt alleen maar voor verwarring als iemand die wil gebruiken

Noem deze niet *Lijst, want het is geen %List. Zorgt alleen maar voor verwarring als iemand die wil gebruiken

3:02 pm

Tom Vermeulen

wat is StockTs?

wat is StockTs?

2:59 pm

Tom Vermeulen

testen?

testen?

2:59 pm

Tom Vermeulen

Mag volgens mij in een oneliner, want dat blijft leesbaar

Mag volgens mij in een oneliner, want dat blijft leesbaar

2:57 pm

Tom Vermeulen

Andere bedenking: is het niet beter om ##class(DOM.DomeinContext).Instance().GeefProductTypeAPI().IsMaatwerkProduct() te gebruiken op basis van de prnr? dan moet ge geen extra parameter gebruiken, ...

Andere bedenking: is het niet beter om ##class(DOM.DomeinContext).Instance().GeefProductTypeAPI().IsMaatwerkProduct() te gebruiken op basis van de prnr? dan moet ge geen extra parameter gebruiken, is de opzet duidelijker en kunnen er properdere testen geschreven worden (eentje met true en eentje met false)
Op een of andere manier blijft het raar overkomen om die QtyVH op deze manier door te geven/gebruiken

2:49 pm

Tom Vermeulen

ik zou hier op het eerste zicht niet direct de qtyvh doorgeven als parameter, maar ophalen via het PRNr en ProductApi. Dan is dat en passant proper testbaar en een simpelere call

ik zou hier op het eerste zicht niet direct de qtyvh doorgeven als parameter, maar ophalen via het PRNr en ProductApi. Dan is dat en passant proper testbaar en een simpelere call

2:25 pm

Tom Vermeulen

vervang hier dan ineens IsMaatwerk door die haslength van hierboven. Dan hebt ge zo geen mogelijk foute omschrijvingen als 'ismaatwerk' dat afhangt van een of ander aantal.

vervang hier dan ineens IsMaatwerk door die haslength van hierboven. Dan hebt ge zo geen mogelijk foute omschrijvingen als 'ismaatwerk' dat afhangt van een of ander aantal.

10:25 am

Tom Vermeulen

testomschrijving zegt dat qthlx en qtyblockedhlx leeg blijven. aftoetsen in assert. klopt het trouwens dat dat leeg moet zijn? moet dat niet 0 of 0.00 zijn? (zie sommige andere testen)

testomschrijving zegt dat qthlx en qtyblockedhlx leeg blijven. aftoetsen in assert.

klopt het trouwens dat dat leeg moet zijn? moet dat niet 0 of 0.00 zijn? (zie sommige andere testen)

Tuesday 09 Jan 2018
2:44 pm

Tommy Hebb

[UST3484] VHIP903: Voorraad: maatwerk: wekelijks doorsturen:

- LegacyPartijAPI van DOM.DomeinContext genomen

Options
    • -1
    • +1
    /EWPAL2.mac.rou
2:18 pm

Tommy Hebb

[UST3484] VHIP903: Voorraad: maatwerk: wekelijks doorsturen:

- I.g.v. Halux als leverancier mogen toeleveringen niet meespelen

Options
    • -0
    • +1
    /EWPAL2.mac.rou
12:27 pm

Tommy Hebb

[UST3484] VHIP903: Voorraad: maatwerk: wekelijks doorsturen:
[UST3484] VHIP903: Voorraad: maatwerk: wekelijks doorsturen:
Tuesday 19 Dec 2017
8:35 pm

Jo Willems

[ICT] VHIP904/903

- Geblokkeerde voorraad niet meer op basis van receptie

Options
    • -19
    • +5
    /EWPAL2.mac.rou
Monday 18 Dec 2017
11:43 am

Jo Willems

[ICT] VHIP904/903

- aanpassing in werkvloervooraad -> conditie weg, + casing van variabele

Options
    • -3
    • +3
    /EWPAL2.mac.rou
Friday 06 Oct 2017
1:52 pm

Tommy Hebb

[UST2942] VHIP 903: Aanpassing code door PV ivm bepaling werkvloerstock
Options
    • -13
    • +21
    /EWPAL2.mac.rou
Tuesday 19 Sep 2017
2:06 pm

Jo Willems

[UST2762] VHIP903: Voorraad: Aantal geblokkeerd ook meegeven aan AX ( = goederen in ontvangst/receptie):

- afsplitsen code die voor ax opgeroepen moet worden.

- bij AX zijn de producten nog niet gestockeerd, maar moeten wel in de blocked stock zitten

- aantal = verwacht aantal uit piece 3, gecontroleerd aantal is nog niet ingevuld

Options
    • -1
    • +19
    /EWPAL2.mac.rou
  2. … 1 more file in changeset.
Wednesday 13 Sep 2017
8:49 am

Tommy Hebb

7:57 am

Tom Vermeulen

  • More
  • CR-525
  • finished reviewing