CacheAdminA_trunk2010/AXimpl/Dynamics/InventoryOut

SyncAll.cls.xml

Checkout Tools

Line History

Line Count Graph
Line Count Graph

Stats

Commits this week: 0
Total Committers: 2
Last Commit: 15 May 20
Commits this week: 0
Total Lines of Code (LoC): 172
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 25 Sep 2019
2:21 pm

Tommy Hebb

1:29 pm

Tom Vermeulen

1:29 pm

Tom Vermeulen

Eerst de comment hieronder lezen. Gelezen? Ok: een ExactAantalKeer(0) gecombineerd met een Verwacht heeft dan net weer eerder nood aan een $$$ElkeWaarde omdat ge in de meeste gevallen wilt dat de...

Eerst de comment hieronder lezen.

Gelezen?

Ok: een ExactAantalKeer(0) gecombineerd met een Verwacht heeft dan net weer eerder nood aan een $$$ElkeWaarde omdat ge in de meeste gevallen wilt dat de Verwachte method nooit of te nimmer, ongeacht de parameters wordt uitgevoerd.

Deze test blijft slagen als ge 1000 door 1111 vervangt (of gelijk welke parameter door gelijk welke waarde), wat ook niet volledig koosjer is

1:25 pm

Tom Vermeulen

Eigenlijk geen $$$ElkeWaarde bij een VerwachtMethodCall. Ge 'verwacht' iets, dus ge weet wat er gaat gebeuren, dan kan je best mbv een ..Matches iets meegeven (en ja, ook bij de andere reeds bestaa...

Eigenlijk geen $$$ElkeWaarde bij een VerwachtMethodCall. Ge 'verwacht' iets, dus ge weet wat er gaat gebeuren, dan kan je best mbv een ..Matches iets meegeven (en ja, ook bij de andere reeds bestaande methods)

11:29 am

Tommy Hebb

[ICT-886] Voorraad: VHIP903: SyncAll - GenerateMaatwerk: Space Tower - MaatwerkStockVerschilAfhandelaar:...
[ICT-886] Voorraad: VHIP903: SyncAll - GenerateMaatwerk: Space Tower - MaatwerkStockVerschilAfhandelaar:...
Monday 22 Jan 2018
2:41 pm

Tom Vermeulen

Friday 19 Jan 2018
4:31 pm

Tommy Hebb

4:31 pm

Tommy Hebb

Nope. Dan geeft de HasNext() netjes 0, zoals verwacht => Global is leeg => Geen stockverschillen gevonden => Default mailtje zal verstuurd worden met melding hiervan (zonder een onbestaand lijstje ...

Nope. Dan geeft de HasNext() netjes 0, zoals verwacht => Global is leeg => Geen stockverschillen gevonden => Default mailtje zal verstuurd worden met melding hiervan (zonder een onbestaand lijstje mee te sturen).

4:23 pm

Tommy Hebb

I'll remember for next time http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/wink.gif

I'll remember for next time

4:21 pm

Tommy Hebb

  • More
  • CR-936
  • finished reviewing
3:22 pm

Tom Vermeulen

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

ongeveer ja

3:10 pm

Tom Vermeulen

Best duidelijk zijn. Neemt niet weg dat het gewoon Set DynIterator = ##class().... moeten zijn

Best duidelijk zijn. Neemt niet weg dat het gewoon Set DynIterator = ##class().... moeten zijn

2:51 pm

Tom Vermeulen

  • More
  • CR-932
  • finished reviewing
2:51 pm

Tom Vermeulen

nitpicking qua naamgeving: ik zelf zou een DataBouwer klasse een BouwHuppeldepup method geven. Als de method StelSamenDingesEnDinges is, kan je dat in een *SamenSteller steken. Moet dat niet verand...

nitpicking qua naamgeving: ik zelf zou een DataBouwer klasse een BouwHuppeldepup method geven. Als de method StelSamenDingesEnDinges is, kan je dat in een *SamenSteller steken.
Moet dat niet veranderen hoor, maar op veel andere plaatsen wordt dat ook gedaan: een *Bepaler heeft een Bepaal() enzo

2:49 pm

Tom Vermeulen

juist een vraagske: moet er iets extra voorzien worden voor het geval de resultset leeg is?

juist een vraagske: moet er iets extra voorzien worden voor het geval de resultset leeg is?

10:13 am

Tommy Hebb

Is dat niet dezelfde test als: "Test: Generate wordt NIET aangeroepen indien enkel items met verkeerde hoofdnode aanwezig"() ?

Is dat niet dezelfde test als: "Test: Generate wordt NIET aangeroepen indien enkel items met verkeerde hoofdnode aanwezig"() ?

9:29 am

Tommy Hebb

Mag deze niet gedimd worden als TECH.Iterator? Maakt niet uit zeker, wegens toch maar een dimmeke?

Mag deze niet gedimd worden als TECH.Iterator? Maakt niet uit zeker, wegens toch maar een dimmeke?

9:22 am

Tommy Hebb

Waarom hier IsGeldig en niet IsValid, zoals in alle filters? Dit dan nog de "Valid"-ator http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/wink.gif

Waarom hier IsGeldig en niet IsValid, zoals in alle filters? Dit dan nog de "Valid"-ator

Thursday 18 Jan 2018
2:16 pm

Tom Vermeulen

[UST3571] VHIP903: Voorraad: InventSync berichten pas om het 30min sturen naar aX
[UST3571] VHIP903: Voorraad: InventSync berichten pas om het 30min sturen naar aX
Wednesday 17 Jan 2018
5:00 pm

Tommy Hebb

[UST3572] VHIP maatwerksync:
[UST3572] VHIP maatwerksync:
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.