Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
[UST3670] LeverAdresRepository aanpassen

- timing opkuisen

    • -2
    • +1
    ./Converter/CustomerInConverter.cls.xml
    • -30
    • +0
    ./Converter/LeverAdresUtils.cls.xml
  1. … 1 more file in changeset.
[UST3670] LeverAdresRepository aanpassen

- BestaatViaAXLeverAdresIdEnKlantNummer toegevoegd

(inclusief hele hoop wlipkes maar die gaan nog weg)

  1. … 10 more files in changeset.
[UST3670] LeverAdresRepository aanpassen

- GeefViaAXLeverAdresIdEnKlantNummer toegevoegd

(inclusief hele hoop wlipkes maar die gaan nog weg)

    • -4
    • +33
    ./Converter/LeverAdresUtils.cls.xml
  1. … 10 more files in changeset.
[UST3667] VHIP900: Sync klant: aanpassing naam van klant geeft geen goed resultaat:

- Als eerste wordt de door AX aangeleverde Name getoetst of die eindigt op een geldige Juridische Vorm en deze wordt in Aanspreking gezet.

    • -6
    • +6
    ./Converter/CustomerInConverter.cls.xml
  1. … 3 more files in changeset.
[UST3663] VHIP900 KlantSync: Cache LeverAdresLijnId (L 1) uit veld Volmacht strippen:

- $$##L enz wordt van TransporterAuth afgestript bij het zetten van Volmacht in een Cache LeverAdres

  1. … 1 more file in changeset.
[UST3662] VHIP900 KlantSync: VerzendingWijze indien niet meegegeven door AX:

- Verzendingwijze leeg indien DlvMode niet meegegeven door AX en bovendien error gelogd

- Kleine aanpassing in detail van errorlog => LocationId wordt nu medegedeeld om te weten over welk LeverAdres het gaat

- Kleine refactoring CustomerInConverter-test => Private methods naar beneden verhuisd en achterhaalde UnitTest verwijderd => We zetten geen verzendingwijze meer op de klant zelf (hoofdadres) want AX stuurt dat niet mee

  1. … 2 more files in changeset.
[UST3658] VHIP900 - update klant - postcode:

- Enkel voor het HoofdAdres: Indien niet Belgisch => Postcode voorafgegaan door LandCode en een streepje

- In test moest VerzendingWijze nog genegeerd worden, want AX zal dat daar niet meesturen en we willen die data op het klantobject dan maar behouden

  1. … 1 more file in changeset.
[UST3659] VHIP900 - verzendwijze op klant:

- ZetVerzendingWijze niet doen op de Klant, enkel op de LeverAdressen

- ZetKlantAdresNr ophalen van het bestaande LeverAdres

    • -2
    • +0
    ./Converter/CustomerInConverter.cls.xml
  1. … 1 more file in changeset.
[UST3652] VHIP900 - Grondige refactoring:

- Test op BestaatViaAXLeverAdresId is voldoende, het is niet nodig eerst nog te checken of de klant bestaat (als de klant niet bestaat, bestaat de AXLeverAdresId toch ook niet)

[UST3629] VHIP900: Sync klant: issue met naam van klant:

- => AxToAdminUtils => ConvertCustNameToNaamEnToenaam

- Indien Name <= 28 -> Naam = Name, Toenaam = leeg

- Indien Name > 28 -> Naam = Name tot laatste spatie binnen eerste 28 karakters, Toenaam = overschot

- Indien daarna Toenaam > 28 -> Toenaam begrensd tot 28 + Warning loggen

    • -12
    • +4
    ./Converter/CustomerInConverter.cls.xml
  1. … 4 more files in changeset.
[UST3629] VHIP900: Sync klant: issue met naam van klant:

- Aanspreking mag voortaan leeg blijven

- Toenaam zal anders ingevuld worden, voor nu mag het leeg blijven

    • -27
    • +5
    ./Converter/CustomerInConverter.cls.xml
  1. … 1 more file in changeset.
[UST3641] VHIP900 - naam van leveradres:

- Description (Naam van HoofdAdres) => lengte afkapping op 27 karakters

- Description (Naam van LeverAdres) => lengte afkapping op 25 karakters

  1. … 1 more file in changeset.
[UST3644] VHIP900 - leveradres : opm1/opm2/volmacht:

- Lengte Opmerking1 en Opmerking2 is maximaal 40

- Lengte Volmacht is maximaal 35

  1. … 1 more file in changeset.
[UST3652] VHIP900 - Grondige refactoring:

- AdresZetter => Op Telefoon voor LeverAdres na worden er geen properties meer opgehaald van een bestaand adres => worden eventueel leeggemaakt vanaf nu

- LeverAdresUtils => LandRepository verwijderd wegens niet meer nodig

- ZetLeverAdressenBestaandeKlant => Aanpassing UnitTests => ZetAdres wordt nu uitgemockt

  1. … 2 more files in changeset.
[UST3652] VHIP900 - Grondige refactoring:

- Afzondering ZetAdres naar eigen klasse.

    • -0
    • +103
    ./Converter/AdresZetter.cls.xml
    • -58
    • +9
    ./Converter/LeverAdresUtils.cls.xml
  1. … 2 more files in changeset.
[UST3652] VHIP900 - Grondige refactoring:

- Boolean als parameter in ZetAdres om te bepalen of het voor MaakPrimairAdres is, zodat private method RolesContains niet in de weg zit om ZetAdres af te zonderen naar aparte klasse.

[UST3652] VHIP900 - Grondige refactoring:

- Er is nu een enkele ZetLeverAdressen, ipv de ZetLeverAdressenNieuweKlant en de ZetLeverAdressenBestaandeKlant

- Er is nu een enkele ZetAdres, ipv de ZetNieuwLeverAdres en de ZetGemeenschappelijkePropertiesOpAdres

- MaakVerzendingWijze handelt nu zelf alle nodige logica af (o.a. betreffende InsertOfUpdate)

    • -76
    • +42
    ./Converter/LeverAdresUtils.cls.xml
    • -4
    • +4
    ./Converter/CustomerInConverter.cls.xml
  1. … 4 more files in changeset.
[UST3647] VHIP900: testbaar maken van LeverAdressenConversie

- Test 1 PostalAddress heeft niet de Delivery-rol en AxLocationId is niet gekend => negeren

- ZetNaam => geen piece meer van Description, maar gewoon Description

- ZetVolmacht => geen piece meer volgens delimiter, want die is er niet meer

- ZetAanspreking => Altijd leeg maken => In Ax staat de aanspreking geconcateneerd achter de Naam en dat mag zo blijven

  1. … 1 more file in changeset.
[UST3647] VHIP900: testbaar maken van LeverAdressenConversie

- Commit zodat THB kan verderdoen

  1. … 1 more file in changeset.
[UST3647] VHIP900: testbaar maken van LeverAdressenConversie

- Beetje cleanup

    • -15
    • +1
    ./Converter/LeverAdresUtils.cls.xml
[UST3647] VHIP900: testbaar maken van LeverAdressenConversie

- Eerste versie van verwijderlogica

    • -0
    • +23
    ./Converter/LeverAdresUtils.cls.xml
  1. … 2 more files in changeset.
[UST3647] VHIP900: testbaar maken van LeverAdressenConversie

- nieuwe test (1 bestand + 1 nieuw adres)

  1. … 1 more file in changeset.
[UST3647] VHIP900: testbaar maken van LeverAdressenConversie

- Eerste minimale test gemaakt

  1. … 1 more file in changeset.
[UST3647] VHIP900: testbaar maken van LeverAdressenConversie

- MoveClass("AXimpl.Dynamics.CustomerIn.Converter.LeverAdresUtils.ZetLeverAdressenBestaandeKlant.Test","vhUnitTest.AXimpl.Dynamics.CustomerIn.Converter.LeverAdresUtils.ZetLeverAdressenBestaandeKlant.Test")

[UST3647] VHIP900: testbaar maken van LeverAdressenConversie

- Eerste volledig lege testklasse

[UST3344] Verzendwijze conversie tussen Admin en AX: review verwerkingen.
    • -2
    • +2
    ./Converter/CustomerInConverter.cls.xml
  1. … 8 more files in changeset.
[UST3397] VHIP900: Sync klant: leveradressen: Bestaande Klant:

- Refactoring => incl afzonderen code ivm adressen naar nieuwe klasse LeverAdresUtils

- Klasse Land is nu TECH.Interface => Mockable nu

- Aanpasssen UnitTests => Op 1 na die nu in comment staat => PRA gaat die nakijken

    • -150
    • +11
    ./Converter/CustomerInConverter.cls.xml
    • -0
    • +209
    ./Converter/LeverAdresUtils.cls.xml
  1. … 7 more files in changeset.
[UST3397] VHIP900: Sync klant: leveradressen: Bestaande Klant:

- LeveradresRepository => Uitbreiding Bewaar()

- ZetLeverAdressenBestaandeKlant()

- DataMKlantNative => GeefAdressenVolgensType is nu Public ipv Private

- To do => UnitTests aanpassen en nieuwe

    • -23
    • +90
    ./Converter/CustomerInConverter.cls.xml
  1. … 5 more files in changeset.
[UST3391] VHIP900: Sync klant: leveradressen: Nieuwe Klant:

- Refactoring gemeenschappelijke code => ZetGemeenschappelijkePropertiesOpAdres

    • -28
    • +28
    ./Converter/CustomerInConverter.cls.xml
  1. … 1 more file in changeset.
[UST3391] VHIP900: Sync klant: leveradressen: Nieuwe Klant:

- Property Toenaam wordt opgegeven

- Veld Address.Nota1External gaat dan toch niet het Telefoon(nummer) bevatten

- De 2 Opmerking-velden gaan dan toch 1 op 1 mappen

    • -5
    • +2
    ./Converter/CustomerInConverter.cls.xml