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

BuilderMoetWerken

Checkout Tools

Line History

Line Count Graph
Line Count Graph

Stats

Commits this week: 0
Total Committers: 2
Last Commit: 16 Feb 17
Commits this week: 0
Total Lines of Code (LoC): 174
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
Tuesday 19 Feb 2019
10:42 am

Tommy Hebb

[UST3970] TAX diepteverstelling op pickinglijst losse onderdelen:

- Niet onbelangrijk => is DiepteVerstelling wel aanwezig

Options
    • -4
    • +4
    ./Test.cls.xml
  2. … 5 more files in changeset.
Wednesday 06 Feb 2019
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
    • -63
    • +0
    ./TestDiepteVerstelling.cls.xml
    • -0
    • +147
    ./Test.cls.xml
    • -59
    • +0
    ./TestBodemOndersteuning.cls.xml
    • -32
    • +1
    ./HFBuilderMock.cls.xml
    • -56
    • +0
    ./Testbase.cls.xml
  6. … 18 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

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:
Tuesday 28 Mar 2017
10:33 am

Kurt Callebaut

Wednesday 22 Mar 2017
10:53 am

Martijn Vierstraete

  • More
  • CR-132
  • finished reviewing
10:51 am

Martijn Vierstraete

Het hergenereren van deze builder zal opnieuw deze conditie toevoegen. De conditie voor rol SetSchroeven in het TaorboxModel mag verwijderd worden

Het hergenereren van deze builder zal opnieuw deze conditie toevoegen. De conditie voor rol SetSchroeven in het TaorboxModel mag verwijderd worden

10:49 am

Martijn Vierstraete

Kan in dit geval niet met de standaardlade gewerkt worden?

Kan in dit geval niet met de standaardlade gewerkt worden?

10:46 am

Martijn Vierstraete

Heb je ook de TaorboxModel.Rollen.SetSchroeven aangepast? Dit kan mogelijks fout lopen bij hergenerer

Heb je ook de TaorboxModel.Rollen.SetSchroeven aangepast? Dit kan mogelijks fout lopen bij hergenerer

Wednesday 15 Mar 2017
2:14 pm

Kurt Callebaut

[UST2222] TAX set schroeven: fase 2: ook diepteverstellingschroeven in set schroeven opnemen
[UST2222] TAX set schroeven: fase 2: ook diepteverstellingschroeven in set schroeven opnemen
Wednesday 22 Feb 2017
2:34 pm

Kurt Callebaut

2:34 pm

Kurt Callebaut

De klasse wordt maar éénmalig gegenereerd en ik vind het niet correct om foutieve code te laten staan, ook al is het een gegenereerde klasse en ook al wordt deze overridden in een andere klasse. Di...

De klasse wordt maar éénmalig gegenereerd en ik vind het niet correct om foutieve code te laten staan, ook al is het een gegenereerde klasse en ook al wordt deze overridden in een andere klasse. Dit kan enkel maar verwarrend werken en kans geven op fouten als anderen de gegenereerde gaan gebruiken.

2:32 pm

Kurt Callebaut

SpecifiekeProductieWijze wordt nergens in de code gebruikt, dan zouden we de rest ook moeten refactoren naar specifieke...

SpecifiekeProductieWijze wordt nergens in de code gebruikt, dan zouden we de rest ook moeten refactoren naar specifieke...

2:30 pm

Kurt Callebaut

Standaardbinnenlade wordt getest met ---> CopyListOfDataTypes(..StandaardLadeItems(),..StandaardBinnenladeItems()) maw gebruikt standaardlade en binnenlade bevat geen setschroeven.

Standaardbinnenlade wordt getest met ---> CopyListOfDataTypes(..StandaardLadeItems(),..StandaardBinnenladeItems()) maw gebruikt standaardlade
en binnenlade bevat geen setschroeven.

1:38 pm

KurtC

[UST2222] TAX set schroeven: fase 2: ook diepteverstellingschroeven in set schroeven opnemen

- beslist om 1 set schroeven steeds 2 schroeven toe te voegen (builder moet werken) voor 1 of 2 diepteverstellingen. (mogelijks L en/of R)

Options
    • -2
    • +13
    ./TestDiepteVerstelling.cls.xml
  2. … 1 more file in changeset.
1:24 pm

KurtC

[UST2222] TAX set schroeven: fase 2: ook diepteverstellingschroeven in set schroeven opnemen

- opsplitsen testfiles.

Options
    • -0
    • +59
    ./TestBodemOndersteuning.cls.xml
    • -0
    • +56
    ./Testbase.cls.xml
    • -151
    • +0
    ./Test.cls.xml
    • -0
    • +52
    ./TestDiepteVerstelling.cls.xml
1:11 pm

KurtC

[UST2222] TAX set schroeven: fase 2: ook diepteverstellingschroeven in set schroeven opnemen

- Test diepteverstelling. (nog opsplitsen naar verschjillende testfiles.) Aantal schroeven momenteel 2, nog te bespreken hoe omgaan met 1 schroef. Geen set of 1 set ?

Options
    • -1
    • +45
    ./Test.cls.xml
  2. … 1 more file in changeset.
9:41 am

KurtC

[UST2222] TAX set schroeven: fase 2: ook diepteverstellingschroeven in set schroeven opnemen

- Herwerkt zodat aantallen voor andere gebruikte rollen bij SetSchroeven ook meegegeven kunnen worden, met oog op diepteverstelling, koppeling.

Options
    • -7
    • +13
    ./Test.cls.xml
    • -2
    • +18
    ./HFBuilderMock.cls.xml
Tuesday 21 Feb 2017
2:10 pm

Jo Claes

  • More
  • CR-91
  • finished reviewing
2:10 pm

Jo Claes

Waarom is er geen versie voor FlatpackV1 ??

Waarom is er geen versie voor FlatpackV1 ??

2:08 pm

Jo Claes

Waarom is deze niet toegevoegd bij de StandaardLadeItems()

Waarom is deze niet toegevoegd bij de StandaardLadeItems()

1:44 pm

Jo Claes

Volgens de overlevering mag je geen Generated klassen aanpassen , maar moet je daar een afgeleide klasse van maken en daar dan de specifieke implementatie in overschrijven

Volgens de overlevering mag je geen Generated klassen aanpassen , maar moet je daar een afgeleide klasse van maken en daar dan de specifieke implementatie in overschrijven

1:43 pm

Jo Claes

HeeftSpecifiekeProductieWijze lijkt mij toch een betere naamkeuze..

HeeftSpecifiekeProductieWijze lijkt mij toch een betere naamkeuze..

Monday 20 Feb 2017
4:07 pm

Kurt Callebaut

  • More
  • CR-91
  • started review
[UST1797] TAX set schroeven: fase 1: ook bodemondersteuningschroeven meeleveren in de set
[UST1797] TAX set schroeven: fase 1: ook bodemondersteuningschroeven meeleveren in de set
Thursday 16 Feb 2017
3:07 pm

KurtC

[UST1797] TAX set schroeven: fase 1: ook bodemondersteuningschroeven meeleveren in de set

- Set schroeven - buildermoetwerken voor V1 en Probox zolang het geen binnenlade is. (heeft impact op de prijs.)

Options
    • -3
    • +33
    ./Test.cls.xml
  2. … 45 more files in changeset.