Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
[ICT-3916] [rvJBA] [Cache] Order vergelijk software
  1. … 4 more files in changeset.
Ergens zou deze method naam moeten duidelijk maken dat het originele product verwijderd wordt lijkt mij?

Ergens zou deze method naam moeten duidelijk maken dat het originele product verwijderd wordt lijkt mij?

Is dit niet 'gevaarlijk'? Moet je niet checken of er 1 product aanwezig is en anders een error gooien?

Is dit niet 'gevaarlijk'? Moet je niet checken of er 1 product aanwezig is en anders een error gooien?

Is écht muggenziften, maar deze 'if' mag op lijn 566. Ik durf het bijna niet typen :-p

Is écht muggenziften, maar deze 'if' mag op lijn 566. Ik durf het bijna niet typen :-p

[ICT-3983] OrderSync in cache uitbreiden voor encowayconfiguraties
[ICT-3983] OrderSync in cache uitbreiden voor encowayconfiguraties
[ICT-3983] OrderSync in cache uitbreiden voor encowayconfiguraties

- Test aanmaken die encowaypad covert

  1. … 3 more files in changeset.
[ICT-3983] OrderSync in cache uitbreiden voor encowayconfiguraties

-

[ICT-3983] OrderSync in cache uitbreiden voor encowayconfiguraties
  1. … 2 more files in changeset.
[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

  1. … 2 more files in changeset.
[ICT-1630] [rvPVR] [900] - Interface klanten van AX naar Admin met 'raar' teken in naam van klant

- Gewone 'tick' ("´") toegevoegd aan niet-printbare karakters

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

Testen aanpassen voor PakbonMagGeboektWordenConverter aanpassing

    • -0
    • +47
    ./InventoryOut/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

  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-1159] Mapping leveranciersnummer VH/HLX:

- De caller ProductItemConverter vertrouwt erop dat hij een lege string terugkrijgt indien SupplierId leeg is => Eerder weggehaalde 1e conversieregel in ConvertSupplierIDHLX teruggezet.

- Check op length en indien leeg exceptie throwen heeft dus geen nut meer => conversie zal nu altijd lukken => kan echter verkeerd zijn indien nummer in AX en Admin niet overeenstemt! => heeft dan specifieke mapping nodig.

- ConversieBezwarenBepaler => ConvertSupplierIDHLX zal nooit nog een exceptie throwen, dus try catch heeft geen nut meer.

- Testjes aangepast conform nieuwe situatie.

    • -0
    • +7
    ./InventoryIn/Converter/ConversieBezwarenBepaler.cls.xml
  1. … 4 more files in changeset.