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 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):

- 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.
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

- Enum TAORLadeKleur, TAORBodemKleur en TAORkantenbandKleur uitgebreid met drie nieuwe kleuren.

- Nieuwe WSDL-TestFiles => Uitbreiding kleuren-enums.

- PlaatAfwerkingBepaler => Toevoeging van kleuren aan juiste lijst => CA en SE = Fineer en GF = Melanine. UnitTest soortgelijk aangepast.

- UnitTests BepaalKorttekstTAOR => TestBodemKleurCode en TestBuitenKleurCode => Assert van nieuwe kleuren voorlopig uitgeschakeld. Wel al uitbreiding voorzien in comment => Eerst moet PM SelectieKenmerken aanpakken => Anders fout: Exact één validator: Er werden geen halffabitems gevonden.

  1. … 27 more files in changeset.
[ICT-1258] TAX: Nebraska eiken decor (E2) niet meer toelaten in configurator - Cache-gedeelte

- Falende testen mbt uitgefaseerde TAX kleur Nebraska eikenhout fixen

  1. … 2 more files in changeset.
schrijfwijze parameter 1 : hoofdletter "V" ? "Is hond dood?" , "Ja, is door voer" :-D misschien toch de casing terug aanpassen, hé.

schrijfwijze parameter 1 : hoofdletter "V" ?

"Is hond dood?" , "Ja, is door voer" :-D
misschien toch de casing terug aanpassen, hé.

If IsActiefTaxLinnen() ? Voor mij is de shortcut die je genomen hebt, goed genoeg http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif Ik merk het gewoon op.

If IsActiefTaxLinnen() ?
Voor mij is de shortcut die je genomen hebt, goed genoeg

Ik merk het gewoon op.

Hilarische typo !!! van de Joost zekers. LOL.

Hilarische typo !!!
van de Joost zekers.
LOL.

[BUS706] TAX: afwerkingen (kleuren) "Linnen" en "Brut eiken" toevoegen
[BUS706] TAX: afwerkingen (kleuren) "Linnen" en "Brut eiken" toevoegen
[BUS706] TAX: afwerkingen (kleuren) "Linnen" en "Brut eiken" toevoegen

- test controleert ook op nieuwe kleuren

  1. … 1 more file in changeset.
[BUS706] TAX: afwerkingen (kleuren) "Linnen" en "Brut eiken" toevoegen
  1. … 129 more files in changeset.
[BUS670] TAX: Berk: beschikbaar stellen voor alle klanten (verschillende webshops)

- fix testen

- definitieve vrijgave

  1. … 3 more files in changeset.
  • More
  • CR-596
  • finished reviewing
[UST3684] TAX: Berk
[UST3684] TAX: Berk