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-2807 [rv WV] TAX - Tijdelijke meerkost voor de hoogteverstelling reeds in schaduwprijs
ICT-2807 [rv WV] TAX - Tijdelijke meerkost voor de hoogteverstelling reeds in schaduwprijs
[ICT-2807] [rv WV] TAX - Tijdelijke meerkost voor de hoogteverstelling reeds in schaduwprijs:

- Kleine refactor na review

    • -1
    • +1
    ./IsHoogteVerstellingAanwezigEvaluator.cls.xml
  1. … 1 more file in changeset.
Oeps! Moest dus zijn: ICT-2807 [rv WV] TAX - Tijdelijke meerkost voor de hoogteverstelling reeds in schaduwprijs

Oeps! Moest dus zijn: ICT-2807 [rv WV] TAX - Tijdelijke meerkost voor de hoogteverstelling reeds in schaduwprijs

De subject van deze crucible review verwijst vreemd genoeg naar "TAX nieuwe kleur W7" :-P

De subject van deze crucible review verwijst vreemd genoeg naar "TAX nieuwe kleur W7" :-P

Oplossen in de kostItemBuilder door conditie Tijd > 0

Oplossen in de kostItemBuilder door conditie Tijd > 0

Zoals in je script : If Tijd > 0 then VoegToeVariabele Want nu staat er bij de integratietesten van Lossecomponenten telkens een onnodig "tijdkost = 0 item"

Zoals in je script :
If Tijd > 0 then VoegToeVariabele

Want nu staat er bij de integratietesten van Lossecomponenten telkens een onnodig "tijdkost = 0 item"

Quick-win : performantie verhoogt door IngegevenKenmerken.KoppelingType = ##class(DOM.PM.enu.TAORKoppelingType).Movento() eerst te zetten in de conditie, en pas in tweede instantie te testen of Ro...

Quick-win : performantie verhoogt door

IngegevenKenmerken.KoppelingType = ##class(DOM.PM.enu.TAORKoppelingType).Movento()

eerst te zetten in de conditie, en pas in tweede instantie te testen of RolAanwezig is.

Mag uiteraard wel op 1 lijn blijven staat.

[ICT-2807] [rv WV] TAX - Tijdelijke meerkost voor de hoogteverstelling reeds in schaduwprijs:

- Fix => KoppelingType ipv Koppeling

    • -1
    • +1
    ./IsHoogteVerstellingAanwezigEvaluator.cls.xml
[ICT-2807] [rv WV] TAX - Tijdelijke meerkost voor de hoogteverstelling reeds in schaduwprijs:

- Uitbreiding enum TAORKostenplaats met MeerkostHoogteVerstelling

- Aanpassing in KostenPlaatsMaker om kost in de schaduw te kunnen steken

- Toevoeging IsHoogteVerstellingAanwezigEvaluator

- Toevoeging MeerkostHoogteVerstelling-KostBuilder

- Toevoeging nieuwe KostBuilder aan SpecKostv004

- Deploy voor nieuwe KostenPlaats MeerkostHoogteVerstelling

    • -0
    • +43
    ./IsHoogteVerstellingAanwezigEvaluator.cls.xml
  1. … 8 more files in changeset.
[ICT-2588] [rvWV] TAX - nieuwe kleur W7 - Walnut medium Brown:

- Nodige aanpassingen in BOL-builders...

- Builder SpecialWorkShop toegevoegd => LINEFEEDING 20 seconden

- Evaluator IsWalnutMediumBrownBinnenladeEvaluator toegevoegd

    • -0
    • +37
    ./IsWalnutMediumBrownBinnenladeEvaluator.cls.xml
  1. … 2 more files in changeset.
Is bij de laatste wijzigingen terug F geworden http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif

Is bij de laatste wijzigingen terug F geworden

Blijft deze code "P" behouden? Of toch de "F" van oorspronkelijke "FlatpackV1" ? Best toch effe checken bij JohanB.

Blijft deze code "P" behouden? Of toch de "F" van oorspronkelijke "FlatpackV1" ?
Best toch effe checken bij JohanB.

[ICT742] [Kanban UST4277] TAXFP1: Flatpack Probox => Flatpack V1:
[ICT742] [Kanban UST4277] TAXFP1: Flatpack Probox => Flatpack V1:
[ICT218] [Kanban ICT490] - TAX: FPV1: Opkuis code flatpack V1:
[ICT218] [Kanban ICT490] - TAX: FPV1: Opkuis code flatpack V1:
[ICT742] [Kanban UST4277] TAXFP1: Flatpack Probox => Flatpack V1:

- Move class vanwege naamswijziging => FlatpackV1

    • -22
    • +0
    ./IsFlatpackProboxVerpakking.cls.xml
  1. … 2 more files in changeset.
[ICT742] [Kanban UST4277] TAXFP1: Flatpack Probox => Flatpack V1:

- Verheuis code FlatpackProbox naar FlatpackV1

- Nog wat opkuis hier en daar

  1. … 87 more files in changeset.
[ICT218] [Kanban ICT490] - TAX: FPV1: Opkuis code flatpack V1:

- Grote opkuis oude ongebruikte FlatpackV1 => Gezocht op FPV1

  1. … 9 more files in changeset.