Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
[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

  1. … 4 more files in changeset.
[VRB619] Voorraad: InventSync: VHIP903 ProdRAF en PickingCompleet: bij Space Tower wel voorraad doorsturen (i.p.v. hardcoded 0):

- Na review => check of het over een spacetower gaat en zoniet => exception

- Na review => indien AxConfigId leeg is => exception

    • -1
    • +1
    ./AxInventSyncCreator/SpaceTowerFysStockBepaler.cls.xml
  1. … 2 more files in changeset.
nog een gevalleke voor het geval het geen spacetowergeval zou zijn

nog een gevalleke voor het geval het geen spacetowergeval zou zijn

TestCase gemaakt

TestCase gemaakt

is heel dit blok niet iets dat best in een testje gesmeten wordt?

is heel dit blok niet iets dat best in een testje gesmeten wordt?

[VRB619] Voorraad: InventSync: VHIP903 ProdRAF en PickingCompleet: bij Space Tower wel voorraad...
[VRB619] Voorraad: InventSync: VHIP903 ProdRAF en PickingCompleet: bij Space Tower wel voorraad...
[VRB619] Voorraad: InventSync: VHIP903 ProdRAF en PickingCompleet: bij Space Tower wel voorraad doorsturen (i.p.v. hardcoded 0):

- FysStock nu bepaald door SpaceTowerFysStockBepaler

- SpaceTowerFysStockBepaler bepaalt FysStock adhv alle aantallen van alle KOM-lijnen op gesloten orders volgens PRNr en AxConfigId

    • -0
    • +27
    ./AxInventSyncCreator/SpaceTowerFysStockBepaler.cls.xml
  1. … 4 more files in changeset.
[VRB619] Voorraad: InventSync: VHIP903 ProdRAF en PickingCompleet: bij Space Tower wel voorraad doorsturen (i.p.v. hardcoded 0):

- I.g.v. IsSpaceTowerProduct => FysStock ophalen van GeefFysischeStockMinReservaties van StockNiveauViaProdResService

  1. … 2 more files in changeset.
DOM.PM.enu.LinkType (hieronder nog een paar keer) (en ook in ander testje)

DOM.PM.enu.LinkType (hieronder nog een paar keer) (en ook in ander testje)

my point exactly (zie comment). Afzonderen naar apart klaske dat getest kan worden

my point exactly (zie comment).
Afzonderen naar apart klaske dat getest kan worden

TestCases gemaakt voor beide public methods (en onderweg maken van product afgezonderd)

TestCases gemaakt voor beide public methods (en onderweg maken van product afgezonderd)

ClassMethod!

ClassMethod!

Als ge de lijn toch verandert, kunt ge ineens de ' door $$$Not vervangen

Als ge de lijn toch verandert, kunt ge ineens de ' door $$$Not vervangen

Boyscout principe: er bestaan geen testjes van deze klasse. Misschien is het niet veel moeite om een paar dingen toe te voegen?

Boyscout principe: er bestaan geen testjes van deze klasse. Misschien is het niet veel moeite om een paar dingen toe te voegen?

Op den duur zou een mens zich beginnen afvragen of al deze StartVerwerking-code niet beter in een aparte klasse gestoken wordt. Ooit is dat begonnen als een klein methodje dat niet te veel doet (ju...

Op den duur zou een mens zich beginnen afvragen of al deze StartVerwerking-code niet beter in een aparte klasse gestoken wordt. Ooit is dat begonnen als een klein methodje dat niet te veel doet (juist wat parameters zetten enzo) en dan was het geen probleem dat het niet in een niet-testbare classmethod steekt. Met al de extra logica die er ondertussen is bijgekomen is dat eigenlijk niet goed meer.

[DEF727] SPT: Voorraadbewegingen ontbreken op eindproduct, bij receptie productie afmelden:
[DEF727] SPT: Voorraadbewegingen ontbreken op eindproduct, bij receptie productie afmelden:
[DEF727] SPT: Voorraadbewegingen ontbreken op eindproduct, bij receptie productie afmelden:

- Alle injecties leeg defaulten Tommy

- TestCase voor SpaceTowerService

  1. … 3 more files in changeset.
[DEF727] SPT: Voorraadbewegingen ontbreken op eindproduct, bij receptie productie afmelden:

- Afzondering GeefSpaceTowerBouwsteenIterator naar eigen SpaceTowerService => zal ook door PickingCompleetConverter gaan gebruikt worden

    • -0
    • +27
    ./SpaceTowerService.cls.xml
  1. … 4 more files in changeset.
  • More
  • CR-897
  • finished reviewing
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.

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.

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

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

  1. … 5 more files in changeset.
Heet nu "AxConfigIds"

Heet nu "AxConfigIds"

"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"

Testje geschreven.

Testje geschreven.