CacheAdminA_trunk2010/vhTest/Mock/AXimpl/Dynamics/CustomerIn/Converter

LeverAdresUtils.cls.xml

Checkout Tools

Line History

Line Count Graph
Line Count Graph

Stats

Commits this week: 0
Total Committers: 2
Last Commit: 27 Feb 18
Commits this week: 0
Total Lines of Code (LoC): 26
Net change in LoC this week: 0

Commit Activity

Commits By Day In Week
52 week commits volume
Commits By Day In Week
Commits by day
Commits By Day In Week
Commits by hour
Commits By Hour In Day
Commit calendar
 

Most active committers (90 days)

loading
loading
  • last updated a few seconds ago
All Commits Reviews
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Tuesday 27 Feb 2018
2:37 pm

Tommy Hebb

1:36 pm

Kurt Callebaut

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.

1:32 pm

Kurt Callebaut

1:27 pm

Kurt Callebaut

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.

12:21 pm

Tommy Hebb

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

Tommy Hebb

[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

Options
    • -2
    • +2
    ./LeverAdresUtils.cls.xml
  2. … 8 more files in changeset.
Wednesday 14 Feb 2018
2:38 pm

Tommy Hebb

2:37 pm

Tommy Hebb

Verwijderd!

Verwijderd!

1:44 pm

Kurt Callebaut

  • More
  • CR-998
  • finished reviewing
1:39 pm

Kurt Callebaut

Overbodig.

Overbodig.

Monday 12 Feb 2018
3:12 pm

Tommy Hebb

[UST3652] VHIP900 - Grondige refactoring:
[UST3652] VHIP900 - Grondige refactoring:
Friday 09 Feb 2018
3:30 pm

Tommy Hebb

[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)

Options
    • -17
    • +4
    ./LeverAdresUtils.cls.xml
  2. … 5 more files in changeset.
Wednesday 31 Jan 2018
3:00 pm

PieterVR

[UST3344] Verzendwijze conversie tussen Admin en AX: review verwerkingen.
Options
    • -2
    • +2
    ./LeverAdresUtils.cls.xml
  2. … 9 more files in changeset.
Wednesday 27 Dec 2017
12:07 pm

Tommy Hebb

10:01 am

Tom Vermeulen

  • More
  • CR-842
  • finished reviewing
Friday 22 Dec 2017
4:55 pm

Tom Vermeulen

andere exceptie nemen

andere exceptie nemen

2:31 pm

Tommy Hebb

[UST3397] VHIP900: Sync klant: leveradressen: Bestaande Klant:
[UST3397] VHIP900: Sync klant: leveradressen: Bestaande Klant:
2:30 pm

Tommy Hebb

[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

Options
    • -0
    • +39
    ./LeverAdresUtils.cls.xml
  2. … 8 more files in changeset.