CacheAdminA_trunk2010/vhUnitTest/AXimpl/Dynamics/CustomerIn/Converter/LeverAdresUtils/ZetLeverAdressenBestaandeKlant

Test.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): 0
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:
Wednesday 21 Feb 2018
1:46 pm

Tom Vermeulen

1:17 pm

Tom Vermeulen

Aangezien de oproepende functie het doorgeeft via SQL is dat nog niet echt nodig. (en code moest rap werkend zijn http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/...

Aangezien de oproepende functie het doorgeeft via SQL is dat nog niet echt nodig. (en code moest rap werkend zijn )

Zal het er wel bijsteken voor de volledigheid

11:19 am

Tommy Hebb

11:10 am

Tommy Hebb

Moet er geen + voor OrderHeader.CustAccount staan, zodat de voorloopnul die klantnummers in AX eventueel hebben eraf gehaald wordt?

Moet er geen + voor OrderHeader.CustAccount staan, zodat de voorloopnul die klantnummers in AX eventueel hebben eraf gehaald wordt?

11:04 am

Tommy Hebb

Moet deze niet manueel verwijderd worden?

Moet deze niet manueel verwijderd worden?

11:04 am

Tommy Hebb

Moet deze niet manueel verwijderd worden?

Moet deze niet manueel verwijderd worden?

11:01 am

Tommy Hebb

KlantId of KlantNummer?

KlantId of KlantNummer?

11:01 am

Tommy Hebb

KlantId of KlantNummer?

KlantId of KlantNummer?

10:54 am

Tommy Hebb

Moet er geen + voor OrderHeader.CustAccount staan, zodat de voorloopnul die klantnummers in AX eventueel hebben eraf gehaald wordt?

Moet er geen + voor OrderHeader.CustAccount staan, zodat de voorloopnul die klantnummers in AX eventueel hebben eraf gehaald wordt?

8:47 am

Tom Vermeulen

[UST3670] LeverAdresRepository aanpassen
[UST3670] LeverAdresRepository aanpassen
Tuesday 20 Feb 2018
3:37 pm

Tom Vermeulen

[UST3670] LeverAdresRepository aanpassen

- BestaatViaAXLeverAdresIdEnKlantNummer toegevoegd

(inclusief hele hoop wlipkes maar die gaan nog weg)

Options
    • -14
    • +14
    ./Test.cls.xml
  2. … 10 more files in changeset.
3:12 pm

Tom Vermeulen

[UST3670] LeverAdresRepository aanpassen

- GeefViaAXLeverAdresIdEnKlantNummer toegevoegd

(inclusief hele hoop wlipkes maar die gaan nog weg)

Options
    • -8
    • +12
    ./Test.cls.xml
  2. … 10 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:
2:29 pm

Tommy Hebb

[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

Options
    • -59
    • +79
    ./Test.cls.xml
  2. … 3 more files in changeset.
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
    • -9
    • +9
    ./Test.cls.xml
  2. … 5 more files in changeset.
10:46 am

Tommy Hebb

[UST3647] VHIP900: testbaar maken van LeverAdressenConversie

- Variatietest: Mixed case: Klant krijgt adressen 1,2,3,4 binnen, 2,4 hebben Delivery-rol, 2 heeft gekend AxLocationId, er is bestaand adres 5 -> 2 aanpassen, 4 opslaan, 1,3 negeren, 5 verwijderen

Options
    • -0
    • +34
    ./Test.cls.xml
10:33 am

Tommy Hebb

[UST3647] VHIP900: testbaar maken van LeverAdressenConversie

- Variatietest: 5 adressen met nieuwe AxLocationId waarvan 3 niet de Delivery-rol hebben => deze laatste negeren en 2 andere opslaan

Options
    • -0
    • +29
    ./Test.cls.xml
Thursday 08 Feb 2018
5:06 pm

Tommy Hebb

[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

Options
    • -0
    • +16
    ./Test.cls.xml
  2. … 1 more file in changeset.
3:32 pm

Tom Vermeulen

[UST3647] VHIP900: testbaar maken van LeverAdressenConversie

- Commit zodat THB kan verderdoen

Options
    • -2
    • +26
    ./Test.cls.xml
  2. … 1 more file in changeset.
2:30 pm

Tom Vermeulen

[UST3647] VHIP900: testbaar maken van LeverAdressenConversie

- Extra test voor verwijderen/toevoegen combo

Options
    • -0
    • +24
    ./Test.cls.xml
2:24 pm

Tom Vermeulen

[UST3647] VHIP900: testbaar maken van LeverAdressenConversie

- Extra test voor verwijderen

Options
    • -0
    • +45
    ./Test.cls.xml