Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
[UST4026] VHIP900 - Business & primary - 1 wijziging terugdraaien:

- Na review => IsPrimary ipv IsPrimory

Primary ipv Primory (ook hieronder en hierboven)

Primary ipv Primory (ook hieronder en hierboven)

[UST4026] VHIP900 - Business & primary - 1 wijziging terugdraaien:
[UST4026] VHIP900 - Business & primary - 1 wijziging terugdraaien:
VHIP900 - Business & primary - 1 wijziging terugdraaien:

- IsPrimary moet Yes zijn opdat het om het hoofdadres gaat

  1. … 1 more file in changeset.
[UST3980] VHIP900 SyncKlant - Business & Primary bij adressen:
[UST3980] VHIP900 SyncKlant - Business & Primary bij adressen:
[UST3980] VHIP900 SyncKlant - Business & Primary bij adressen:

- Hoofdadres => IsPrimary op een adres wordt in AX niet meer onderhouden en is dus niet langer betrouwbaar

  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:

- TestCase voor MaakPrimairAdres van LeverAdresUtils

  1. … 1 more file in changeset.