Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Deze voorwaarde komt vele malen voor in de Builders (idem voor ..IsNietUitvoeringTablet() ) Kunnen we een propere en efficiente oplossing vinden waarbij die method niet telkens opnieuw in de builde...

Deze voorwaarde komt vele malen voor in de Builders (idem voor ..IsNietUitvoeringTablet() )
Kunnen we een propere en efficiente oplossing vinden waarbij die method niet telkens opnieuw in de builder moet gedefinieerd worden. Laten we dat samen eens bekijken.

Deze method komt vele malen voor in de Builders (idem voor =Servodrive) Kunnen we een propere en efficiente oplossing vinden waarbij die method niet telkens opnieuw in de builder moet gedefinieerd ...

Deze method komt vele malen voor in de Builders (idem voor =Servodrive)
Kunnen we een propere en efficiente oplossing vinden waarbij die method niet telkens opnieuw in de builder moet gedefinieerd worden. Laten we dat samen eens bekijken.

Het "tussen-niveau" ...HalffabItemBuilderBasis is bij SST eigenlijk niet nodig. Dit was wel zo bij de "Matten" omdat je MatX,MatY, ... hebt. Hier, bij SST, heb je maar 1 klasse, t.t.z. Builder en B...

Het "tussen-niveau" ...HalffabItemBuilderBasis is bij SST eigenlijk niet nodig. Dit was wel zo bij de "Matten" omdat je MatX,MatY, ... hebt.
Hier, bij SST, heb je maar 1 klasse, t.t.z. Builder en BuilderBasis vallen samen.
Maar wat mij betreft hoeft dit niet weggewerkt te worden in caché. Misschien wel bij het porten naar .NET

FYI: het was uiteindelijk de dubbele definitie van de method GeefGekoppeldIKType() die voor enige verwarring zorgde. Vandaar de review opm.

[ICT-1633] Implementatie Halffabbuilders SpaceStep
[ICT-1633] Implementatie Halffabbuilders SpaceStep
ICT-918: Uitfaseren LBX Tipon : aanpassen Automatische Testen
ICT-918: Uitfaseren LBX Tipon : aanpassen Automatische Testen
ICT-849: Testen committen na wijzigingen LBX product - zie johan B.
ICT-849: Testen committen na wijzigingen LBX product - zie johan B.
ok, thx.

ok, thx.

Merk op dat de som van de inkorten Breedte ("ink B") moet gelijk zijn aan de nieuwe breedte 520 http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/warning.gif Helaas...

Merk op dat de som van de inkorten Breedte ("ink B") moet gelijk zijn aan de nieuwe breedte 520
Helaas ...
Wat is er fout? --> Blijkbaar foutieve UitsparingData in de CSifonLadeZijWandTypeFree() sinds 12/01/2015 (PBO)
Jij moet hiervoor nu niks doen. Ik zal binnenkort wel de nodige data aanpassen.
Dit is immers ook nog allemaal Sifon Type2. Ik maak een kaartje om daar LBX sifon Type3 lades van te maken

Method is niet (meer) nodig, denk.

Method is niet (meer) nodig, denk.

Public methods bovenaan zetten (geldt ook overrides http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif )

Public methods bovenaan zetten (geldt ook overrides )

opmerking i.v.m. "y" in syfon/sifon reeds : zie boven. Is geen drama, maar we moeten wel vermijden dat anderen de foute schrijfwijze zouden overnemen. Staat deze al in productie?

opmerking i.v.m. "y" in syfon/sifon reeds : zie boven.
Is geen drama, maar we moeten wel vermijden dat anderen de foute schrijfwijze zouden overnemen.
Staat deze al in productie?

VH spelling conventie : geen tussen-s in onze benamingen.

VH spelling conventie : geen tussen-s in onze benamingen.

VH spelling conventie : geen tussen-s in onze benamingen. Dus UitsparingDiepteSifonType3

VH spelling conventie : geen tussen-s in onze benamingen.
Dus UitsparingDiepteSifonType3

Probeer zo veel mogelijk om Sifonlade met "i", niet met "y" te schrijven. (JB durft ook wel eens "y" te gebruiken, wij @ICT liever niet) Indien deze echt te lastig om aan te passen, dan mag je 'm z...

Probeer zo veel mogelijk om Sifonlade met "i", niet met "y" te schrijven. (JB durft ook wel eens "y" te gebruiken, wij @ICT liever niet)
Indien deze echt te lastig om aan te passen, dan mag je 'm zo laten.

[UST4197] HX: Productiedetailblad: matten bij sifonlade LBX: één mat met aanduiding X, Q, Y
[UST4197] HX: Productiedetailblad: matten bij sifonlade LBX: één mat met aanduiding X, Q, Y
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 )

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é

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!

[UST4063] LBX: MeenemerSet voor binnenlade: BuilderHF:
[UST4063] LBX: MeenemerSet voor binnenlade: BuilderHF: