VerzendWijze.cls.xml

Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
[ICT-5256][Caché] Mapping verzendwijze 'Opo' bij syncen van klant OPO Duitsland
[ICT-5256][Caché] Mapping verzendwijze 'Opo' bij syncen van klant OPO Duitsland
Displaylist en valuelist zijn uitgebreid.

Displaylist en valuelist zijn uitgebreid.

aan de valuelist (en hieronder bij de displaylist, ook nog het item toevoegen) (en hier ook op de spaties letten)

aan de valuelist (en hieronder bij de displaylist, ook nog het item toevoegen)

(en hier ook op de spaties letten)

[ICT-2962] MaakVerzendWijze Gefaald: Nieuwe Verzendwijze nodig in Cache:
[ICT-2962] MaakVerzendWijze Gefaald: Nieuwe Verzendwijze nodig in Cache:
Is inderdaad helemaal niet mijn keuze(KPR) momenteel wordt de extra enum enkel gebruikt om zo snel mogelijk te vertalen naar een 'klassieke' verzendwijze. Als er natuurlijk ergens stukken code zij...

Is inderdaad helemaal niet mijn keuze(KPR)

momenteel wordt de extra enum enkel gebruikt om zo snel mogelijk te vertalen naar een 'klassieke' verzendwijze. Als er natuurlijk ergens stukken code zijn die die valuelist itereren en er iets zots mee doen... maar op het eerste zicht is dat niet direct een risico. Valuelist wordt in een paar AdresUtils methods en nog wat andere dingen gebruikt.

Het is misschien niet uw eigen keuze, maar "DiHF" bevat kleine letter i. Persoonlijk zou ik kiezen om alles in hoofdletters te zetten. Maar wellicht is dat nu al te laat. Tweede bedenking : is het...

Het is misschien niet uw eigen keuze, maar "DiHF" bevat kleine letter i.
Persoonlijk zou ik kiezen om alles in hoofdletters te zetten. Maar wellicht is dat nu al te laat.

Tweede bedenking : is het overal oké om een code van 4 letters te nemen? Dit is de eerste in de lijst, zie ik

[ICT-2806] Interface klanten - verzendwijze DiHF
[ICT-2806] Interface klanten - verzendwijze DiHF
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-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.