CacheAdminA_trunk2010/AXimpl/Dynamics/InventoryOut

MaatwerkStockVerschilAfhandelaar.cls.xml

Checkout Tools

Line History

Line Count Graph
Line Count Graph

Stats

Commits this week: 0
Total Committers: 1
Last Commit: 22 Jan 18
Commits this week: 0
Total Lines of Code (LoC): 163
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
Tuesday 23 Jan 2018
9:24 am

Tommy Hebb

9:24 am

Tommy Hebb

Kweetet. 'k Zag het zelf ook direct na het committen en heb het aangepast, maar in Crucible zie je dat dan niet, da's gecached :-/

Kweetet. 'k Zag het zelf ook direct na het committen en heb het aangepast, maar in Crucible zie je dat dan niet, da's gecached :-/

7:55 am

Tom Vermeulen

  • More
  • CR-940
  • finished reviewing
7:55 am

Tom Vermeulen

vergeet uwen [USTxxxx] volgende keer niet toe te voegen aan de omschrijving http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif

vergeet uwen [USTxxxx] volgende keer niet toe te voegen aan de omschrijving

Monday 22 Jan 2018
10:05 am

Tommy Hebb

VHIP903 VoorraadSync Maatwerk: Inhoud mail => stream ipv string:
VHIP903 VoorraadSync Maatwerk: Inhoud mail => stream ipv string:
10:02 am

Tommy Hebb

VHIP903 VoorraadSync Maatwerk: Inhoud mail => stream ipv string:

- Opkuis van enkele comments ivm iteratie van AX-query

- De inhoud van de mail in nu een Stream ipv een string => MAXSTRING-error vermijden

Options
    • -12
    • +17
    ./MaatwerkStockVerschilAfhandelaar.cls.xml
  2. … 3 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.

3:31 pm

Tommy Hebb

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

- Na review => MaatwerkStockVerschilAfhandelaar is nu zelf verantwoordelijk om de inhoud van de mail, die nu altijd gestuurd wordt, te bepalen, waarbij indien geen verschillen dit als dusdanig gemeld wordt in de mail, ipv een lege lijst te sturen.

- Na review => Uitbreiding UnitTests MaatwerkStockVerschilAfhandelaar

- Na review => MailVoorraadbeheer wordt nu uitgemockt in de SyncAll Maatwerk UnitTests

Options
    • -28
    • +34
    ./MaatwerkStockVerschilAfhandelaar.cls.xml
  2. … 3 more files in changeset.
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.

1:24 pm

Tommy Hebb

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

- Na review => Naming => CheckStockTussenOrderLijnenEnStockAdmin

Options
    • -1
    • +1
    ./MaatwerkStockVerschilAfhandelaar.cls.xml
  2. … 5 more files in changeset.
11:57 am

Tommy Hebb

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

- Na review => Naming hier en daar en UnitTest voor MaatwerkIteratorFilter

Options
    • -4
    • +4
    ./MaatwerkStockVerschilAfhandelaar.cls.xml
  2. … 5 more files in changeset.
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: