Wim Vermeulen

12:47 pm

Wim Vermeulen

More
Re: CR-2763
commented on review

Over de Z-positie (en Size) van de Rug wil ik effe mondeling met jou afstemmen. Kan best op dinsdag, lijkt me.

Over de Z-positie (en Size) van de Rug wil ik effe mondeling met jou afstemmen.
Kan best op dinsdag, lijkt me.

12:32 pm

Wim Vermeulen

More
Re: CR-2763
commented on review

De uitdrukking voor Center.Z komt ondertussen ook 4x voor --> parameter van maken? Of bij Uitfrezing Rug staat dezelfde code maar dan "... + 2" i.p.v. "... + 8" Hier weet jij wel raad mee hé.

De uitdrukking voor Center.Z komt ondertussen ook 4x voor --> parameter van maken?
Of bij Uitfrezing Rug staat dezelfde code maar dan "... + 2" i.p.v. "... + 8"
Hier weet jij wel raad mee hé.

12:25 pm

Wim Vermeulen

More
Re: CR-2763
commented on review

comment aanpassen naar "Uitfrezing MAT Sifonuitsparing Rechts" (of ondertussen in t Engels)

12:24 pm

Wim Vermeulen

More
Re: CR-2763
commented on review

comment aanpassen naar "Uitfrezing MAT Sifonuitsparing Links" (of ondertussen in t Engels)

12:24 pm

Wim Vermeulen

More
Re: CR-2763
commented on review

comment aanpassen naar "Uitfrezing MAT Spoelbak" (of ondertussen in t Engels)

12:19 pm

Ik denk dat in de nieuwere versies alle parameters reeds in het Engels staan. Dat is hier geen review-opmerking waard. Te negeren dus http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/w...

Ik denk dat in de nieuwere versies alle parameters reeds in het Engels staan.
Dat is hier geen review-opmerking waard. Te negeren dus

12:16 pm

Wim Vermeulen

More
Re: CR-2763
commented on review

De uitdrukkingen (UitsparingData_Z1_Breedte/2) + 3 + UitsparingData_X_Breedte UitsparingData_Z1_Breedte + 3 + UitsparingData_X_Breedte - AfstandBinnenkantCorpusTotBinnenkantLade + UitsparingDat...

De uitdrukkingen

 (UitsparingData_Z1_Breedte/2) + 3 + UitsparingData_X_Breedte

 UitsparingData_Z1_Breedte + 3 + UitsparingData_X_Breedte - AfstandBinnenkantCorpusTotBinnenkantLade + UitsparingData_M_Breedte + (UitsparingData_Z2_Breedte/2)

komen zeker 4x exact hetzelfde voor in deze json
Lijkt me handig als je deze dan eerst in een variabele steekt, met een sprekende naam :-P
In de naam misschien iets in de aard van "Offset.PostionFromLeft.Z1" en "Offset.PostionFromLeft.Z2" of zoiets (meestal zijn jou naamgevingen beter dan mijn suggesties

11:56 am

Wim Vermeulen

More
CR-2878
finished reviewing

4:55 pm

Wim Vermeulen

More
CR-2872
Closed

2:26 pm

Wim Vermeulen

More
CR-2872
started review

[ICT-3847][rvMCL] PRJ029 - Uitfaseren WWW03 : ProductInfo via Vhisie4 : code productie-waardig maken...

6:49 pm

Wim Vermeulen

More
CR-2853
Closed

6:48 pm

Wim Vermeulen

More
Re: CR-2853
commented on review

Tom Vermeulen De waarden in de config-items komen al niet meer overeen met de waarden in de deploy-klasse. Voor sommige items is er reeds een nieuwe deploy-klasse, dus de vorige vervalt sowieso. Du...

Tom Vermeulen
De waarden in de config-items komen al niet meer overeen met de waarden in de deploy-klasse.
Voor sommige items is er reeds een nieuwe deploy-klasse, dus de vorige vervalt sowieso. Dus lijkt me beter dat je deze niet meer terugvindt als je een "Zoek" is caché start.

2:38 pm

Wim Vermeulen

More
CR-2853
started review

[ICT-5325][rvTVE] PRJ029 Ken je assets Print01 : verwijzingen naar Watchfolder op Print01 in ConfigItems...

11:21 am

Wim Vermeulen

More
CR-2631
Closed

11:14 am

Wim Vermeulen

More
CR-2832
finished reviewing

11:13 am

Wim Vermeulen

More
CR-2833
Closed

4:33 pm

Wim Vermeulen

More
CR-2833
started review

[ICT-5186][rvNVT] FOP : aanpassing layout leverbon, op blanco papier i.p.v. logo-papier

3:36 pm

Wim Vermeulen

More
Re: CR-2832
commented on review

Deze lijn staat binnen de For-loop. Is dat de bedoeling? Volgens mij beter na de loop uitvoeren. Dan toch best effe testen hé http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icon...

Deze lijn staat binnen de For-loop.
Is dat de bedoeling? Volgens mij beter na de loop uitvoeren. Dan toch best effe testen hé

5:08 pm

Wim Vermeulen

More
Re: CR-2832
commented on review

De pxLijn toevoegen kan je best toch conditioneel maken, hé. Anders If List.Count() > 0 then pxLijn maken en toevoegen.

De pxLijn toevoegen kan je best toch conditioneel maken, hé.
Anders

If List.Count() > 0 then pxLijn maken en toevoegen.

5:52 pm

Wim Vermeulen

More
CR-2809
Closed

5:50 pm

Wim Vermeulen

More
Re: CR-2809
replied to Tom Vermeulen

OnAfterAllTests() toegevoegd.

4:25 pm

Wim Vermeulen

More
Re: CR-2809
commented on review

@Tom : deze klasse is een vereenvoudigde kopie van "APPS.common.Document.impl.DocBase.WebServerUrlBepaler" voor FOP. Geen inheritance of uncle Bob-principes toegepast http://subversion02.vanhoecke....

@Tom : deze klasse is een vereenvoudigde kopie van "APPS.common.Document.impl.DocBase.WebServerUrlBepaler" voor FOP.
Geen inheritance of uncle Bob-principes toegepast

4:09 pm