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

HFSetSchroeven

Checkout Tools

Line History

Line Count Graph
Line Count Graph

Stats

Commits this week: 0
Total Committers: 1
Last Commit: 29 Jan 19
Commits this week: 0
Total Lines of Code (LoC): 107
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
    • -24
    • +22
    ./ConditieDiepteVerstellingSchroefBepaler.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
    • -1
    • +10
    ./ConditieDiepteVerstellingSchroefBepaler.cls.xml
  2. … 5 more files in changeset.
Tuesday 19 Feb 2019
10:42 am

Tommy Hebb

[UST3970] TAX diepteverstelling op pickinglijst losse onderdelen:

- Niet onbelangrijk => is DiepteVerstelling wel aanwezig

Options
    • -4
    • +9
    ./ConditieDiepteVerstellingSchroefBepaler.cls.xml
  2. … 5 more files in changeset.
Wednesday 06 Feb 2019
3:52 pm

Tommy Hebb

[UST3970] TAX diepteverstelling op pickinglijst losse onderdelen:

- Na review => mini-mini-mini-refactor

Options
    • -4
    • +1
    ./ConditieBodemOndersteuningSchroefBepaler.cls.xml
    • -4
    • +1
    ./ConditieKoppelingSchroefBepaler.cls.xml
1:38 pm

Tommy Hebb

Monday 04 Feb 2019
11:55 am

Tom Vermeulen

10:25 am

Tom Vermeulen

idem als in de bodemondersteuning

idem als in de bodemondersteuning

10:22 am

Tom Vermeulen

als ge toch '> 0' doet, kunt ge evengoed de boolean overslaan http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif

als ge toch '> 0' doet, kunt ge evengoed de boolean overslaan

Friday 01 Feb 2019
12:26 pm

Tommy Hebb

[UST3970] TAX diepteverstelling op pickinglijst losse onderdelen:

- ConditieBodemOndersteuningSchroef en ConditieKoppelingSchroef en ConditieDiepteVerstellingSchroef afgezonderd naar hun respectievelijke Bepalers met elk een eigen testcase

- Bovengenoemde Bepalers geinjecteerd in constructor van TAOR.Builder.HFSetSchroevenv000 voor testcase BuilderMoetWerken

- Nieuwe testcase BuilderMoetWerken => check op resultaat bovengenoemde condities => via mocks

- Oude en foute testcase-opzet voor BuilderMoetWerken verwijderd => nogal onleesbare opzet en bovendien niet SOLID => verweven verantwoordelijkheden

Options
    • -0
    • +34
    ./ConditieKoppelingSchroefBepaler.cls.xml
    • -0
    • +26
    ./ConditieBodemOndersteuningSchroefBepaler.cls.xml
    • -1
    • +1
    ./ConditieDiepteVerstellingSchroefBepaler.cls.xml
  4. … 20 more files in changeset.
Wednesday 30 Jan 2019
8:21 am

Tom Vermeulen

Best hier een paar puntjes voorzetten, ander gaat ge wat vonken krijgen op het bord http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif

Best hier een paar puntjes voorzetten, ander gaat ge wat vonken krijgen op het bord

Tuesday 29 Jan 2019
2:41 pm

Tommy Hebb

[UST3970] TAX diepteverstelling op pickinglijst losse onderdelen:

- Na review - Rework na refactor door afzondering en bijhorende testcase

Options
  1. ./HFSetSchroeven
    • -0
    • +41
    ./ConditieDiepteVerstellingSchroefBepaler.cls.xml
  3. … 4 more files in changeset.
Monday 28 Jan 2019
9:57 am

Tom Vermeulen

in apart klaske met zijn eigen testjes steken. veiliger voor de toekomst.

in apart klaske met zijn eigen testjes steken. veiliger voor de toekomst.

Friday 25 Jan 2019
4:31 pm

Tommy Hebb

[UST3970] TAX diepteverstelling op pickinglijst losse onderdelen:
[UST3970] TAX diepteverstelling op pickinglijst losse onderdelen: