Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
[ICT-1078] AT 1467: Pakbon pas aanmaken op moment van transport

Aanpassen unit testen volgens nieuwe code implementatie

[ICT-1078] Fase 2 sturen bericht dat pakbon mag geboekt worden

- refactoring code wat effecienter maken.

    • -15
    • +0
    ./BatchJobPakbonMagGeboektWorden/SpyImplementatie.cls.xml
  1. … 1 more file in changeset.
[ICT-1078] Fase 2 sturen bericht dat pakbon mag geboekt worden

BatchJobPakbonMagGeboektWorden rekening houden met de dueoutime van het order, extra conditie: enkel het PakbonMagGeboektWorden bericht sturen als dit op vandaag staat.

Indien een order reeds vroeger gepicked wordt mag de bon nog niet verzonden worden op die avond als de dueouttime op overmorgen staat

    • -1
    • +7
    ./BatchJobPakbonMagGeboektWorden/SpyImplementatie.cls.xml
    • -1
    • +10
    ./BatchJobPakbonMagGeboektWorden/Test.cls.xml
  1. … 3 more files in changeset.
[ICT-1078] Fase 2 sturen bericht dat pakbon mag geboekt worden

Logging toevoegen als aanduiding start en einde BatchJobPakbonMagGeboektWorden

Testen aanpassen

[ICT-1078] Fase 2 sturen bericht dat pakbon mag geboekt worden

Testen aanpassen voor PakbonMagGeboektWordenConverter aanpassing

  1. … 3 more files in changeset.
[ICT-1078] Fase 2 sturen bericht dat pakbon mag geboekt worden

Pakbon code aanpassen, gebruik maken van Bon.GeefVerzendwijzeID ipv Bon.GeefVerzendwijze

    • -1
    • +1
    ./BatchJobPakbonMagGeboektWorden/Test.cls.xml
  1. … 1 more file in changeset.
[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.
[ICT-1481] Door vreemde kronkels geraakt soms een datum van 1 of 2 maanden in de toekomst als pickingdatum...
[ICT-1481] Door vreemde kronkels geraakt soms een datum van 1 of 2 maanden in de toekomst als pickingdatum...
[ICT-1481] Door vreemde kronkels geraakt soms een datum van 1 of 2 maanden in de toekomst als pickingdatum in ^ORDW. Dat is probleem voor facturatie en voorraad-boekingen.
  1. … 1 more file in changeset.
[ICT-1078] Fase 2 sturen bericht dat pakbon mag geboekt worden

Refactor: gebruik maken van Flow/Uitlevering/HoofdingRepository ipv meerdere loops + unit testen aanpassen

    • -0
    • +43
    ./BatchJobPakbonMagGeboektWorden/SpyImplementatie.cls.xml
    • -4
    • +7
    ./BatchJobPakbonMagGeboektWorden/Test.cls.xml
  1. … 1 more file in changeset.
[ICT-1078] Fase 2 sturen bericht dat pakbon mag geboekt worden

Refactor: rename methodes ZetVlagHoofding en GeefVlagHoofding naar specifiekere methodes met duidelijke naamgeving

    • -1
    • +1
    ./BatchJobPakbonMagGeboektWorden/Test.cls.xml
  1. … 5 more files in changeset.
test of meerdere uitleveringen goed afgehandeld worden, cfr opm repository over flow.uitlevering.hoofding

test of meerdere uitleveringen goed afgehandeld worden, cfr opm repository over flow.uitlevering.hoofding

Repository maken over Flow.Uitlevering.Hoofding ? encapsulaties gebruiken waar het kan? overweeg met PRA om te zien of het de investering waard is. Indien niet gebruik maken van encapsulatie: nu wo...

Repository maken over Flow.Uitlevering.Hoofding ? encapsulaties gebruiken waar het kan? overweeg met PRA om te zien of het de investering waard is.
Indien niet gebruik maken van encapsulatie:
nu wordt 2 x $Order uitgevoerd voor één uitlevering. zo ook voor de volgende whiles
Dat kan beter
Na de rest van de code te bekijken : Repository maken (resulteert in één iterator en testbaarheid of meer dan één uitlevering correct afgehandeld worden.

Dit gaat alleen voor de eerste bon werken niet meer voor de 2de.

Dit gaat alleen voor de eerste bon werken niet meer voor de 2de.

ik zou dit toch over een paar lijnen spreiden, gaat wat leesbaarder worden dan.

ik zou dit toch over een paar lijnen spreiden, gaat wat leesbaarder worden dan.

Private?

Private?

Wat doen we met Data?

Wat doen we met Data?

grrrrr...

grrrrr...

grrrr....

grrrr....

Maak hier een specifieke method voor waarvan de naam betekenis heeft. 63 zegt niet veel. cfr andere opmerking op de impl hiervan.

Maak hier een specifieke method voor waarvan de naam betekenis heeft. 63 zegt niet veel. cfr andere opmerking op de impl hiervan.

cfr andere opmerking in implementatie

cfr andere opmerking in implementatie

Lijkt me 2 mooie methods om private te zetten en dan in specifiek genaamde methods voor speciefieke gegevens te gebruiken. Nu staat er ergens anders in de code een getal 63 te blinken....

Lijkt me 2 mooie methods om private te zetten en dan in specifiek genaamde methods voor speciefieke gegevens te gebruiken.
Nu staat er ergens anders in de code een getal 63 te blinken....

[ICT-1078] Fase 2 sturen bericht dat pakbon mag geboekt worden

Refactoring volgens coding guidelines + wlip's verwijderen

    • -10
    • +9
    ./BatchJobPakbonMagGeboektWorden/Test.cls.xml
  1. … 4 more files in changeset.
[ICT-1078] Fase 2 sturen bericht dat pakbon mag geboekt worden

Gebruik maken van de helper klasse zijn IsVerzendwijzeMetUitgesteldPakbonBoekenAx methode en de gelijkaardige methode uit de klasse AXimpl.Dynamics.InventoryOut.EventHandler.PickingCompleet.PakbonMagGeboektWorden verwijderen en helperklasse injecten + test verwijderen , worden in de helperklasse zijn testen opgevangen

  1. … 3 more files in changeset.
[ICT-1078] Fase 2 sturen bericht dat pakbon mag geboekt worden

Toevoegen testen methode IsVerzendwijzeMetUitgesteldPakbonBoekenAxTest uit helperklasse

    • -0
    • +48
    ./EventHandler/PakbonMagGeboektWordenHelper/IsVerzendwijzeMetUitgesteldPakbonBoekenAxTest.cls.xml
[ICT-1078] Fase 2 sturen bericht dat pakbon mag geboekt worden

Toevoegen testen methode BonMarkerenAlsVerzondenTest uit helperklasse

[ICT-1078] Fase 2 sturen bericht dat pakbon mag geboekt worden

Hernoemen GeefVlag en zetvlag naar GeeftVlagHooofdign en ZetVlagHooding

    • -1
    • +1
    ./BatchJobPakbonMagGeboektWorden/Test.cls.xml
  1. … 5 more files in changeset.
[ICT-1078] Fase 2 sturen bericht dat pakbon mag geboekt worden

Toevoegen AfhandelenBijWijzigenVerzendwijze aan helper klasse en bijhorende testen

Methode aanroepen vanuit de KFVZW zijn SET methode

  1. ./EventHandler/PakbonMagGeboektWordenHelper
  2. … 2 more files in changeset.