CacheAdminA_trunk2010/vhUnitTest/AXimpl/Dynamics/InventoryOut/SyncAll

Maatwerk.cls.xml

Checkout Tools

Line History

Line Count Graph
Line Count Graph

Stats

Commits this week: 0
Total Committers: 1
Last Commit: 25 Sep 19
Commits this week: 0
Total Lines of Code (LoC): 352
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

2:21 pm

Tommy Hebb

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

- Na review => Verduidelijking dat het allemaal gerelateerd is aan handelingen met ProductNr 1000

Options
    • -5
    • +8
    ./Maatwerk.cls.xml
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
    • -11
    • +89
    ./Maatwerk.cls.xml
  2. … 1 more file in changeset.
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

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?

Thursday 18 Jan 2018
10:57 am

Tommy Hebb

[UST3572] VHIP maatwerksync:

- De testen sturen nu geen mailtje meer

Options
    • -2
    • +14
    ./Maatwerk.cls.xml
10:19 am

Tommy Hebb

[UST3572] VHIP maatwerksync:

- DataBouwer => is nu Mockable => IndienMethodCall("StelSamenTeSyncenPRNrAxConfigIdCombos").DanDoeNiks() => SyncAll compileert nu wa rapper

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

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
    • -0
    • +5
    ./Maatwerk.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
    • -5
    • +5
    ./Maatwerk.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,