CacheAdminA_trunk2010/vhUnitTest/DOM/PM/Maatwerk/Calc

HF

Checkout Tools

Line History

Line Count Graph
Line Count Graph

Stats

Commits this week: 0
Total Committers: 10
Last Commit: 17 Jul 15
Commits this week: 0
Total Lines of Code (LoC): 5.412
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
Wednesday 06 Sep 2023
8:32 am

Peter Vercoutter

[ICT-3672] Uitbreiding LBX BOMBOL in .NET

- Test mag berekingwijze encoway negeren

Options
    • -6
    • +9
    ./HalffabSpec/BepaalHFItemBuilderSequence/TestAllHFSpecs.cls.xml
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
    ./impl/TAOR/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
    ./impl/TAOR/Builder/HFSetSchroeven/ConditieDiepteVerstellingSchroefBepaler/Test.cls.xml
    • -11
    • +29
    ./impl/TAOR/Builder/ProductieRoute/DiepteVerstelling/Test.cls.xml
    • -6
    • +24
    ./impl/TAOR/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
    ./impl/TAOR/Builder/ProductieRoute/DiepteVerstelling/Test.cls.xml
    • -1
    • +4
    ./impl/TAOR/Builder/ProductieRoute/Koppeling/Test.cls.xml
  3. … 1 more file in changeset.
Tuesday 14 Jun 2022
4:36 pm

Joannes Lemmens

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 )

Friday 13 May 2022
8:25 am

Peter Vercoutter

Thursday 12 May 2022
5:27 pm

Joannes Lemmens

Woohoow, het is klaar! Op Jira heb ik weer afbeeldingen gezet van hoe het er nu uitziet, inclusief voor het productiedetailblad (wat oorspronkelijk niet op het kaartje stond)

Woohoow, het is klaar! Op Jira heb ik weer afbeeldingen gezet van hoe het er nu uitziet, inclusief voor het productiedetailblad (wat oorspronkelijk niet op het kaartje stond)

5:22 pm

Joannes Lemmens

[ICT-2777] [rv PVR] HLX - LBX - Pickinglijst losse onderdelen - betere aanduiding ivm afdekkap buiten...
[ICT-2777] [rv PVR] HLX - LBX - Pickinglijst losse onderdelen - betere aanduiding ivm afdekkap buiten...
Thursday 05 May 2022
8:20 am

Joannes Lemmens

[ICT-2777] [rv PVR] HLX - LBX - Pickinglijst losse onderdelen - betere aanduiding ivm afdekkap buiten + afdekkap binnen:

- Rechtzetting van eerdere wijziging. Dit stukje code komt ergens anders te staan

Options
    • -12
    • +2
    ./impl/LBX/Afdekkappen/TestHFAfdekkapBinnenv000N.cls.xml
  2. … 1 more file in changeset.
Monday 25 Apr 2022
5:28 pm

Joannes Lemmens

Op het kaartje in Jira heb ik een afbeelding gepost van hoe het eruit ziet na de wijzigingen (of de papieren op m'n bureau mocht ge daar ooit eens passeren).

Op het kaartje in Jira heb ik een afbeelding gepost van hoe het eruit ziet na de wijzigingen (of de papieren op m'n bureau mocht ge daar ooit eens passeren).

5:19 pm

Joannes Lemmens

[ICT-2777] [rv PVR] HLX - LBX - Pickinglijst losse onderdelen - betere aanduiding ivm afdekkap buiten + afdekkap binnen:

- De dingen die in bold moesten komen in bold gezet (en unit test aangepast)

Options
    • -2
    • +12
    ./impl/LBX/Afdekkappen/TestHFAfdekkapBinnenv000N.cls.xml
  2. … 2 more files in changeset.
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
    ./impl/TAOR/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
    ./impl/TAOR/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