CacheAdminA_trunk2010/DOM/PM/Maatwerk/Calc/Kost/impl/TAOR

SpecKostv004.cls.xml

Checkout Tools

Line History

Line Count Graph
Line Count Graph

Stats

Commits this week: 0
Total Committers: 5
Last Commit: 29 Mar 22
Commits this week: 0
Total Lines of Code (LoC): 206
Net change in LoC this week: 0

Commit Activity

Commits By Day In Week
52 week commits volume
Commits By Day In Week
Commits by day
Commits By Day In Week
Commits by hour
Commits By Hour In Day
Commit calendar
 

Most active committers (90 days)

loading
loading
  • last updated a few seconds ago
All Commits Reviews
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Monday 30 May 2022
4:41 pm

Tommy Hebb

Tuesday 24 May 2022
1:09 pm

Wim Vermeulen

1:09 pm

Wim Vermeulen

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.

10:08 am

Wim Vermeulen

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 )

Wednesday 20 Apr 2022
3:37 pm

Tommy Hebb

[ICT-2588] [rvWV] TAX - nieuwe kleur W7 - Walnut medium Brown:
[ICT-2588] [rvWV] TAX - nieuwe kleur W7 - Walnut medium Brown:
3:35 pm

Tommy Hebb

3:35 pm

Tommy Hebb

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
Thursday 31 Mar 2022
9:38 am

Tommy Hebb

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

Wednesday 30 Mar 2022
5:49 pm

Wim Vermeulen

5:49 pm

Wim Vermeulen

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

5:46 pm

Wim Vermeulen

Oplossen in de kostItemBuilder door conditie Tijd > 0

Oplossen in de kostItemBuilder door conditie Tijd > 0

5:45 pm

Wim Vermeulen

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"

5:20 pm

Wim Vermeulen

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.

Tuesday 29 Mar 2022
2:25 pm

Tommy Hebb

[ICT-2588] [rvWV] TAX - nieuwe kleur W7 - Walnut medium Brown:

- TERUG TOEGEVOEGD NU (Voorlopig toevoeging SWS-kostbuilder er terug uitgehaald => zit even in de weg van MeerkostHoogteVerstelling in de schaduw (ICT-2807) die sneller in productie moet)

Options
    • -0
    • +1
    ./SpecKostv004.cls.xml
1:54 pm

Tommy Hebb

[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

Options
    • -0
    • +1
    ./SpecKostv004.cls.xml
  2. … 8 more files in changeset.
1:49 pm

Tommy Hebb

[ICT-2588] [rvWV] TAX - nieuwe kleur W7 - Walnut medium Brown:

- Voorlopig toevoeging SWS-kostbuilder er terug uitgehaald => zit even in de weg van MeerkostHoogteVerstelling in de schaduw (ICT-2807) die sneller in productie moet

Options
    • -1
    • +0
    ./SpecKostv004.cls.xml
Tuesday 22 Mar 2022
4:29 pm

Tommy Hebb

[ICT-2588] [rvWV] TAX - nieuwe kleur W7 - Walnut medium Brown:

- Nodige aanpassingen in BOL-builders...

- Builder SpecialWorkShop toegevoegd => LINEFEEDING 20 seconden

- Evaluator IsWalnutMediumBrownBinnenladeEvaluator toegevoegd

Options
    • -0
    • +1
    ./SpecKostv004.cls.xml
  2. … 2 more files in changeset.
Thursday 28 Oct 2021
9:44 am

Peter Vercoutter

Tuesday 05 Oct 2021
12:49 pm

Tommy Hebb

12:21 pm

Tom Vermeulen

11:09 am

Tommy Hebb

[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):
Thursday 23 Sep 2021
4:17 pm

Tommy Hebb

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

4:06 pm

Tom Vermeulen

is deze wel nodig?

is deze wel nodig?

Tuesday 21 Sep 2021
2:51 pm

Tommy Hebb

[ICT-2203] [rvTVE] PM - TAX front voor LBX - oplossing voor "OudZwart" (Z8/Z9):

- Uitzondering i.g.v. OudZwart dekte de hele lading niet, er was meer aan de hand (zie volgend punt...)

- Correctie => Front, Rug en Bodem => zijn nooit bicolor (enkel zijkant kan dat zijn), dus geen opcentiemen in dat geval

- Een volledige lade bevat zijkanten, dus dan eventueel wel bicolor met bijhorende opcentiemen

Options
    • -1
    • +3
    ./SpecKostv004.cls.xml
  2. … 1 more file in changeset.
Monday 20 Sep 2021
10:41 am

Tommy Hebb

[ICT-2203] [rvTVE] PM - TAX front voor LBX - oplossing voor "OudZwart" (Z8/Z9):

- Kleine refactor en afzondering van Bicolor-percentage-verhoging

- Uitzondering voorzien indien LadeKleur OudZwart => Toch normaal unicode-percentage van 2

Options
    • -1
    • +17
    ./SpecKostv004.cls.xml
  2. … 1 more file in changeset.
Tuesday 14 Sep 2021
4:50 pm

Wim Vermeulen

De vhUtils standaard-kenmerken kiezen is soms nogal tricky: *liefst zo standaard mogelijk (behalve bij speciallekes zoals LadeMetVeelExtras) *liefst zo ondubbelzinnig mogelijk : bvb. X en Y versc...

De vhUtils standaard-kenmerken kiezen is soms nogal tricky:

  • liefst zo standaard mogelijk (behalve bij speciallekes zoals LadeMetVeelExtras)
  • liefst zo ondubbelzinnig mogelijk : bvb. X en Y verschillend bij sifonlade --> check.
  • in de meeste gevallen zo werkelijkheidsgetrouw mogelijk


in deze method vind ik enige verwarring :

  • StandaardMat heeft LadeType=TAX, die heeft (nog) geen spoelbak.
  • Spoelbak voor LBX? of eerder voor TBX/MVX? --> wellicht zijn beide methods zinvol, en dan zou ik één method expliciet MatVoorLbxSpoelbakLade() noemen.
  • bij deze LBX moeten de uitsparingBreedtes X en Y dan best wel de juiste afmeting bevatten, want dat is ook een "vaste maat"


In eerdere gesprekken had ik al wel aangegeven dat een goede keuze van kenmerkwaarden, niet zo vanzelfsprekend is, en dat ik je hierbij zeker wel zou bijstaan. Bij deze, call me

4:42 pm

Wim Vermeulen

Friday 10 Sep 2021
5:22 pm

Wim Vermeulen

Ik heb nog wat uitleg over die "E" "Exceptions" in de Json-export, maar dat is wat veel om hier in de review-opmerking te zetten. Contacteer me maar eens. Eventueel zelfs interessant voor een meeti...

Ik heb nog wat uitleg over die "E" "Exceptions" in de Json-export, maar dat is wat veel om hier in de review-opmerking te zetten.
Contacteer me maar eens. Eventueel zelfs interessant voor een meeting met JBA en THB.

4:56 pm

Wim Vermeulen

copy-paste typo (2x) : #dim ... As MVX-kenmerken ??

copy-paste typo (2x) :
#dim ... As MVX-kenmerken ??

4:48 pm

Wim Vermeulen

copy-paste 2x zelfde

copy-paste 2x zelfde