CustomerUpdatePart2Converter.cls.xml

Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
  • More
  • CR-281
  • finished reviewing
?? Iets te ijverig

?? Iets te ijverig

?? Iets te ijverig

?? Iets te ijverig

[ICT260] Voorbereiding interfaces
[ICT260] Voorbereiding interfaces
[ICT260] Voorbereiding interfaces

- MoveClass("AXimpl.AifMessage.Converter.CustomerUpdatePart2Converter","AXimpl.Dynamics.CustomerUpsertOut.Converter.CustomerUpdatePart2Converter")

    • -354
    • +0
    ./CustomerUpdatePart2Converter.cls.xml
  1. … 3 more files in changeset.
[ICT260] Voorbereiding interfaces

- MoveClass("AXimpl.AifMessage.Converter.CustomerCreateConverter","AXimpl.Dynamics.CustomerUpsertOut.Converter.CustomerCreateConverter")

    • -3
    • +3
    ./CustomerUpdatePart2Converter.cls.xml
  1. … 7 more files in changeset.
[ICT260] Voorbereiding interfaces

- gebruikers van "AX.common.impl.NoConversionNeededException" aangepast

    • -1
    • +1
    ./CustomerUpdatePart2Converter.cls.xml
  1. … 2 more files in changeset.
[ICT260] Voorbereiding interfaces

- MovePackage("AXaif.EntityKey.","AXif.Common.EntityKey.MessageParts.")

    • -5
    • +5
    ./CustomerUpdatePart2Converter.cls.xml
  1. … 15 more files in changeset.
[ICT260] Voorbereiding interfaces

- MoveClass("AXimpl.AifMessage.RequestComposer","AXimpl.RequestComposer")

    • -2
    • +2
    ./CustomerUpdatePart2Converter.cls.xml
  1. … 24 more files in changeset.
[ICT260] Voorbereiding interfaces

- MovePackage("AXaif.Customer.","AXif.Dynamics.CustomerOut.MessageParts.")

    • -25
    • +25
    ./CustomerUpdatePart2Converter.cls.xml
  1. … 6085 more files in changeset.
  • More
  • CR-190
  • finished reviewing
  • More
  • CR-191
  • resumed reviewing
[DEF572] - Klanten die op regio 98 - 99 geplaatst worden gaan niet goed naar AX

- Privatiseren

- Testje aangepast - verduidelijkt

    • -0
    • +1
    ./CustomerUpdatePart2Converter.cls.xml
  1. … 1 more file in changeset.
Private?

Private?

Is het niet nodig om het gedrag van deze method te verduidelijken dmv een paar testjes? Of door op te nemen in reeds bestaande tests? En ja, ik weet dat het een lokale (private) method is en dat ge...

Is het niet nodig om het gedrag van deze method te verduidelijken dmv een paar testjes? Of door op te nemen in reeds bestaande tests? En ja, ik weet dat het een lokale (private) method is en dat ge unittests schrijft en geen methodtests en jadajadajada, maar voor een leek zoals ik is het niet onmiddellijk duidelijk hoe het gewijzigde gedrag het probleem zoals omschreven in de titel oplost. VoilĂ .

Let the rebuttal begin...

private?

private?

[DEF571] - AX ContactInfo problemen met combinatie delete en update
[DEF571] - AX ContactInfo problemen met combinatie delete en update
[DEF572] - Klanten die op regio 98 - 99 geplaatst worden gaan niet goed naar AX
[DEF572] - Klanten die op regio 98 - 99 geplaatst worden gaan niet goed naar AX
[DEF571] - AX ContactInfo problemen met combinatie delete en update

- spatie vervangen door een $char(0) --> eens kijken hoe dit onthaald wordt

    • -1
    • +1
    ./CustomerUpdatePart2Converter.cls.xml
[DEF571] - AX ContactInfo problemen met combinatie delete en update

- lege waarde bleek niet te werken dan maar een spatie in zetten

    • -1
    • +1
    ./CustomerUpdatePart2Converter.cls.xml
  1. … 2 more files in changeset.
[DEF572] - Klanten die op regio 98 - 99 geplaatst worden gaan niet goed naar AX

- Hier en daar toch zeggen dat we data moeten updaten of anders lukt het toch niet goed

    • -0
    • +39
    ./CustomerUpdatePart2Converter.cls.xml
[DEF571] - AX ContactInfo problemen met combinatie delete en update

- delete vervangen door update met lege waarden

    • -1
    • +2
    ./CustomerUpdatePart2Converter.cls.xml
  1. … 1 more file in changeset.
[UST1737] - Basis functionaliteit klanten-update

- Het juiste type command gebruiken -> code werkte wel daar er maar een gedeelte uit het command wordt gehaalt , maar beter dat we het juiste command gebruiken ipv op toeval te rekenen.

    • -2
    • +2
    ./CustomerUpdatePart2Converter.cls.xml
Het is kort voor "Als de kredietlimiet 1 is (zo zichtbaar in Admin), dan moet er geen 1000 naar AX worden gestuurd, in andere gevallen vermenigvuldigen we wel met 1000 (en zetten wer er 20 nullen a...

Het is kort voor "Als de kredietlimiet 1 is (zo zichtbaar in Admin), dan moet er geen 1000 naar AX worden gestuurd, in andere gevallen vermenigvuldigen we wel met 1000 (en zetten wer er 20 nullen achter de komma bij)", maar dat vond ik te lang als omschrijving. (en caché vond dat ook)

  • More
  • CR-48
  • finished reviewing
Test benaming moet toch eerder zijn als kredietlimiet 1000 is dan geven we 1 door

Test benaming moet toch eerder zijn als kredietlimiet 1000 is dan geven we 1 door