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

8:52 am

Martijn Vierstraete

More
Re: CR-1612
commented on review

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)

9:49 am

Martijn Vierstraete

More
Re: CR-1612
commented on review

Verwarrend commentaar (vooral lijn 62 & 64)

9:42 am

Martijn Vierstraete

More
Re: CR-1612
commented on review

Test: Test:

11:06 am

Kurt Callebaut

More
Re: CR-1612
commented on review

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.

11:05 am

Kurt Callebaut

More
CR-1612
started review

[VRB604] MAG: Transport: GLS-koppeling combineren met verzendwijzes met langere transportduur?

2:37 pm

Kurt Callebaut

More
Re: CR-1029
commented on review

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

More
Re: CR-1029
commented on review

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