CacheAdminA_trunk2010/vhUnitTest/ECONtools/PM

Maatwerk

Checkout Tools

Line History

Line Count Graph
Line Count Graph

Stats

Commits this week: 0
Total Committers: 1
Last Commit: 05 Jan 17
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 10 Jan 2017
11:07 am

Tom Vermeulen

11:07 am

Tom Vermeulen

Friday 06 Jan 2017
9:56 am

Jo Claes

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

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
[UW324] conversie zijwandverhogingtype van ext naar edi
7:50 am

Tom Vermeulen

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.

Thursday 05 Jan 2017
5:18 pm

Jo Claes

  • More
  • CR-9
  • finished reviewing
5:18 pm

Jo Claes

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

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

Tom Vermeulen

  • More
  • CR-9
  • started review
[UST2040] Vh4 TBX B.E.: e-Con naar Edi mapping van ingegeven kenmerken TBX
[UST2040] Vh4 TBX B.E.: e-Con naar Edi mapping van ingegeven kenmerken TBX