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:

- Uitbreiding voor WalnutMediumBrown => 1-letterige code = P

  1. … 1 more file in changeset.
[ICT-2588] [rvWV] TAX - nieuwe kleur W7 - Walnut medium Brown:

- Uitbreiding kleurenums van TAOR.

- Uitzondering voorzien voor nieuwe kleur in KenmerkDomeinPostProcessor => VHintra-configurator heeft hier geen zaken meer mee => incl. WSDL-aanpassing

- Uitbreiding PlaatAfwerkingBepaler => nieuwe melamine-kleur

- Voorlopige uitsluiting nieuwe kleur in enkele testen (te checken eens selectiekenmerken goed staan)

  1. … 27 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):
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):

- VerwerkOudZwart => KleurCode = Q

- Uitschakelen test uit vorige commit ongedaan gemaakt => OudZwart wordt nu ook getest

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

- OudZwart = Z9 = een melaminekleur

- OudZwart voorlopig uitgesloten in testen voor productieaansturing en kleurcode => bijhorende klassen mogen in dit stadium nog niet aangepast worden

  1. … 3 more files in changeset.
[ICT-1682] TAX - 3 nieuwe kleuren toevoegen - fase 3 en 4:

- Voorziening KleurCode voor barcode voor kleuren:

- SepiaEik => G

- ChalkAsh => H

- Lakdraagfolie => R

- Eerder ingevoegde uitsluiting van de nieuwe kleuren in de UnitTest weer verwijderd

  1. … 1 more file in changeset.
Inclusief: ICT-1572 en ICT-1578 (voor de andere 2 nieuwe kleuren) Sommige testen zijn voorlopig uitgeschakeld voor de nieuwe kleuren, omdat alvast de SelectieKenmerken beschikbaar moeten zijn (voor...

Inclusief: ICT-1572 en ICT-1578 (voor de andere 2 nieuwe kleuren)
Sommige testen zijn voorlopig uitgeschakeld voor de nieuwe kleuren, omdat alvast de SelectieKenmerken beschikbaar moeten zijn (voor PM), zonder dat ProductieAansturing e.d. al aangepakt zijn. Scope van deze story is louter uitbreiding enum en beschikbaar stellen van de nieuwe kleuren zodat PM de SelectieKenmerken kan aanpakken.

ICT-1573: [rvTVE] TAX - Nieuwe kleur lakdraagfolie GF - fase 1 - onderdeel 1/2/3
ICT-1573: [rvTVE] TAX - Nieuwe kleur lakdraagfolie GF - fase 1 - onderdeel 1/2/3
[ICT-1572] TAX - nieuw kleur sepia eikenfineer SE - fase 1 - onderdeel 1/2/3

[ICT-1578] TAX - Nieuwe kleur Chalk Ash CA - fase 1 - onderdeel 1/2/3

[ICT-1573] TAX - Nieuwe kleur lakdraagfolie GF - fase 1 - onderdeel 1/2/3

- Assert van nieuwe kleuren voorlopig uitgeschakeld => Anders fout vanwege: APPS.Halux.PPS.Activiteit.impl.TAOR.common.KleurConverter

[BUS565] TAX extra kleur: kleurconverter verwerkt kleur dakar - UT TestAlleGedefinieerd()

(i.s.m. MartijnV)

    • -0
    • +37
    ./Converteer/Test.cls.xml