CacheAdminA_trunk2010/vhUnitTest/DOM/PM/Maatwerk/Calc/HF/impl

TAOR

Checkout Tools

Line History

Line Count Graph
Line Count Graph

Stats

Commits this week: 0
Total Committers: 8
Last Commit: 11 Aug 15
Commits this week: 0
Total Lines of Code (LoC): 2.997
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 07 Nov 2022
11:06 am

Tommy Hebb

Wednesday 02 Nov 2022
6:51 pm

Wim Vermeulen

6:51 pm

Wim Vermeulen

TestFiles reviewen, is in dit soort stories bijna onbegonnen werk. Ik heb zo goed en zo kwaad mogelijk de changes bekeken. Alvast bedankt om het aantal testfiles te filteren. Het geheel lijkt me we...

TestFiles reviewen, is in dit soort stories bijna onbegonnen werk. Ik heb zo goed en zo kwaad mogelijk de changes bekeken.
Alvast bedankt om het aantal testfiles te filteren.
Het geheel lijkt me wel oké te zijn.

Ook in eCon enkele commits gereviewed.

6:38 pm

Wim Vermeulen

De dossiercode "LI7K" was blijkbaar een onbedoelde tussenstap (Tricky situatie met de TAOR Losse componenten) In combinatie met svn rev. 69413 is dit wel een correcte overgang. Hierbij is deze cha...

De dossiercode "LI7K" was blijkbaar een onbedoelde tussenstap (Tricky situatie met de TAOR Losse componenten)
In combinatie met svn rev. 69413 is dit wel een correcte overgang.

Hierbij is deze change nagekeken en goedgekeurd!

Friday 21 Oct 2022
10:35 am

Tommy Hebb

De meeste testfiles heb ik maar buiten de review gehouden. Een aantal zijn toch toegevoegd, bij wijze van check van het resultaat is dat, denk ik, wel ok en het is bovendien soms ook moeilijk om ui...

De meeste testfiles heb ik maar buiten de review gehouden. Een aantal zijn toch toegevoegd, bij wijze van check van het resultaat is dat, denk ik, wel ok en het is bovendien soms ook moeilijk om uit te maken a.d.h.v. de changesets of het al dan niet over code gaat of testfiles.
P.s.: Ook kleine change in Econ (zie svn).

10:31 am

Tommy Hebb

[ICT-3120] [rvWV] PM: TAX: Front HoogteVerstelling MoventoKoppeling: release alle klanten:
[ICT-3120] [rvWV] PM: TAX: Front HoogteVerstelling MoventoKoppeling: release alle klanten:
Wednesday 12 Oct 2022
2:21 pm

Tommy Hebb

[ICT-3120] [rvWV] PM: TAX: Front HoogteVerstelling MoventoKoppeling: release alle klanten:

- DiepteVerstellingSchroef => BuilderMoetWerken => voortaan altijd false vanwege bovenstaande + losse schroefjes geregeld door andere builder (SetSchroeven).

- HFSetSchroeven => ConditieDiepteVerstellingSchroefBepaler => Logica vóór vorige commit teruggezet, maar in comment for future reference, want...

=> Voortaan NOOIT schroefjes los meegeleverd, want weten niet welk front-materiaal de klant gebruikt en daarvan hangt keuze van schroefjes voortaan af

Options
    • -0
    • +9
    ./Builder/HFSetSchroeven/ConditieDiepteVerstellingSchroefBepaler/Test.cls.xml
  2. … 2 more files in changeset.
Friday 07 Oct 2022
5:00 pm

Tommy Hebb

[ICT-3120] [rvWV] PM: TAX: Front HoogteVerstelling MoventoKoppeling: release alle klanten:

- Nog wat UnitTest die irrelevant geworden zijn (Schroef- en DrevelKoppeling) uitgeschakeld.

- DiepteVerstelling wordt voortaan altijd los meegeleverd => geen boringen meer omdat nieuwe MoventoKoppeling => frezen ipv boren.

- DiepteVerstellingSchroef => BuilderMoetWerken => voortaan altijd false vanwege bovenstaande + losse schroefjes geregeld door andere builder (SetSchroeven).

- HFSetSchroeven => ConditieDiepteVerstellingSchroefBepaler => Logica i.g.v. DrevelKoppeling gespecifieerd, om afwijking met MoventoKoppeling te hebben. Bij DrevelKoppeling verandert er niets (maar gaat er nooit meer bijzijn).

=> I.g.v. MoventoKoppeling => Altijd schroefjes los meegeleverd, want ook DiepteVerstelling wordt dan los meegeleverd.

Options
    • -0
    • +78
    ./Builder/HFSetSchroeven/ConditieDiepteVerstellingSchroefBepaler/Test.cls.xml
    • -11
    • +29
    ./Builder/ProductieRoute/DiepteVerstelling/Test.cls.xml
    • -6
    • +24
    ./Builder/ProductieRoute/Koppeling/Test.cls.xml
  4. … 3 more files in changeset.
Tuesday 04 Oct 2022
12:00 pm

Tommy Hebb

[ICT-3120] [rvWV] PM: TAX: Front HoogteVerstelling MoventoKoppeling: release alle klanten:

- Alles is nu MoventoKoppeling => Sortering is alfabetisch en incl KoppelingType, dus volgorde nu anders

- 2 UnitTesten uitgeschakeld ivm SchroefKoppeling => ProductieRoute wegens irrelevant geworden => Is altijd MoventoKoppeling voortaan. We willen de code onder test echter nog niet verwijderen vanwege de manier waarop de infasering aangepakt werd (defaulting MoventoKoppeling op nogal vieze manier => zorgt ervoor dat deze UT nooit meer kan werken).

Options
    • -1
    • +4
    ./Builder/ProductieRoute/DiepteVerstelling/Test.cls.xml
    • -1
    • +4
    ./Builder/ProductieRoute/Koppeling/Test.cls.xml
  3. … 1 more file in changeset.
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:
Wednesday 09 Mar 2022
4:53 pm

Tommy Hebb

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

- Uitbreiding kleurenums van TAOR.

- Uitzondering voorzien voor nieuwe kleur in KenmerkDomeinPostProcessor => VHintra-configurator heeft hier geen zaken meer mee => incl. WSDL-aanpassing

- Uitbreiding PlaatAfwerkingBepaler => nieuwe melamine-kleur

- Voorlopige uitsluiting nieuwe kleur in enkele testen (te checken eens selectiekenmerken goed staan)

Options
    • -1
    • +1
    ./PlaatAfwerkingBepaler/GeefPlaatAfwerking/Test.cls.xml
  2. … 27 more files in changeset.
Friday 04 Mar 2022
9:27 am

Tommy Hebb

Thursday 03 Mar 2022
5:21 pm

Wim Vermeulen

Tuesday 01 Mar 2022
4:35 pm

Tommy Hebb

[ICT-2765] [rvWV] PM: Maatwerk: TAX: Builders: KantenbandKleurVolgtLadeKleur ProjectSetting wegwerken:...
[ICT-2765] [rvWV] PM: Maatwerk: TAX: Builders: KantenbandKleurVolgtLadeKleur ProjectSetting wegwerken:...
4:23 pm

Tommy Hebb

[ICT-2765] [rvWV] PM: Maatwerk: TAX: Builders: KantenbandKleurVolgtLadeKleur ProjectSetting wegwerken:

- Implementatie onder ProjectSetting IsActiefKantenbandKleurVolgtLadeKleur staat nu per default actief en ProjectSetting alsook oude codepad werd weggewerkt

Options
    • -11
    • +5
    ./KantenbandKleurBepaler/Test.cls.xml
  2. … 6 more files in changeset.
Monday 13 Dec 2021
11:34 am

Tommy Hebb

Friday 10 Dec 2021
5:12 pm

Wim Vermeulen

5:12 pm

Wim Vermeulen

Kleine tip : wanneer ik $$$ElkeWaarde zie staan, dan probeer ik die (als reviewer of als developer) steeds toch eens te vervangen door iets specifieker : in dit geval poging met ..IngegevenKenmerke...

Kleine tip :
wanneer ik $$$ElkeWaarde zie staan, dan probeer ik die (als reviewer of als developer) steeds toch eens te vervangen door iets specifieker : in dit geval poging met ..IngegevenKenmerken
maar dat lukte helaas niet.
Dus mag het blijven staan zoals het nu is

Wednesday 08 Dec 2021
11:33 am

Tommy Hebb

[ICT-2623] [rvWV] HX: TAX: Movento-koppeling: DiepteVerstelling-builder:
[ICT-2623] [rvWV] HX: TAX: Movento-koppeling: DiepteVerstelling-builder:
11:32 am

Tommy Hebb

[ICT-2623] [rvWV] HX: TAX: Movento-koppeling: DiepteVerstelling-builder:

- De ProductieRoute voor de DiepteVerstelling wordt bepaald a.d.h.v. het KoppelingType. Voor de MoventoKoppeling geldt dezelfde logica als voor de DrevelKoppeling.

Options
    • -0
    • +54
    ./Builder/ProductieRoute/DiepteVerstelling/Test.cls.xml
  2. … 1 more file in changeset.
Monday 29 Nov 2021
9:38 am

Tommy Hebb

9:19 am

Wim Vermeulen

9:19 am

Wim Vermeulen

dto ...MetKar hier gebruikt in de impl van BuilderZonderKar? Indien de definitie/implementatie hetzelfde is voor "metKar" en "ZonderKar", dan best de klassenaam van de dto renamen naar iets gemeens...

dto ...MetKar hier gebruikt in de impl van BuilderZonderKar?
Indien de definitie/implementatie hetzelfde is voor "metKar" en "ZonderKar", dan best de klassenaam van de dto renamen naar iets gemeenschappelijk, bvb. zonder "Kar" (if you know what I mean :-D )

Tuesday 23 Nov 2021
10:38 am

Tommy Hebb

[ICT-2564] [rvWV] HX: TAX: Movento-koppeling: Builders:
[ICT-2564] [rvWV] HX: TAX: Movento-koppeling: Builders:
Friday 29 Oct 2021
11:16 am

Tommy Hebb

[ICT-2564] [rvWV] HX: TAX: Movento-koppeling: Builders:

- Uitbreiding in Koppeling-builder => ProductieRoute voor Movento-Koppeling

- Kleine refactor in Koppeling-Builder => IsGemonteerdeLade gebruikt ipv opsomming verpakkingen voor gemonteerd => waar bovendien PalletVerpakking nog miste

- Toevoeging KoppelingAdapter-builder voor MoventoKoppeling

Options
    • -0
    • +37
    ./Builder/ProductieRoute/Koppeling/Test.cls.xml
  2. … 9 more files in changeset.
Tuesday 05 Oct 2021
12:49 pm

Tommy Hebb