Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
[ICT-1630] [rvPVR] [900] - Interface klanten van AX naar Admin met 'raar' teken in naam van klant

- Gewone 'tick' ("´") toegevoegd aan niet-printbare karakters

    • -0
    • +7
    ./Converter/CustomerInConverter.cls.xml
  1. … 1 more file in changeset.
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?

- Voorbereiding - met landcode.

- code weg die niet meer gebruikt wordt.

    • -0
    • +7
    ./Converter/CustomerInConverter.cls.xml
  1. … 1 more file in changeset.
Hier zou ik denken, leeg adres mee te geven in de result en IsAxHoofdadresmeegegeven te zetten adhv isobject(leveradres), dan hoef je de boolean niet mee te ge ven byref.

Hier zou ik denken, leeg adres mee te geven in de result en IsAxHoofdadresmeegegeven te zetten adhv isobject(leveradres), dan hoef je de boolean niet mee te ge ven byref.

Volgens mij heb je hier moeilijk gemaakt door de byref parameter mee te geven. Enerzijds kan je een klasse maken met twee properties die zowel leveradres als de boolean IsAxHoofdadres als returnva...

Volgens mij heb je hier moeilijk gemaakt door de byref parameter mee te geven.

Enerzijds kan je een klasse maken met twee properties die zowel leveradres als de boolean IsAxHoofdadres als returnvalue geeft.

Beter nog is de 2 dingen die je doet in 1 method opsplitsen in 2 methods. nl bepalen of er een hoofdadres meegegeven is en het zetten ervan in een andere method.

[UST3689] VHIP900: Sync klant: onderzoek landcode Oostenrijk AT of AU:
[UST3689] VHIP900: Sync klant: onderzoek landcode Oostenrijk AT of AU:
[UST3689] VHIP900: Sync klant: onderzoek landcode Oostenrijk AT of AU:

- CustomerInConverter => mini-refactoring betreffende LanguageId

- CustomerInConverter => Bij UpdateKlant => ZetAdres => uitvoering is afhankelijk van of AX hoofdadres doorgaf => anders hoofdadres niet updaten

- LeverAdresUtils => MaakPrimairAdres => Indien geen hoofdadres doorgegeven door AX => Error loggen

- vhUnitTest...ZetLeverAdressenBestaandeKlant => moest nog verwijderd worden, want move class gebeurd naar vhUnitTest...ZetLeverAdressen wegens naamwijziging geteste method

  1. … 8 more files in changeset.
Verwijderd!

Verwijderd!

  • More
  • CR-998
  • finished reviewing
Overbodig.

Overbodig.

[UST3652] VHIP900 - Grondige refactoring:
[UST3652] VHIP900 - Grondige refactoring:
[UST3652] VHIP900 - Grondige refactoring:

- Afzondering ZetAdres naar eigen klasse.

    • -0
    • +20
    ./Converter/AdresZetter.cls.xml
  1. … 3 more files in changeset.
[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)

    • -17
    • +4
    ./Converter/LeverAdresUtils.cls.xml
  1. … 5 more files in changeset.
[UST3344] Verzendwijze conversie tussen Admin en AX: review verwerkingen.
  1. … 9 more files in changeset.
  • More
  • CR-842
  • finished reviewing
andere exceptie nemen

andere exceptie nemen

[UST3397] VHIP900: Sync klant: leveradressen: Bestaande Klant:
[UST3397] VHIP900: Sync klant: leveradressen: Bestaande Klant:
[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

    • -19
    • +0
    ./Converter/CustomerInConverter.cls.xml
    • -0
    • +39
    ./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

    • -0
    • +12
    ./Converter/CustomerInConverter.cls.xml
  1. … 5 more files in changeset.
[UST3324] VHIP900: Sync klant: nieuwe klant, allersimpelste velden

- refactor CustomerInConverter

- standaard gedrag nieuwe klant

    • -0
    • +33
    ./Converter/CustomerInConverter.cls.xml
  1. … 6 more files in changeset.