Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
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-855
  • finished reviewing
[UST3258] VHIP504: ORDER2AX: Transportkost en -omschrijving (Hein; TAX export)
[UST3258] VHIP504: ORDER2AX: Transportkost en -omschrijving (Hein; TAX export)
[UST3258] VHIP504: ORDER2AX: Transportkost en -omschrijving (Hein; TAX export)

- hier bedoelen ze waarschijnlijk Orderlijn ipv order als type.

    • -1
    • +1
    ./Converter/OrderlijnConverteerder.cls.xml
  1. … 2 more files in changeset.
[UST3333] Ax LocationId meegeven in 504-berichten: LocationId invullen, ofwel vanuit order-adres-node, ofwel vanuit klant-adres-node (ofwel leeg laten): Testjes gemaakt.
    • -0
    • +7
    ./Converter/OrderHoofdingConverteerder.cls.xml
  1. … 3 more files in changeset.
  • More
  • CR-831
  • finished reviewing
ik zou insert of update enkel verwachten bij een Zet, maar niet bij een geef method ??

ik zou insert of update enkel verwachten bij een Zet, maar niet bij een geef method ??

deze code hoort thuis bij het bepalen van de verzendwijze. nu wordt er op 2 plaatsen bepaald wat het eventuele resultaat is. De config noemt immers ook AXimpl.AxToAdmin..... en hoort hier niet thuis.

deze code hoort thuis bij het bepalen van de verzendwijze. nu wordt er op 2 plaatsen bepaald wat het eventuele resultaat is.
De config noemt immers ook AXimpl.AxToAdmin..... en hoort hier niet thuis.

type meegeven.

type meegeven.

deze code hoort thuis bij het bepalen van de verzendwijze. nu wordt er op 2 plaatsen bepaald wat het eventuele resultaat is. De config noemt immers ook AXimpl.AxToAdmin..... en hoort hier niet thuis.

deze code hoort thuis bij het bepalen van de verzendwijze. nu wordt er op 2 plaatsen bepaald wat het eventuele resultaat is.
De config noemt immers ook AXimpl.AxToAdmin..... en hoort hier niet thuis.

type meegeven.

type meegeven.

deze code hoort thuis bij het bepalen van de verzendwijze. nu wordt er op 2 plaatsen bepaald wat het eventuele resultaat is. De config noemt immers ook AXimpl.AxToAdmin..... en hoort hier niet thuis.

deze code hoort thuis bij het bepalen van de verzendwijze. nu wordt er op 2 plaatsen bepaald wat het eventuele resultaat is.
De config noemt immers ook AXimpl.AxToAdmin..... en hoort hier niet thuis.

type meegeven.

type meegeven.

Casing volgens onze code conventions. hoofdletters voor If en Do, etc...

Casing volgens onze code conventions. hoofdletters voor If en Do, etc...

titel klopt niet volgens inhoud.

titel klopt niet volgens inhoud.

AdminToAx is gemaakt om formattering uit te voeren en conversies te voorzien die geen data nodig hebben. Voor opzoeken van gegevens in globals gebruiken we lookups of repositories. Is er een reden ...

AdminToAx is gemaakt om formattering uit te voeren en conversies te voorzien die geen data nodig hebben.
Voor opzoeken van gegevens in globals gebruiken we lookups of repositories.
Is er een reden waarom we geen persistent class gebruiken en een repository die de standaard geef, maak, bestaat etc.. aanbiedt ?

AxToAdmin is gemaakt om formattering uit te voeren en conversies te voorzien die geen data nodig hebben. Voor opzoeken van gegevens in globals gebruiken we lookups of repositories. Is er een reden ...

AxToAdmin is gemaakt om formattering uit te voeren en conversies te voorzien die geen data nodig hebben.
Voor opzoeken van gegevens in globals gebruiken we lookups of repositories.
Is er een reden waarom we geen persistent class gebruiken en een repository die de standaard geef, maak, bestaat etc.. aanbiedt ?

[UST3344] VerzendWijze mapping AX <-> Admin. Waarschuwing bij Admin enu.
[UST3344] VerzendWijze mapping AX <-> Admin. Waarschuwing bij Admin enu.
[UST3344] VerzendWijze mapping AX <-> Admin. Verzendwijze naar AX vertalen vanuit vertaaltabel.
    • -1
    • +1
    ./Converter/OrderHoofdingConverteerder.cls.xml
  1. … 3 more files in changeset.