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
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
12:27 pm

Tommy Hebb

[UST3484] VHIP903: Voorraad: maatwerk: wekelijks doorsturen:
[UST3484] VHIP903: Voorraad: maatwerk: wekelijks doorsturen:
Wednesday 13 Sep 2017
8:49 am

Tommy Hebb

7:57 am

Tom Vermeulen

  • More
  • CR-525
  • finished reviewing
Tuesday 12 Sep 2017
5:23 pm

Tommy Hebb

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