Activity CacheAdminA_trunk2010/DOM/PM/Maatwerk/Calc/HF/impl/LBX/Builder/Generated

4:11 pm

Tommy Hebb

More
Re: CR-1580
replied to Wim Vermeulen

My thinking exactly http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif

My thinking exactly

3:47 pm

Deze genereert de method GeefOmschrijvingVoorDocs() voor Halffab. Normaal gezien bevat deze een opsomming van de selectiekenmerken. In dit uitzonderlijke geval zou hoef je de LadeKleur NIET toe te ...

Deze genereert de method GeefOmschrijvingVoorDocs() voor Halffab.
Normaal gezien bevat deze een opsomming van de selectiekenmerken.
In dit uitzonderlijke geval zou hoef je de LadeKleur NIET toe te voegen, aangezien het enkel gebruikt wordt als work-around om een "Verpakkingsprobleem" te omzeilen.
Dus lijkt me geen echte meerwaarde om de kleur te vermelden.

1:49 pm

Tommy Hebb

More
CR-1580
started review

[BUS723] PM: Maatwerkcalculatie: LBX: Fronthouder bij K-hoogte binnenlade: selectie afh. van kleur:

9:11 am

Tommy Hebb

More
55199
changed 3 files
root: CR-1580

[BUS723] PM: Maatwerkcalculatie: LBX: Fronthouder bij K-hoogte binnenlade: selectie afh. van kleur:

- Toevoeging kenmerk LadeKleur bij FrontHouder Binnenlade

- -0
- +1
./HFFrontHouderv000.cls.xml
- History
- Source
- Diff
./HFFrontHouderv000.cls.xml
… 2 more files in changeset.

12:41 pm

Tommy Hebb

More
CR-1398
Closed

12:40 pm

Tommy Hebb

More
Re: CR-1398
replied to Wim Vermeulen

Andere schrijfwijze voor een property enerzijds en een method anderzijds binnen één en dezelfde klasse is ook een rommeltje hé, dus ik denk dat we 't beter laten gelijk het is en er de volgende kee...

Andere schrijfwijze voor een property enerzijds en een method anderzijds binnen één en dezelfde klasse is ook een rommeltje hé, dus ik denk dat we 't beter laten gelijk het is en er de volgende keer van in 't begin rekening mee houden. (En gij had mij gezegd dat het met een grote W was toen ik de code aan 't schrijven was ;-p )

12:33 pm

Wim Vermeulen

More
CR-1398
finished reviewing

12:32 pm

Wim Vermeulen

More
Re: CR-1398
commented on review

Zwart + Inox http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif is oké http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/s...

Zwart + Inox
is oké

12:32 pm

Wim Vermeulen

More
Re: CR-1398
commented on review

Doorheen het hele verhaal : ik was vergeten te zeggen dat de schrijfwijze van voorwandtype foutief is. Het moet namelijk met kleine "W" zijn. want het is niet de Wand van de Voor (en daar het Type ...

Doorheen het hele verhaal :
ik was vergeten te zeggen dat de schrijfwijze van voorwandtype foutief is. Het moet namelijk met kleine "W" zijn.
want het is niet de Wand van de Voor (en daar het Type van) , neeje, het is gaat om het Type van de Voorwand (naar analogie met Voorstuk)
Maar m'n beklag is op een voorganger, die dt verzonnen heeft :-/

Anyway, kort gezegd : de property zelf en rechtstreek eraan gekoppelde of gegenereerde code hoef je niet te wijzigen (wegens complex en zelfs risico-vol!) maar methodnamen als deze hier en lokale variabelen geef ik de voorkeur op correcte schrijfwijze , namelijk :
VoorwandType

Overleg best effe met mij, alvorens iets aan te passen!

4:46 pm