HFKantenBandFrontv000.cls.xml

Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
FYI: Ik vind het zeker oké dat je hier de ##super hebt weggelaten. (de gegenereerde code die daar stond was eigenlijk zelfs niet correct : LadeHoogte is enum en dus niet in MM uitgedrukt http://sub...

FYI: Ik vind het zeker oké dat je hier de ##super hebt weggelaten. (de gegenereerde code die daar stond was eigenlijk zelfs niet correct : LadeHoogte is enum en dus niet in MM uitgedrukt )
De impl in deze afgeleide klasse is wel correct.

De implementatie voor "Lengte" hieronder zou ik wel in een private method steken, analoog aan GeefFrontPlaatHoogte()
Zo blijft deze method simpel leesbaar en overzichtelijk. En dan is de scope ook duidelijk, nl. dat Kleur W7 enkel impact heeft op de kenmerk "Lengte, en niet op de "Breedte" van het Front.

copy-paste foutje : "... kan niet gemaild worden." (ook al is de conditie altijd false :-P )

copy-paste foutje :
"... kan niet gemaild worden."
(ook al is de conditie altijd false :-P )

[ICT-2588] [rvWV] TAX - nieuwe kleur W7 - Walnut medium Brown:
[ICT-2588] [rvWV] TAX - nieuwe kleur W7 - Walnut medium Brown:
[ICT-2588] [rvWV] TAX - nieuwe kleur W7 - Walnut medium Brown:

- Correctie => Indien W7 => Kantenband voor aan de zijkanten van het Front meetellen bij dimensie Lengte

[ICT-2588] [rvWV] TAX - nieuwe kleur W7 - Walnut medium Brown:

- Nodige aanpassingen in BOM-builders...

- Indien W7 => Ook KantenBand aan de zijkanten van het Front = Hoogte * 2 => moet gedefinieerd staan voor VHintra als Breedte en niet als Hoogte

[ICT-2588] [rvWV] TAX - nieuwe kleur W7 - Walnut medium Brown:

- Nodige aanpassingen in BOM-builders...

- Geen FrontEindkap indien W7

- Indien W7 => FrontPlaat-lengte = InterneBreedte - 10

- Indien W7 => Ook KantenBand aan de zijkanten van het Front = Hoogte * 2

  1. … 2 more files in changeset.
[ICT-2203] [rvTVE] PM - TAX front voor LBX - oplossing voor "OudZwart" (Z8/Z9):
[ICT-2203] [rvTVE] PM - TAX front voor LBX - oplossing voor "OudZwart" (Z8/Z9):
[ICT-2203] [rvTVE] PM - TAX front voor LBX - oplossing voor "OudZwart" (Z8/Z9):

- KantenbandKleur hoeft niet meer geconvereerd te worden voor de korttekst e.d., enkel nog voor productie-aansturing (maar dat is in andere klasse)

Zoals besproken blijkbaar ongewild in deze review terechtgekomen http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif

Zoals besproken blijkbaar ongewild in deze review terechtgekomen

is deze wel nodig?

is deze wel nodig?

[ICT-2203] [rvTVE] PM - TAX front voor LBX - oplossing voor "OudZwart" (Z8/Z9):

- KantenBandFront-builder misbruikt eigenlijk de LadeKleur. KantenbandKleur wordt niet uitgebreid met OudZwart => Indien LadeKleur = OudeZwart => KantenbandKleur = Zwart

De ladekleur in deze testcase is fineer. De kantenband wordt dus nu ook fineer, de ingevulde data is vergelijkbaar met andere fineer-data

De ladekleur in deze testcase is fineer. De kantenband wordt dus nu ook fineer, de ingevulde data is vergelijkbaar met andere fineer-data

[BUS705] TAX: Fronten: kantenband volgt frontkleur: reviewopmerkingen
  1. … 1 more file in changeset.
Methodnaam best aanpassen naar ... iets zoals : Test: Positie 17-18: Front - Kantenband volgt ladekleur ipv kantenbandkleur

Methodnaam best aanpassen naar ... iets zoals :

Test: Positie 17-18: Front - Kantenband volgt ladekleur ipv kantenbandkleur 
Pass 3 en 4 zijn er nu bijgekomen (lees: ingevuld). Zou dit door het verschil tussen melamine- en fineer-kantenband zijn?

Pass 3 en 4 zijn er nu bijgekomen (lees: ingevuld). Zou dit door het verschil tussen melamine- en fineer-kantenband zijn?

Exit na Pass4. Dit klopt met vorige opmerking.

Exit na Pass4. Dit klopt met vorige opmerking.

Printer bij Pass1 i.p.v. bij Pass2 --> te checken.

Printer bij Pass1 i.p.v. bij Pass2 --> te checken.

Tja, geen idee of de wijzigingen voor de Kantenband-data perfect zijn. Dit lijkt me het normale gevolg te zijn van de aanpassing voor het front, ik zou durven denken dat de software hier universeel...

Tja, geen idee of de wijzigingen voor de Kantenband-data perfect zijn.
Dit lijkt me het normale gevolg te zijn van de aanpassing voor het front, ik zou durven denken dat de software hier universeel geschreven is (voor rug, zijkant, front).

Hoor eens bij Pieter, of rechtstreeks bij de SQL-specialist van de kantenband machine

Voor de method GenereerKenmerken() zou je ook de ##super(...) kunnen gebruiken. Nu heb je de method keihard overschreven, en zal dus nooit meer de aanpassingen van de parent-class volgen (tenzij du...

Voor de method GenereerKenmerken() zou je ook de ##super(...) kunnen gebruiken.
Nu heb je de method keihard overschreven, en zal dus nooit meer de aanpassingen van de parent-class volgen (tenzij dubbel onderhoud)

Wat denk je van deze:

	#dim SelectieKenmerken As %ArrayOfDataTypes = ##super(IngegevenKenmerken)
	If IngegevenKenmerken.IsBinnenlade {
		Do SelectieKenmerken.SetAt(IngegevenKenmerken.LadeKleur, "LadeKleur")
	}
	Quit SelectieKenmerken
[BUS705] TAX: Fronten: kantenband volgt frontkleur (met spatie)
[BUS705] TAX: Fronten: kantenband volgt frontkleur (met spatie)
[BUS705] TAX: Fronten: kantenband volgt frontkleur
  1. … 5 more files in changeset.
[UST1249] TAX LC: rug/bodem/zijkant/front: juiste items in boom

- Kantenband opgesplitst

- BouwsteenCreatie aanpassing

    • -0
    • +29
    ./HFKantenBandFrontv000.cls.xml
  1. … 41 more files in changeset.