|
|
|
 |
- last updated a few seconds ago
Monday 08 Aug 2022
Hier staat nog een regel commentaar dat waarschijnlijk weg mag
Hier staat nog een regel commentaar dat waarschijnlijk weg mag
Geen idee of dit van u is, maar hier staat een schrijffout http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/wink.gif
Geen idee of dit van u is, maar hier staat een schrijffout 
Friday 05 Aug 2022
[ ICT-2896] MyLegrabox: Legrabox design zijkanten in configurator [ ICT-2896] MyLegrabox: Legrabox design zijkanten in configurator
Monday 18 Jul 2022
Tuesday 12 Jul 2022
Wednesday 06 Jul 2022
[ ICT-3008] HTTP POST in Cache mist nog enkele verwerkers voor MVX [ ICT-3008] HTTP POST in Cache mist nog enkele verwerkers voor MVX
Wednesday 26 Jan 2022
FYI : Bij een volgende uitbreiding van deze If-else, zullen we best de impl afzonderen in aparte method (of klasse). Bvb. wanneer Revego (of eenvoudig maatwerk) erbij komt, kan de eerste if zijn : ...
FYI :
Bij een volgende uitbreiding van deze If-else, zullen we best de impl afzonderen in aparte method (of klasse).
Bvb. wanneer Revego (of eenvoudig maatwerk) erbij komt, kan de eerste if zijn :
if Not(GeefFamilie() = "Lade") then ... else ...
Of als je dat nu reeds wil doen, feel free
FYI: We hadden samen bekeken, in welke klasse we Mat-materiaal best zouden toevoegen. Het is nu pas, tijdens review, dat me opvalt dat deze klasse (met "...Omschrijving" in de naam) NIET taalafhank...
FYI:
We hadden samen bekeken, in welke klasse we Mat-materiaal best zouden toevoegen.
Het is nu pas, tijdens review, dat me opvalt dat deze klasse (met "...Omschrijving" in de naam) NIET taalafhankelijk is, maar dat is dus eerder toevallig, omdat LBX, TAX en TBX in alle talen hetzelfde is.
Het had zeker niet verkeerd geweest om deze klasse verder uit te breiden met vertaling en MatMateriaal. My bad!
Voorlopig zullen we het hierbij laten. De huidige oplossing is zeker ook oké.
[ ICT-2707] HTTP post voor matten uitbreiden [ ICT-2707] HTTP post voor matten uitbreiden
Thursday 28 Oct 2021
Tuesday 14 Sep 2021
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
Friday 10 Sep 2021
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.
copy-paste typo (2x) : #dim ... As MVX-kenmerken ??
copy-paste typo (2x) :
#dim ... As MVX-kenmerken ??
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
Bij voorkeur Z1 en Z2 verschillend kiezen. idem voor X en Y
Bij voorkeur Z1 en Z2 verschillend kiezen.
idem voor X en Y
Uitsparing Breedte.M moet je verplaatsen naar Dubbele uitsparing.
Uitsparing Breedte.M moet je verplaatsen naar Dubbele uitsparing.
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.
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.
VH code conventions : VulAanMatKenmerken()
VH code conventions : VulAanMatKenmerken()
Als het object MatKenmerken één en dezelfde oref blijft (dus geen %Clone ofzo) dan is het niet nodig om de SetAt() te doen, want de pointer in de GekoppeldeKenmerken verwjist nog steeds naar hetzel...
Als het object MatKenmerken één en dezelfde oref blijft (dus geen %Clone ofzo) dan is het niet nodig om de SetAt() te doen, want de pointer in de GekoppeldeKenmerken verwjist nog steeds naar hetzelfde object in memory.
Een kleine bedenking: ik ben niet helemaal zeker of de method wel thuishoort in deze klasse, omdat die enkel geschikt is voor MatKenmerken. Voorlopig is dit oké, en functioneel zeker geen probleem...
Een kleine bedenking:
ik ben niet helemaal zeker of de method wel thuishoort in deze klasse, omdat die enkel geschikt is voor MatKenmerken.
Voorlopig is dit oké, en functioneel zeker geen probleem.
Ik wil niet verder uitweiden nu, over deze bedenking, dus enkel als review-opmerking, en om te vermijden dat we vanalles in deze klasse gaan steken dat niet voor alle IngegevenKenmerken van toepassing is.
Er is geen IF nodig, want : if "true" or "false" --> altijd dus :-P
Er is geen IF nodig, want :
if "true" or "false" --> altijd dus
:-P
Friday 03 Sep 2021
toegevoegde parameter "IsBerekendViaDotNet" ook doorgeven via ##super()
toegevoegde parameter "IsBerekendViaDotNet" ook doorgeven via ##super()
|
