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

Builder

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.560
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
    ./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
    ./HFSetSchroeven/ConditieDiepteVerstellingSchroefBepaler/Test.cls.xml
    • -11
    • +29
    ./ProductieRoute/DiepteVerstelling/Test.cls.xml
    • -6
    • +24
    ./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
    ./ProductieRoute/DiepteVerstelling/Test.cls.xml
    • -1
    • +4
    ./ProductieRoute/Koppeling/Test.cls.xml
  3. … 1 more file 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
    ./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
    ./ProductieRoute/Koppeling/Test.cls.xml
  2. … 9 more files in changeset.
Friday 09 Apr 2021
9:03 am

Peter Vercoutter

Thursday 08 Apr 2021
12:35 pm

Jelle Baele

Wednesday 07 Apr 2021
10:08 am

Peter Vercoutter

[ICT-2297] TAX - bodemondersteuning in bouwstenen met 0 stuks
[ICT-2297] TAX - bodemondersteuning in bouwstenen met 0 stuks
10:05 am

Peter Vercoutter

[ICT-2297] TAX - bodemondersteuning in bouwstenen met 0 stuks

- Om 0 stuks te voorkomen op het productiedetailblad moet de builder gewoonweg niet werken in de situatie waar 0 stuks uitkomen

Options
    • -12
    • +22
    ./HFBodemOndersteuning/Test.cls.xml
  2. … 2 more files in changeset.
Friday 22 Nov 2019
10:08 am

Martijn Vierstraete

Tuesday 12 Nov 2019
3:32 pm

Tommy Hebb

[ICT-717] [ICT502] [rvWV] UT VerpakkingTypes: IsgemonteerdeLade() uitmocken:

- Na review => Eerst Super uitvoeren in constructors

- Rechtzetting in naamgeving UnitTest

Options
    • -1
    • +1
    ./ProductieRoute/BodemOndersteuning/Test.cls.xml
  2. … 4 more files in changeset.
3:27 pm

Tommy Hebb

3:22 pm

Tommy Hebb

Aangezien daar geen automatische testen voor zijn, doen we dat af en toe maar eens manueel hé http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/wink.gif Stom copy-p...

Aangezien daar geen automatische testen voor zijn, doen we dat af en toe maar eens manueel hé
Stom copy-paste-foutje hé

Friday 08 Nov 2019
5:16 pm

Wim Vermeulen

5:15 pm

Wim Vermeulen

Tekst in methodnaam klopt niet meer. Is nog steeds "Onderdelenpicking. om de reviewer te testen, zekers? http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/wink.gif

Tekst in methodnaam klopt niet meer. Is nog steeds "Onderdelenpicking.

om de reviewer te testen, zekers?

4:43 pm

Wim Vermeulen

Graa deze aanpassen naar de nieuwere afspraak. Dus eerste lijn Do ##super() en op t einde Quit $$$OK

Graa deze aanpassen naar de nieuwere afspraak.
Dus eerste lijn Do ##super() en op t einde Quit $$$OK