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

TAOR

Checkout Tools

Line History

Line Count Graph
Line Count Graph

Stats

Commits this week: 0
Total Committers: 11
Last Commit: 23 Jun 15
Commits this week: 0
Total Lines of Code (LoC): 4.109
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
Thursday 01 Dec 2022
11:38 am

Joannes Lemmens

10:53 am

Tommy Hebb

Wednesday 30 Nov 2022
2:45 pm

Joannes Lemmens

[ICT-3198] TAX - Dubbele dwarsverdeling - Caché gedeelte
[ICT-3198] TAX - Dubbele dwarsverdeling - Caché gedeelte
Tuesday 20 Sep 2022
10:42 am

Tommy Hebb

10:38 am

Tommy Hebb

Ge ziet niets over 't hoofd :-D Ge hebt helemaal gelijk! Is een 1-op-1-property, dus mag automatisch geconverteerd worden. In tegenstelling tot DwarsverdelingAantal, want daar is de property-naam v...

Ge ziet niets over 't hoofd :-D Ge hebt helemaal gelijk! Is een 1-op-1-property, dus mag automatisch geconverteerd worden. In tegenstelling tot DwarsverdelingAantal, want daar is de property-naam verschillend (cq. AantalDwarsverdelingen in APPS).

10:14 am

Tommy Hebb

Aangepast naar "Dwarsverdelingen te picken"

Aangepast naar "Dwarsverdelingen te picken"

Monday 19 Sep 2022
12:02 pm

Wim Vermeulen

12:02 pm

Wim Vermeulen

Als opmerking in de kantlijn (dus je moet niks aanpassen) : hier moeten we eigenlijk gebruik maken van het berekende HalffabItem en daarvan de (bewerkte) lengte opvragen. Nu staat er duplicate logi...

Als opmerking in de kantlijn (dus je moet niks aanpassen) : hier moeten we eigenlijk gebruik maken van het berekende HalffabItem en daarvan de (bewerkte) lengte opvragen. Nu staat er duplicate logica.
Maar ik geloof best, zonder het na te kijken, dat in deze klasse nog code staat die vertrekt van de ingegevenKenmerken i.p.v. halffabItem.

11:44 am

Wim Vermeulen

DwarsverdelingKleur zit volgens mij "impliciet" in de ConverteerGegenereerdeProperties() hiervoor is geen "uitzondering nodig". Tijdens de uitbreidingsfase en review lijkt het me handig om te zien ...

DwarsverdelingKleur zit volgens mij "impliciet" in de ConverteerGegenereerdeProperties()
hiervoor is geen "uitzondering nodig".
Tijdens de uitbreidingsfase en review lijkt het me handig om te zien dat beide properties zijn toegevoegd. Maar eenmaal in productie (en ondersteund door automatische testen) kan dit het standaard pad volgen.
Of zie ik iets over t hoofd?

11:10 am

Wim Vermeulen

Tekst correct? Zonder kar --> geen vakken, right?

Tekst correct?
Zonder kar --> geen vakken, right?

Tuesday 13 Sep 2022
3:25 pm

Tommy Hebb

Staat al in productie en is voordien al uitgebreid end-to-end getest.

Staat al in productie en is voordien al uitgebreid end-to-end getest.

3:24 pm

Tommy Hebb

[ICT-2728] [rvWV] TAX configurator : uitbreiden met een dwarsverdeling:
[ICT-2728] [rvWV] TAX configurator : uitbreiden met een dwarsverdeling:
8:09 am

Joannes Lemmens

Monday 05 Sep 2022
8:25 am

Jelle Baele

Thursday 01 Sep 2022
3:53 pm

Joannes Lemmens

[ICT-3077] TAX dwarsverdeling - kleine foutjes/aanvullingen
[ICT-3077] TAX dwarsverdeling - kleine foutjes/aanvullingen
Tuesday 02 Aug 2022
9:32 am

Wim Vermeulen

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.

Thursday 28 Oct 2021
9:44 am

Peter Vercoutter