AdminToAxUtils.cls.xml

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?

- Land overbodig.

  1. … 1 more file in changeset.
[VRB604] MAG: Transport: GLS-koppeling combineren met verzendwijzes met langere transportduur?

- Covevrsie GL4 naar DI1. Mogelijks landafhankelijk, maar in deze fase niet omdat AX GL4 (nog) niet kent.Dus steeds GL4 naar DI1 omzetten.

  1. … 1 more file in changeset.
[VRB604] MAG: Transport: GLS-koppeling combineren met verzendwijzes met langere transportduur?

- Voorbereiding - met landcode.

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

- beetje refactor nodig zodat alle convertoren de utils gebruiken.

- mogelijkheid om geen error te gooien.

Thx voor de tip. Heb dit aangepast. Maar... eerlijk gezegd had ik aan de naam "$$$IfLength" echt niet verwacht dat die een prefix en een suffix zou toevoegen!

Thx voor de tip. Heb dit aangepast. Maar... eerlijk gezegd had ik aan de naam "$$$IfLength" echt niet verwacht dat die een prefix en een suffix zou toevoegen!

[UST3344] Verzendwijze conversie tussen Admin en AX: review verwerkingen.
  1. … 9 more files in changeset.
Ik begrijp de opmerking over de naam van de config. Ik ga erop letten in de toekomst!

Ik begrijp de opmerking over de naam van de config. Ik ga erop letten in de toekomst!

Het leek mij niet verkeerd om dit als logica te beschouwen die niet zozeer puur bij verzendwijzes hoort, maar met een keuze wat bij deze specifieke interface te doen als we de verzendwijze niet vin...

Het leek mij niet verkeerd om dit als logica te beschouwen die niet zozeer puur bij verzendwijzes hoort, maar met een keuze wat bij deze specifieke interface te doen als we de verzendwijze niet vinden, vandaar.
En ook omdat deze locatie toelaat om de context (Ordernummer) goed mee te loggen.

Het leek mij niet verkeerd om dit als logica te beschouwen die niet zozeer puur bij verzendwijzes hoort, maar met een keuze wat bij deze specifieke interface te doen als we de verzendwijze niet vin...

Het leek mij niet verkeerd om dit als logica te beschouwen die niet zozeer puur bij verzendwijzes hoort, maar met een keuze wat bij deze specifieke interface te doen als we de verzendwijze niet vinden (bvb DI1 als het een insert is, of niets (niet wijzigen/updaten) in geval van update), vandaar.
En ook omdat deze locatie toelaat om de context (CustAccount-update) goed mee te loggen.

Het leek mij niet verkeerd om dit als logica te beschouwen die niet zozeer puur bij verzendwijzes hoort, maar met een keuze wat bij deze specifieke interface te doen als we de verzendwijze niet vin...

Het leek mij niet verkeerd om dit als logica te beschouwen die niet zozeer puur bij verzendwijzes hoort, maar met een keuze wat bij deze specifieke interface te doen als we de verzendwijze niet vinden, vandaar.
En ook omdat deze locatie toelaat om de context (SalesId) goed mee te loggen.

Idem als omgekeerde klasse: info toegevoegd aan de klasse en method.

Idem als omgekeerde klasse: info toegevoegd aan de klasse en method.

Aha, ik wist niet dat de conventie was om enkel formattering uit te voeren en conversies te voorzien die geen data nodig hebben, in deze klasse. Zal het in commentaar erbij zetten, alsook de sugges...

Aha, ik wist niet dat de conventie was om enkel formattering uit te voeren en conversies te voorzien die geen data nodig hebben, in deze klasse.
Zal het in commentaar erbij zetten, alsook de suggestie om bij een volgende noemenswaardige wijziging, dit in overweging te nemen!

Type toegevoegd.

Type toegevoegd.

Type toegevoegd.

Type toegevoegd.

GeefPrimairAdres hernoemd naar MaakPrimairAdres; is meteen ook consistenter met MaakVerzendingWijze.

GeefPrimairAdres hernoemd naar MaakPrimairAdres; is meteen ook consistenter met MaakVerzendingWijze.

  • More
  • CR-923
  • finished reviewing
Een beetje stout, maar wat met bvb. "NietWoDo" ? is dat ook mogelijk?

Een beetje stout, maar wat met bvb. "NietWoDo" ?
is dat ook mogelijk?

Beter dan de 2 lijnen is : Set AdminVerzendWijzeCode = $$$IfLength(AdminVerzendWijzeCode,"","&",) _verzendWijzeCodeZonderWeekDag _ ";" _ WeekDag

Beter dan de 2 lijnen is :
Set AdminVerzendWijzeCode = $$$IfLength(AdminVerzendWijzeCode,"","&",) _verzendWijzeCodeZonderWeekDag _ ";" _ WeekDag

[UST3417] Andere separator (dan normaal in Admin) tussen aanleverdagen, omdat vhPOPEDIT de oude niet...
[UST3417] Andere separator (dan normaal in Admin) tussen aanleverdagen, omdat vhPOPEDIT de oude niet...
  • More
  • CR-868
  • finished reviewing