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-1633] Implementatie Halffabbuilders SpaceStep

- Implementatie van Halffabbuilders en rollen voor enkel hargecodeerde proudcten van SpaceStep

- Implementatie SST builders toevoegen aan LBX model

  1. … 39 more files in changeset.
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:
[UST4063] LBX: MeenemerSet voor binnenlade: BuilderHF:

- Conditie VolledigeSet bij toevoegen rol MeenemerSet op LegraboxPureModel

- BuilderMoetWerken => MeenemerSet moet VolledigeSet zijn

- VoegToeMeenemerSet is uit comment gehaald op BuilderHFv001

- Deploy => MaakKenmerkenData en VulInProductKenmerken

- Testje => TestM/ZMetVoorstukEnMeenemerSet

  1. … 4 more files in changeset.
[UST4063] LBX: MeenemerSet voor binnenlade: BuilderHF:

- MeenemerSet => Voorlopig in comment op BuilderHFv001

  1. … 1 more file in changeset.
Voor alle zekerheid de 'storage-name' behouden, je weet nl. niet of dto's als property mee bewaard worden

Voor alle zekerheid de 'storage-name' behouden, je weet nl. niet of dto's als property mee bewaard worden

[UST3783] LBX-TB-001 : Interne wizard - niet meeleveren "synchro koppelstuk" mag weg.
[UST3783] LBX-TB-001 : Interne wizard - niet meeleveren "synchro koppelstuk" mag weg.
[UST3783] LBX-TB-001 : Interne wizard - niet meeleveren "synchro koppelstuk" mag weg.

- niet meeleveren, etc...

- (lots of troubles gehad met opsplitsen TAX en LBX... testfiles TAX waren geimpacteerd, bleek uiteindelijk over een klasse te gaan die na opnieuw compileren wel de testen liet slagen... pfff.

  1. … 81 more files in changeset.
[UST2235] LBX VR Afdekkap: rol, sel.kenm., halffabbuilder, projectsetting (was deel van UST2105)

- Dus ook voor versie 000 -> test Geefhalffabitemsboom wertkt met product 1022966 op oudere versie

  1. … 1 more file in changeset.
  • More
  • CR-110
  • finished reviewing
Builder opnieuw genereren, het selectiekenmerk voor LadeKleur ontbreekt nog.

Builder opnieuw genereren, het selectiekenmerk voor LadeKleur ontbreekt nog.

[UST2235] LBX VR Afdekkap: rol, sel.kenm., halffabbuilder, projectsetting (was deel van UST2105)
[UST2235] LBX VR Afdekkap: rol, sel.kenm., halffabbuilder, projectsetting (was deel van UST2105)
[UST2235] LBX VR Afdekkap: rol, sel.kenm., halffabbuilder, projectsetting (was deel van UST2105)

- Rol + Halffab + BuilderMoetWerken voor C, K en rughoogte M, ladezijkant Pure.

  1. … 7 more files in changeset.
[UST1499] LBX TOB SETS: opkuis

- Verwijzingen naar projectsetting verwijderd

  1. … 23 more files in changeset.
[UST1486] LBX TOB SETS: Halffab Builder vervangen

- Halffabrikaten van de TOB meenemer en eenheid vervangen door een Set

  1. … 4 more files in changeset.