DataMMATBasisCreator.cls.xml

Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
De timings moeten niet mee tot in productie, lijkt me. Dus de TimerUtils-lijnen terug uit de code halen vooraleer te mergen naar Prod, tenzij dit expliciet in de opdracht (jira) staat.

De timings moeten niet mee tot in productie, lijkt me. Dus de TimerUtils-lijnen terug uit de code halen vooraleer te mergen naar Prod,
tenzij dit expliciet in de opdracht (jira) staat.

De timings moeten niet mee tot in productie, lijkt me. Dus de TimerUtils-lijnen terug uit de code halen vooraleer te mergen naar Prod, tenzij dit expliciet in de opdracht (jira) staat.

De timings moeten niet mee tot in productie, lijkt me. Dus de TimerUtils-lijnen terug uit de code halen vooraleer te mergen naar Prod,
tenzij dit expliciet in de opdracht (jira) staat.

[ICT-1751] PM: Maatwerk: Calculatie LBX traagheid: langtekst onherwerkte halffabboom buiten de loop
[ICT-1751] PM: Maatwerk: Calculatie LBX traagheid: langtekst onherwerkte halffabboom buiten de loop
[ICT-1751] PM: Maatwerk: Calculatie LBX traagheid: langtekst onherwerkte halffabboom buiten de loop

-UnitTesten en integratietesten aangepast + halffabboomonherwerkt wordt nu niet meer opgevangen

  1. … 12 more files in changeset.
[ICT-1751] PM: Maatwerk: Calculatie LBX traagheid: langtekst onherwerkte halffabboom buiten de loop

-Optimalisatie van LangtekstCreator, halfabBoomOnherwerkt wordt nu gebouwd in ProductCreator dus wordt nu maar 1 keer gebouwd i.p.v. per taal

-TODO: testen aanpassen

  1. … 9 more files in changeset.
Private methods onderaan plaatsen. (zie zelfde opm bij andere UT)

Private methods onderaan plaatsen.
(zie zelfde opm bij andere UT)

Ik heb enkele bedenkingen, die we gemakkelijker mondeling kunnen bespreken. Let's skype http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif

Ik heb enkele bedenkingen, die we gemakkelijker mondeling kunnen bespreken.
Let's skype

Private methods onderaan plaatsen. Alternatief is om er een OnBeforeOneTest() van te maken, maar dat zal hier niet lukken aangezien je data meegeeft (Kenm en variant) Dus toch private en onderaan...

Private methods onderaan plaatsen.

Alternatief is om er een OnBeforeOneTest() van te maken, maar dat zal hier niet lukken aangezien je data meegeeft (Kenm en variant)

Dus toch private en onderaan
En dan zou ik er een Function van maken i.p.v. "Do ..." van maken,
Bijvoorbeeld : GeefDataMMatCreator() met dezelfde args.

Hier zou ik de "StandaardLade" method oproepen en dan enkel de properties Kleur en Materiaal overschrijven. Da's 4 lijnen code-duplicatie minder. Anderszijds wou ik opmerken dat je de kleur-variat...

Hier zou ik de "StandaardLade" method oproepen en dan enkel de properties Kleur en Materiaal overschrijven.
Da's 4 lijnen code-duplicatie minder.

Anderszijds wou ik opmerken dat je de kleur-variaties beter in de UT zelf implementeert.
De Utils.MatKenmerken klasse is niet bedoeld om alle mogelijke variaties van kleuren, die we ooit ergens (in een UT) zouden nodig hebben, hier te gaan toevoegen.
1 kleur-variatie lijkt me nog wel te kunnen in deze klasse (is al schemerzone), maar 3 toch liever niet.

Ter illustratie : in deze klasse kan bijvoorbeeld wel : TaorIndelingMat , ExtraBredeMat, ... (die zijn nu uiteraard niet nodig)

ICT-1497: Matten: kleur wordt niet weergegeven bij type Orgalux en Taor
ICT-1497: Matten: kleur wordt niet weergegeven bij type Orgalux en Taor
[ICT-1497] Matten: kleur wordt niet weergegeven bij type Orgalux en Taor

Refactor duplicate methode, gebruikt van abstracte klasse

  1. … 2 more files in changeset.
[ICT-1497] Matten: kleur wordt niet weergegeven bij type Orgalux en Taor
  1. … 1 more file in changeset.
[ICT-128][BUS776] PM: Maatwerk: Ago Flock Fluweel matjes: LBX en TAX en mat-op-maat inbouwen

- DataMMATBasisCreator en DataMMATTAORCreator gebruiken de MATKleurUtils klasse.

  1. … 1 more file in changeset.
[ICT-128][BUS776] PM: Maatwerk: Ago Flock Fluweel matjes: LBX en TAX en mat-op-maat inbouwen

- ProductCreator : korttekst via method GeefKleurCodeVoorKorttekst() aangepast

[ICT-128][BUS776] PM: Maatwerk: Ago Flock Fluweel matjes: LBX en TAX en mat-op-maat inbouwen

- ProductCreator : korttekst via method GeefKleurCodeVoorKorttekst() omdat de 2e piece van de code bevat MatMateriaal.

  1. … 1 more file in changeset.
Wim, ik denk dat dit een misverstand is: jouw opmerking gaat over lijnen die ik in hun geheel weghaal hé.

Wim, ik denk dat dit een misverstand is: jouw opmerking gaat over lijnen die ik in hun geheel weghaal hé.

  • More
  • CR-308
  • finished reviewing
If (val=A) || (val=B) aanpassen naar ##class(TECH.StringUtils).Equals(val, A, B, ... )

If (val=A) || (val=B) aanpassen naar ##class(TECH.StringUtils).Equals(val, A, B, ... )

[VRB435] Matjes op maat moeten steeds de classificatie van hun generisch afgeleid product krijgen. Is...
[VRB435] Matjes op maat moeten steeds de classificatie van hun generisch afgeleid product krijgen. Is...
[VRB435] Matjes op maat moeten steeds de classificatie van hun generisch afgeleid product krijgen. Is op dit moment hetzelfde als wat ze van Caché krijgen, dus geen testresultaten geïmpacteerd. Manueel getest via Admin+Vhintra.
[UST1536] AGO-FIBER mat: LBX multicreate

- De gegeneerde producten waren veel te duur omdat er met millimeter ipv vierkante millimeter werd gerekend

- GBO heeft hier het hele proces doorgetest en goedbevonden

  1. … 2 more files in changeset.
[UW224] Geen cijfers na de komma bij matjes. Manueel getest.