CacheAdminA_trunk2010/vhUnitTest/DOM/PM/impl/IKCanonischeWaarde

TestTAORKenmerken.cls.xml

Checkout Tools

Line History

Line Count Graph
Line Count Graph

Stats

Commits this week: 0
Total Committers: 9
Last Commit: 23 Nov 22
Commits this week: 0
Total Lines of Code (LoC): 172
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
Thursday 01 Dec 2022
11:38 am

Joannes Lemmens

10:53 am

Tommy Hebb

Wednesday 30 Nov 2022
2:45 pm

Joannes Lemmens

[ICT-3198] TAX - Dubbele dwarsverdeling - Caché gedeelte
[ICT-3198] TAX - Dubbele dwarsverdeling - Caché gedeelte
Monday 07 Nov 2022
11:06 am

Tommy Hebb

Wednesday 02 Nov 2022
6:51 pm

Wim Vermeulen

6:51 pm

Wim Vermeulen

TestFiles reviewen, is in dit soort stories bijna onbegonnen werk. Ik heb zo goed en zo kwaad mogelijk de changes bekeken. Alvast bedankt om het aantal testfiles te filteren. Het geheel lijkt me we...

TestFiles reviewen, is in dit soort stories bijna onbegonnen werk. Ik heb zo goed en zo kwaad mogelijk de changes bekeken.
Alvast bedankt om het aantal testfiles te filteren.
Het geheel lijkt me wel oké te zijn.

Ook in eCon enkele commits gereviewed.

6:38 pm

Wim Vermeulen

De dossiercode "LI7K" was blijkbaar een onbedoelde tussenstap (Tricky situatie met de TAOR Losse componenten) In combinatie met svn rev. 69413 is dit wel een correcte overgang. Hierbij is deze cha...

De dossiercode "LI7K" was blijkbaar een onbedoelde tussenstap (Tricky situatie met de TAOR Losse componenten)
In combinatie met svn rev. 69413 is dit wel een correcte overgang.

Hierbij is deze change nagekeken en goedgekeurd!

Friday 21 Oct 2022
10:35 am

Tommy Hebb

De meeste testfiles heb ik maar buiten de review gehouden. Een aantal zijn toch toegevoegd, bij wijze van check van het resultaat is dat, denk ik, wel ok en het is bovendien soms ook moeilijk om ui...

De meeste testfiles heb ik maar buiten de review gehouden. Een aantal zijn toch toegevoegd, bij wijze van check van het resultaat is dat, denk ik, wel ok en het is bovendien soms ook moeilijk om uit te maken a.d.h.v. de changesets of het al dan niet over code gaat of testfiles.
P.s.: Ook kleine change in Econ (zie svn).

10:31 am

Tommy Hebb

[ICT-3120] [rvWV] PM: TAX: Front HoogteVerstelling MoventoKoppeling: release alle klanten:
[ICT-3120] [rvWV] PM: TAX: Front HoogteVerstelling MoventoKoppeling: release alle klanten:
Tuesday 20 Sep 2022
10:42 am

Tommy Hebb

10:38 am

Tommy Hebb

Ge ziet niets over 't hoofd :-D Ge hebt helemaal gelijk! Is een 1-op-1-property, dus mag automatisch geconverteerd worden. In tegenstelling tot DwarsverdelingAantal, want daar is de property-naam v...

Ge ziet niets over 't hoofd :-D Ge hebt helemaal gelijk! Is een 1-op-1-property, dus mag automatisch geconverteerd worden. In tegenstelling tot DwarsverdelingAantal, want daar is de property-naam verschillend (cq. AantalDwarsverdelingen in APPS).

10:14 am

Tommy Hebb

Aangepast naar "Dwarsverdelingen te picken"

Aangepast naar "Dwarsverdelingen te picken"

Monday 19 Sep 2022
12:02 pm

Wim Vermeulen

12:02 pm

Wim Vermeulen

Als opmerking in de kantlijn (dus je moet niks aanpassen) : hier moeten we eigenlijk gebruik maken van het berekende HalffabItem en daarvan de (bewerkte) lengte opvragen. Nu staat er duplicate logi...

Als opmerking in de kantlijn (dus je moet niks aanpassen) : hier moeten we eigenlijk gebruik maken van het berekende HalffabItem en daarvan de (bewerkte) lengte opvragen. Nu staat er duplicate logica.
Maar ik geloof best, zonder het na te kijken, dat in deze klasse nog code staat die vertrekt van de ingegevenKenmerken i.p.v. halffabItem.

11:44 am

Wim Vermeulen

DwarsverdelingKleur zit volgens mij "impliciet" in de ConverteerGegenereerdeProperties() hiervoor is geen "uitzondering nodig". Tijdens de uitbreidingsfase en review lijkt het me handig om te zien ...

DwarsverdelingKleur zit volgens mij "impliciet" in de ConverteerGegenereerdeProperties()
hiervoor is geen "uitzondering nodig".
Tijdens de uitbreidingsfase en review lijkt het me handig om te zien dat beide properties zijn toegevoegd. Maar eenmaal in productie (en ondersteund door automatische testen) kan dit het standaard pad volgen.
Of zie ik iets over t hoofd?

11:10 am

Wim Vermeulen

Tekst correct? Zonder kar --> geen vakken, right?

Tekst correct?
Zonder kar --> geen vakken, right?

Tuesday 13 Sep 2022
3:25 pm

Tommy Hebb

Staat al in productie en is voordien al uitgebreid end-to-end getest.

Staat al in productie en is voordien al uitgebreid end-to-end getest.

3:24 pm

Tommy Hebb

[ICT-2728] [rvWV] TAX configurator : uitbreiden met een dwarsverdeling:
[ICT-2728] [rvWV] TAX configurator : uitbreiden met een dwarsverdeling:
Thursday 28 Oct 2021
9:44 am

Peter Vercoutter

Tuesday 14 Sep 2021
4:50 pm

Wim Vermeulen

De vhUtils standaard-kenmerken kiezen is soms nogal tricky: *liefst zo standaard mogelijk (behalve bij speciallekes zoals LadeMetVeelExtras) *liefst zo ondubbelzinnig mogelijk : bvb. X en Y versc...

De vhUtils standaard-kenmerken kiezen is soms nogal tricky:

  • liefst zo standaard mogelijk (behalve bij speciallekes zoals LadeMetVeelExtras)
  • liefst zo ondubbelzinnig mogelijk : bvb. X en Y verschillend bij sifonlade --> check.
  • in de meeste gevallen zo werkelijkheidsgetrouw mogelijk


in deze method vind ik enige verwarring :

  • StandaardMat heeft LadeType=TAX, die heeft (nog) geen spoelbak.
  • Spoelbak voor LBX? of eerder voor TBX/MVX? --> wellicht zijn beide methods zinvol, en dan zou ik één method expliciet MatVoorLbxSpoelbakLade() noemen.
  • bij deze LBX moeten de uitsparingBreedtes X en Y dan best wel de juiste afmeting bevatten, want dat is ook een "vaste maat"


In eerdere gesprekken had ik al wel aangegeven dat een goede keuze van kenmerkwaarden, niet zo vanzelfsprekend is, en dat ik je hierbij zeker wel zou bijstaan. Bij deze, call me

4:42 pm

Wim Vermeulen

Friday 10 Sep 2021
5:22 pm

Wim Vermeulen

Ik heb nog wat uitleg over die "E" "Exceptions" in de Json-export, maar dat is wat veel om hier in de review-opmerking te zetten. Contacteer me maar eens. Eventueel zelfs interessant voor een meeti...

Ik heb nog wat uitleg over die "E" "Exceptions" in de Json-export, maar dat is wat veel om hier in de review-opmerking te zetten.
Contacteer me maar eens. Eventueel zelfs interessant voor een meeting met JBA en THB.

4:56 pm

Wim Vermeulen

copy-paste typo (2x) : #dim ... As MVX-kenmerken ??

copy-paste typo (2x) :
#dim ... As MVX-kenmerken ??

4:48 pm

Wim Vermeulen

copy-paste 2x zelfde

copy-paste 2x zelfde

4:47 pm

Wim Vermeulen

copy-paste 2x zelfde

copy-paste 2x zelfde

4:46 pm

Wim Vermeulen

Mijn eerste review-opmerking ging zijn : "In een AssertEquals-expressie zou geen $Select() mogen staan." Dat topic wil ik eventueel nog met jou bespreken. Door dit op te merken, viel m'n oog wel o...

Mijn eerste review-opmerking ging zijn : "In een AssertEquals-expressie zou geen $Select() mogen staan."
Dat topic wil ik eventueel nog met jou bespreken.

Door dit op te merken, viel m'n oog wel op een typo in bovenstaande lijn: RugBreedteX en in de $Select() staat BreedteM :-O

4:30 pm

Wim Vermeulen

Bij voorkeur Z1 en Z2 verschillend kiezen. idem voor X en Y

Bij voorkeur Z1 en Z2 verschillend kiezen.
idem voor X en Y

3:55 pm

Wim Vermeulen

Uitsparing Breedte.M moet je verplaatsen naar Dubbele uitsparing.

Uitsparing Breedte.M moet je verplaatsen naar Dubbele uitsparing.

3:55 pm

Wim Vermeulen

Deze IF legt een onnodige dependency, en heeft geen meerwaarde binnen de converter. Integendeel, bij uitbreiding voor LBX sifon of TBX sifon moet je hier iets toevoegen, terwijl de code binnen de I...

Deze IF legt een onnodige dependency, en heeft geen meerwaarde binnen de converter.
Integendeel, bij uitbreiding voor LBX sifon of TBX sifon moet je hier iets toevoegen, terwijl de code binnen de IF toch "universeel" is.

3:44 pm

Wim Vermeulen

Ook de MatKenmerken hebben property "NietMeeleveren", dus ik zou deze method wel laten oproepen, ook al is de impl dan enkel met de eerste en de laatste lijn : %New() en Quit. Oftewel rechtstreeks ...

Ook de MatKenmerken hebben property "NietMeeleveren", dus ik zou deze method wel laten oproepen, ook al is de impl dan enkel met de eerste en de laatste lijn : %New() en Quit.
Oftewel rechtstreeks in de method Converteer() , dan mag de commentaar hier volledig weg.