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

VerwerkAfgeslotenTransport methode toegevoegd aan PakbonMagGeboektWordenHelper om zo het event PakbonMagGeboektWorden te triggeren voor alle bonnen van het opgegeven transsport nummer

    • -0
    • +7
    ./EventHandler/PakbonMagGeboektWordenHelper.cls.xml
  1. … 2 more files in changeset.
[ICT-1078] Fase 2 sturen bericht dat pakbon mag geboekt worden

Testen aanpassen voor PakbonMagGeboektWordenConverter aanpassing

    • -0
    • +47
    ./EventHandler/PakbonMagGeboektWordenHelper.cls.xml
  1. … 3 more files 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

    • -0
    • +7
    ./Converter/InventoryOutConverter.cls.xml
  1. … 4 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

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

Hernoemen methode RaisePakbonMagGeboektWordenEvent naar AfhandelaarRaisePakbonMagGeboektWordenEvent om verwarring te voorkomen + referenties aanpassen

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

Refactor AanduidenAxBerichtPakbonMagGeboektWordenVerzonden Gebruik maken van objecten ipv rechtstreeks op de globals te werken

  1. … 1 more file in changeset.
Geen engelstalige stukken in naamgeving Code wordt overbodig want GeefBon gebruiken en dan verder ook Bon.GeefVerzendwijze

Geen engelstalige stukken in naamgeving
Code wordt overbodig want GeefBon gebruiken en dan verder ook Bon.GeefVerzendwijze

// GeefVerzendwijze methode gebruiken uit apps.common.dataservice ipv zelf rechtstreeks %OpenId te doen

// GeefVerzendwijze methode gebruiken uit apps.common.dataservice ipv zelf rechtstreeks %OpenId te doen

//$Not van maken ipv '

//$Not van maken ipv '

// GeefBon --> Geef klant --> via objecten oproepen, niet rechtstreeks via de global

// GeefBon --> Geef klant --> via objecten oproepen, niet rechtstreeks via de global

//$Not van maken in plaats van ..VerzendwijzeUitgesteldPakbonBoekenAx(bon, klant) = $$$False

//$Not van maken in plaats van ..VerzendwijzeUitgesteldPakbonBoekenAx(bon, klant) = $$$False

een else kant toevoegen met een log else{ // log toevoegen dat event uitgesteld zal worden verzonden }

een else kant toevoegen met een log

else

Unknown macro: { // log toevoegen dat event uitgesteld zal worden verzonden }
// var hernoemen naar GeneratePakbonMagGeboektWordenResult en ook het vereder gebruik , argument in methode aanpassen

// var hernoemen naar GeneratePakbonMagGeboektWordenResult en ook het vereder gebruik , argument in methode aanpassen

Niet gewoon gebruik maken van de globals maar de gemapte objecten gebruiken zoals Geefbon Daaraan een methode toevoegen als en Zetvlag(new) De zet vlag zal dan onderstaande set piece bevatten

Niet gewoon gebruik maken van de globals maar de gemapte objecten gebruiken zoals Geefbon
Daaraan een methode toevoegen als en Zetvlag(new)
De zet vlag zal dan onderstaande set piece bevatten

ctor toevoegen en injecteren van de geeforderapiservice niet via context

ctor toevoegen en injecteren van de geeforderapiservice niet via context

VerwerkRecord hernoemen naar VerwerkBon (dit is iets duidelijker)

VerwerkRecord hernoemen naar VerwerkBon (dit is iets duidelijker)

Naamgevingen verduidelijken *Geen engelse stukken vb VerzendwijzeLONG.. *{} plaatsen volgens cache standaard *Zie ook confluence notities voor dit ticket
  • Naamgevingen verduidelijken
  • Geen engelse stukken vb VerzendwijzeLONG..
  • {} plaatsen volgens cache standaard
  • Zie ook confluence notities voor dit ticket