Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Je zou de LandRepository ook kunnen toevoegen aan AxToAdminUtils, en hier DlvAddrCountry ipv landnr meegeven (zoals ook Dlv omgezet wordt naar Verzendwijze in de utils klasse)

Je zou de LandRepository ook kunnen toevoegen aan AxToAdminUtils, en hier DlvAddrCountry ipv landnr meegeven (zoals ook Dlv omgezet wordt naar Verzendwijze in de utils klasse)

Verwarrend commentaar (vooral lijn 62 & 64)

Verwarrend commentaar (vooral lijn 62 & 64)

Test: Test:

Test: Test:

blijkt dat AX voor salesorderin iso2 doorstuurt en dus niet landcode. (test aangepast.) Voor andere zoals adres stuurt AX dan weer iso3.

blijkt dat AX voor salesorderin iso2 doorstuurt en dus niet landcode. (test aangepast.)
Voor andere zoals adres stuurt AX dan weer iso3.

[VRB604] MAG: Transport: GLS-koppeling combineren met verzendwijzes met langere transportduur?
[VRB604] MAG: Transport: GLS-koppeling combineren met verzendwijzes met langere transportduur?
[VRB604] MAG: Transport: GLS-koppeling combineren met verzendwijzes met langere transportduur?

- Test voor AdresZetter - GL4 verzendwijze

    • -0
    • +36
    ./Converter/AdresZetter/Test.cls.xml
VRB583 voor aangemaakt.

VRB583 voor aangemaakt.

LergacyPartijApi gebruiken

LergacyPartijApi gebruiken

2 opmerkingskes: *geen testjes voor deze method *deze werking laat gaten toe, als ik het goe gezien heb -> problemen voor HoutenLaden (met veeeeeeel leveradressen?)

2 opmerkingskes:

  • geen testjes voor deze method
  • deze werking laat gaten toe, als ik het goe gezien heb -> problemen voor HoutenLaden (met veeeeeeel leveradressen?)
[UST4086] VHIP900/412 - toevoeging tag ivm standaard leveradres:

- UniekId ipv KlantAdresNr meegeven aan VerwijderViaID

  1. … 1 more file in changeset.
[UST4086] VHIP900/412 - toevoeging tag ivm standaard leveradres:

- Opkuisen BestaandLeverAdres op oude L-nr vooraleer op L1 te zetten

  1. … 1 more file in changeset.
[UST4086] VHIP900/412 - toevoeging tag ivm standaard leveradres:
[UST4086] VHIP900/412 - toevoeging tag ivm standaard leveradres:
[UST4086] VHIP900/412 - toevoeging tag ivm standaard leveradres:

- KlantId als klasse-parameter

[UST4086] VHIP900/412 - toevoeging tag ivm standaard leveradres:

- Verduidelijking in naamgeving vraiabelen => NieuwPrimaryDlvAddrAdres en BestaandPrimaryDlvAddrAdres en AnderBestaandAdres

- Nog een vergeten assertje

[UST4086] VHIP900/412 - toevoeging tag ivm standaard leveradres:

- Adres is PrimaryDlvAddr en is bekend in cache maar staat niet op L 1 en ander adres staat op L 1 => ander adres verzetten naar nieuw KlantAdresNr en PrimaryDlvAddr verzetten naar L 1

[UST4086] VHIP900/412 - toevoeging tag ivm standaard leveradres:

- Adres is PrimaryDlvAddr en is bekend in cache en staat op L 1 => Bewaar

[UST4086] VHIP900/412 - toevoeging tag ivm standaard leveradres:

- Adres is PrimaryDlvAddr en is nog onbekend in cache en L 1 is bezet => L 1 veplaatsen naar nieuw KlantAdresNr en this adres op L 1 zetten

  1. … 1 more file in changeset.
[UST4086] VHIP900/412 - toevoeging tag ivm standaard leveradres:

- Adres is PrimaryDlvAddr en is nog onbekend in cache en L 1 is nog vrij

  1. … 1 more file in changeset.
[UST4086] VHIP900/412 - toevoeging tag ivm standaard leveradres:

- LeverAdresUtils => ZetLeverAdressen => Voorafgaandelijke refactor ter bevordering van leesbaarheid

- DataMKlantNative => Afzondering GeefVolgendVrijKlantAdresNr => nodig voor verdere implementatie van deze story => om onterecht op L 1 staand adres te kunnen verzetten en daarna bewaar op repo te doen

  1. … 2 more files in changeset.
ik zou nog kunnen beginnen neuten over de casing van adres/Adres. Het probleem is dat het in feite mer kleine a moet, want het is een repo van Leveradressen. Het is wel zo dat historisch gezien er ...

ik zou nog kunnen beginnen neuten over de casing van adres/Adres. Het probleem is dat het in feite mer kleine a moet, want het is een repo van Leveradressen. Het is wel zo dat historisch gezien er altijd (foutief) met een grote A geschreven werd (zie andere lijntjes in deze klasse). Doe er mee wat ge wilt

[UST4044] VHIP900 - KlantSync: Opkuisen juiste leveradres igv dubbele aanwezigheid AxLocationId:

- Na review => GeefRecentsteUniekIdVolgensAxLeveradresId op LeverAdresService ipv op LeverAdresRepository

  1. … 7 more files in changeset.
Eigenlijk hoort dit niet in de repository. Een repository moet enkel bewaren, geven zoeken enz en krijgt/geeft een object van het type waarover de repository het heeft. Dit hoort eerder in een Leve...

Eigenlijk hoort dit niet in de repository. Een repository moet enkel bewaren, geven zoeken enz en krijgt/geeft een object van het type waarover de repository het heeft.
Dit hoort eerder in een LeveradresService. Pas op, ik wil hier niet beweren dat dat momenteel altijd en overal zo te goei gedaan wordt!

[UST4044] VHIP900 - KlantSync: Opkuisen juiste leveradres igv dubbele aanwezigheid AxLocationId:

- Aanpassen testje => VerwachtMethodCall met ExactAantalKeer geeft meer duidelijkheid en zekerheid

[UST4044] VHIP900 - KlantSync: Opkuisen juiste leveradres igv dubbele aanwezigheid AxLocationId:
[UST4044] VHIP900 - KlantSync: Opkuisen juiste leveradres igv dubbele aanwezigheid AxLocationId: