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-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-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.
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
  1. … 129 more files in changeset.
  • More
  • CR-596
  • finished reviewing
[UST3684] TAX: Berk
[UST3684] TAX: Berk
[UST3684] TAX: Berk

- toevoegen en afblokken nieuwe kleur

  1. … 26 more files in changeset.
  • More
  • CR-272
  • finished reviewing
  • More
  • CR-269
  • finished reviewing
En wat met de kantenbandkleur?

En wat met de kantenbandkleur?