Activity CacheAdminA_trunk2010/vhUnitTest/ECONtools/PM/Maatwerk/dto/ProductConverter/Converteer

11:07 am

Jo Claes

More
Re: CR-11
commented on review

In dit geval lijkt het me beter om in het vervolg een dummy TBXLadeCodeBepaler te gebruiken zodat je niet iedere keer die lijn met ).DanDoeNiks() hebt Wat ik wel raar vind is dat de method een waa...

In dit geval lijkt het me beter om in het vervolg een dummy TBXLadeCodeBepaler te gebruiken zodat je niet iedere keer die lijn met ).DanDoeNiks() hebt

Wat ik wel raar vind is dat de method een waarde retourneert en je een DanDoeNiks() method gebruikt .. het zou misschien beter zijn om .DanReturn() te doen of een method aan te maken .DanReturnNiks()
kwestie om daar geen verwarring in te hebben

9:47 am

Jo Claes

More
CR-11
finished reviewing

9:46 am

Jo Claes

More
Re: CR-11
commented on review

Waarom is dit geen enumeratie ??? Ik weet niet of het een piste is, maar het lijkt mij handig dat we enumeratie hebben van kenmerknamen in de verschillende delen, zodat de enumeratie naam hetzelfd...

Waarom is dit geen enumeratie ???

Ik weet niet of het een piste is, maar het lijkt mij handig dat we enumeratie hebben van kenmerknamen in de verschillende delen, zodat de enumeratie naam hetzelfde is maar de waarde verschillend.

Kunnen dan op dat soort enumeratie een extra parameter met values zetten zodat we waarden kunnen converteren van het ene naar het andere

9:37 am

Jo Claes

More
CR-11
joined review

9:37 am

Jo Claes

More
CR-11
started review

[UW324] conversie zijwandverhogingtype van ext naar edi

7:50 am

Tom Vermeulen

More
Re: CR-9
replied to Jo Claes

Dacht dat over een paar dagen/weken de converter ook LBX en TAX ging moeten verwerken, daarom geen TBXKenmerken in naam. De tests hebben wel TBX in hun naam. Andere convertoren behandelen nu ook al...

Dacht dat over een paar dagen/weken de converter ook LBX en TAX ging moeten verwerken, daarom geen TBXKenmerken in naam. De tests hebben wel TBX in hun naam. Andere convertoren behandelen nu ook alle types schuifkes.

5:18 pm

Jo Claes

More
CR-9
finished reviewing

5:18 pm

Jo Claes

More
Re: CR-9
commented on review

In de naamgeving van de klasse ontbreekt het feit dat het over TbxKenmerken gaat en ECONimpl is een root package cfr AXimpl en WSimpl Dus eigenlijk zou ECONimpl.PM.Maatwerk.TbxKenmerkenEdiConverte...

In de naamgeving van de klasse ontbreekt het feit dat het over TbxKenmerken gaat en ECONimpl is een root package
cfr AXimpl en WSimpl

Dus eigenlijk zou ECONimpl.PM.Maatwerk.TbxKenmerkenEdiConverter een correctere naam zijn --> natuurlijk is het dan noodzakelijk om de UT's hun naam te wijzigen

5:12 pm

Jo Claes

More
Re: CR-9
commented on review

Is eigenlijk een beetje muggenziften maar het zou nog dat tikkeltje beter zijn mocht de code er ongeveer zo uitzien do ..AssertEdiNaarEcon(##class(vhTest.Utils.APPS.EDI.common.dto.LadeTBX).Standaa...

Is eigenlijk een beetje muggenziften maar het zou nog dat tikkeltje beter zijn mocht de code er ongeveer zo uitzien

do ..AssertEdiNaarEcon(##class(vhTest.Utils.APPS.EDI.common.dto.LadeTBX).StandaardLade(),##class(vhTest.Utils.ECON.PM.Maatwerk.dto.TbxKenmerken).StandaardLade())

2:49 pm