CacheAdminA_trunk2010/AX/Uitgaand/PM/impl

Convertor

Checkout Tools

Line History

Line Count Graph
Line Count Graph

Stats

Commits this week: 0
Total Committers: 2
Last Commit: 27 Jun 17
Commits this week: 0
Total Lines of Code (LoC): 494
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 23 Jan 2018
2:42 pm

Tommy Hebb

2:02 pm

Tom Vermeulen

  • More
  • CR-944
  • finished reviewing
10:50 am

Tommy Hebb

Is nu A en B, met beter geformuleerde asserts (AantalA, BestaatA, ...) en een verbeterde VoegToeProductOrderLijn() en VerwijderData(), die tegelijkertijd nu makkelijker uitbreidbaar zijn.

Is nu A en B, met beter geformuleerde asserts (AantalA, BestaatA, ...) en een verbeterde VoegToeProductOrderLijn() en VerwijderData(), die tegelijkertijd nu makkelijker uitbreidbaar zijn.

10:07 am

Tommy Hebb

Na overleg hebben we dit issue laten vallen wegens het sop dat de kolen niet waard is?

Na overleg hebben we dit issue laten vallen wegens het sop dat de kolen niet waard is?

9:42 am

Tommy Hebb

Zoals besproken LegacyPartijAPI.GeefKlantNummerVoorPartijID() gebruikt, wegens niet voor AX bedoeld, maar voor Caché-intern gebruik.

Zoals besproken LegacyPartijAPI.GeefKlantNummerVoorPartijID() gebruikt, wegens niet voor AX bedoeld, maar voor Caché-intern gebruik.

8:19 am

Tom Vermeulen

testje? $Char(0) is nl een tricky geval

testje? $Char(0) is nl een tricky geval

8:15 am

Tom Vermeulen

Ik weet dat dit op veel plaatsen gebruikt wordt, maar er bestaat ook de AXimpl.AdminToAxUtils (die gaat wel lpadden). En eigenlijk is dat misschien wel juister, aangezien ge hier overal met AX-klan...

Ik weet dat dit op veel plaatsen gebruikt wordt, maar er bestaat ook de AXimpl.AdminToAxUtils (die gaat wel lpadden). En eigenlijk is dat misschien wel juister, aangezien ge hier overal met AX-klantids aan het werken zijt (dus hoe dan ook 5 karakters lang (ook in repo))

8:07 am

Tom Vermeulen

Ik zou deze elvendertig calls ook in de onbeforeonetest steken. Op het eerste zicht zie ik maar 1 verwachtmethodcall hierop en principe overrulet die de reeds bestaande indienmethodcall. Op die man...

Ik zou deze elvendertig calls ook in de onbeforeonetest steken. Op het eerste zicht zie ik maar 1 verwachtmethodcall hierop en principe overrulet die de reeds bestaande indienmethodcall. Op die manier blijven de testen wat meer beperkt tot hun essentie. (of ik heb mij miskeken)

8:04 am

Tom Vermeulen

Ik zou al die namen met 8'en en 9's hernoemen. Vind het momenteel nogal moeilijk om direct te zien wat ze betekenen. Wat doet VoegToeProductOrdelijn8(9) bv? Kan je die twee methods al niet dezelfde...

Ik zou al die namen met 8'en en 9's hernoemen. Vind het momenteel nogal moeilijk om direct te zien wat ze betekenen. Wat doet VoegToeProductOrdelijn8(9) bv?
Kan je die twee methods al niet dezelfde naam geven en een extra parameter gebruiken (al dan niet gedefault)? Dat bespaart weeral een (bijna) duplicate private method. En waarom 8 en 9 en niet 1 en 2? precies dat de eerste 7 nog ergens anders staan

Monday 22 Jan 2018
2:29 pm

Tommy Hebb

[UST3554] VHIP906 Nachtelijke controle verkooporders: Tuning:
[UST3554] VHIP906 Nachtelijke controle verkooporders: Tuning:
Tuesday 24 Oct 2017
11:06 am

Tommy Hebb

10:04 am

Tom Vermeulen

  • More
  • CR-661
  • finished reviewing
Thursday 19 Oct 2017
11:07 am

Tommy Hebb

[UST3018] 903 Voorraadsync: Aanpassing SoprContractIdConverter:
[UST3018] 903 Voorraadsync: Aanpassing SoprContractIdConverter:
Friday 22 Sep 2017
1:33 pm

Tommy Hebb

11:58 am

Tom Vermeulen

  • More
  • CR-557
  • finished reviewing
11:49 am

Tommy Hebb

[UST2877] VHIP migratie - BlumKunde-aanpassing: notatie in AX moet prefix hebben 'vhbe-'
[UST2877] VHIP migratie - BlumKunde-aanpassing: notatie in AX moet prefix hebben 'vhbe-'
Thursday 24 Aug 2017
10:38 am

Tommy Hebb

Wednesday 23 Aug 2017
3:49 pm

Tom Vermeulen

  • More
  • CR-450
  • finished reviewing
1:56 pm

Tom Vermeulen

Je gebruikt principieel en principaal door elkaar, best vervangen door principaal (doe een CTRL-F, het zijn vijf files)

Je gebruikt principieel en principaal door elkaar, best vervangen door principaal (doe een CTRL-F, het zijn vijf files)

Monday 21 Aug 2017
2:19 pm

Tom Vermeulen

kan in 1 lijn (hierboven)

kan in 1 lijn (hierboven)

2:16 pm

Tom Vermeulen

%New in constructor

%New in constructor

2:14 pm

Tom Vermeulen

Beter een $IsObject ipv $$$HasLength als het over objecten gaat ipv strings. (jaja, untyped enz, maar toch)

Beter een $IsObject ipv $$$HasLength als het over objecten gaat ipv strings. (jaja, untyped enz, maar toch)

Friday 18 Aug 2017
5:08 pm

Tommy Hebb

[UST2729] VHIP903: Period. voorraad: SoprContract opnemen voor elke SOPR-variant:
[UST2729] VHIP903: Period. voorraad: SoprContract opnemen voor elke SOPR-variant:
Tuesday 01 Aug 2017
4:02 pm

Tommy Hebb

4:02 pm

Tom Vermeulen

  • More
  • CR-379
  • finished reviewing
2:17 pm

Tommy Hebb

2:16 pm

Tom Vermeulen

  • More
  • CR-380
  • finished reviewing
2:15 pm

Tom Vermeulen

't is misschien overdreven, maar hier zoudt ge Quit $$$If(KlantNummer="5005",..AxProductConverter.GeefBlumKunde(AdmlinOrderLijn.GeefProductID(),"") doe ermee wat ge wilt http://subversion02.vanho...

't is misschien overdreven, maar hier zoudt ge

Quit $$$If(KlantNummer="5005",..AxProductConverter.GeefBlumKunde(AdmlinOrderLijn.GeefProductID(),"")

doe ermee wat ge wilt

2:05 pm

Tom Vermeulen

standaardklant/-order gebruiken (en desnoods aanmaken in vhTest/Utils/...)

standaardklant/-order gebruiken (en desnoods aanmaken in vhTest/Utils/...)

11:04 am

Tommy Hebb

[UST2664] VHIP Migratie - Sync Sopr/ProductConvertor:
[UST2664] VHIP Migratie - Sync Sopr/ProductConvertor: