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
Friday 15 May 2020
1:47 pm

Tommy Hebb

[ICT-1512] InventSync dag - AxConfigId is leeg, maar is noodzakelijk bij een Spacetowerproduct:

- SyncAll => Method GenerateDelayedInventSyncs => gebruikt nu een andere filter in de iterator, nml: AXimpl.Admin.DOM.PM.ProductIteratorFilter => Houdt Spacetowers tegen. De vorige filter deed dat niet en is enkel bedoeld voor InventSync gebaseerd op orderlijnen omdat de AxConfigId nodig is.

- AXimpl.Admin.DOM.PM.ProductIteratorFilter => werkt nu niet alleen meer met een ResultSet maar ook met een PRNr.

- AXimpl.Dynamics.InventoryOut.Converter.InventoryOutConverter => InventSync-excepties worden nu ook gemaild naar ICT_Meldingen

Options
    • -1
    • +1
    ./SyncAll.cls.xml
  2. … 4 more files in changeset.
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:...
11:22 am

Tommy Hebb

[ICT-886] Voorraad: VHIP903: SyncAll - GenerateMaatwerk: Space Tower - MaatwerkStockVerschilAfhandelaar:

- Geen CheckStockTussenOrderLijnenEnStockAdmin i.g.v. IsSpaceTowerProduct

- Aanpassing en uitbreiding TestCase

Options
    • -2
    • +10
    ./SyncAll.cls.xml
  2. … 1 more file in changeset.
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:27 pm

Tom Vermeulen

[UST3571R] VHIP903: Voorraad: InventSync berichten pas om het 30min sturen naar aX

- Opmerkingen verwerkt

Options
    • -2
    • +2
    ./SyncAll.cls.xml
  2. … 1 more file in changeset.
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:
4:58 pm

Tommy Hebb

[UST3572] VHIP maatwerksync:

- Afzonderen StelSamenTeSyncenPRNrAxConfigIdCombos uit de constructor naar de method GenerateDelayedInventSyncs

Options
    • -1
    • +9
    ./SyncAll.cls.xml
  2. … 3 more files in changeset.
4:08 pm

Tom Vermeulen

[UST3571] VHIP903: Voorraad: InventSync berichten pas om het 30min sturen naar aX

- GenerateDelayedInventSyncs met filter doen werken en nog wat extras

- testen komen nog

Options
    • -5
    • +17
    ./SyncAll.cls.xml
2:15 pm

Tom Vermeulen

[UST3571] VHIP903: Voorraad: InventSync berichten pas om het 30min sturen naar aX

- SyncAll uitgebreid met iets voor de inventsyncs

Options
    • -1
    • +26
    ./SyncAll.cls.xml
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
    • -4
    • +1
    ./SyncAll.cls.xml
  2. … 3 more files in changeset.