HFSetSchroevenv000.cls.xml

Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
[UST3970] TAX diepteverstelling op pickinglijst losse onderdelen:

- Niet onbelangrijk => is DiepteVerstelling wel aanwezig

  1. … 5 more files in changeset.
idem als in de bodemondersteuning

idem als in de bodemondersteuning

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

[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

  1. … 22 more files in changeset.
[UST3970] TAX diepteverstelling op pickinglijst losse onderdelen:

- refactor liet het bord bloedrood worden en aangezien THB/TommyH er niet is vandaag, commit ik deze even.

(initialen hierboven gebruikt om er voor te zorgen dat je deze revisie ZEKER meeneemt)

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

[UST3970] TAX diepteverstelling op pickinglijst losse onderdelen:

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

  1. … 5 more files in changeset.
in apart klaske met zijn eigen testjes steken. veiliger voor de toekomst.

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

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

- Voor Productiedetailblad wordt nu onderscheid gemaakt in aantal schroeven volgens => IsSchroefKoppelingAanwezig EN IsGemonteerdeLade = False => in GeefConditieDiepteVerstellingSchroef

  • More
  • CR-132
  • finished reviewing
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

Kan in dit geval niet met de standaardlade gewerkt worden?

Kan in dit geval niet met de standaardlade gewerkt worden?

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

[UST2222] TAX set schroeven: fase 2: ook diepteverstellingschroeven in set schroeven opnemen
[UST2222] TAX set schroeven: fase 2: ook diepteverstellingschroeven in set schroeven opnemen
[UST1797] TAX set schroeven: fase 1: ook bodemondersteuningschroeven meeleveren in de set

- Conditie bodemondersteuning / koppelingschroef ook adhv isgemonteerd zodat flatpackprocom meegenomen wordt.

  1. … 9 more files in changeset.
[UST2222] TAX set schroeven: fase 2: ook diepteverstellingschroeven in set schroeven opnemen

- Correctie conditie diepteverstelling is afhankelijk van gemonteerde lade en is een set L+R

  1. … 1 more file in changeset.
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.

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...

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.

[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)

  1. … 1 more file in changeset.
[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 ?

  1. … 1 more file in changeset.
[UST2222] TAX set schroeven: fase 2: ook diepteverstellingschroeven in set schroeven opnemen

- MoetBuilderWerken condities verschillend per onderdeel bodemondersteuning, diepteverstelling, koppeling.

  1. … 1 more file in changeset.
[UST2222] TAX set schroeven: fase 2: ook diepteverstellingschroeven in set schroeven opnemen

- set schroeven berekenen met Diepteverstelling.

- Todo MoetBuilderWerken in base class herbekijken.

  • More
  • CR-91
  • finished reviewing
Waarom is er geen versie voor FlatpackV1 ??

Waarom is er geen versie voor FlatpackV1 ??