This is a list of all comments for CR-91.

Review Summary: No summary


----------------------------------------
File: vhTestFiles/UnitTests/APPS.PM.Maatwerk.End2End.TAOR/TestTAORBodemOndersteuning/TestStandaardladeverpakkingMet3BodemondersteuningenBepaaltDoorBreedte1800/FlatpackV1Vandaag.txt
Revision Comment by Jo Claes on 21 February 2017, 14:09 
http://subversion02:8060/cru/CR-91#c232

  Waarom is er geen versie voor FlatpackV1 ??

----------------------------------------
File: vhUnitTest/End2End/TAOR/GebruikteHalffabItemsVoorPrijsberekening/Test.cls.xml
Revision Comment by Jo Claes on 21 February 2017, 14:08 defect
http://subversion02:8060/cru/CR-91#c231

  Waarom is deze niet toegevoegd bij de StandaardLadeItems()

  Reply by Kurt Callebaut on 22 February 2017, 14:29
  > Standaardbinnenlade wordt getest met --->
  > CopyListOfDataTypes(..StandaardLadeItems(),..StandaardBinnenladeItems())
  > maw gebruikt standaardlade 
  > en binnenlade bevat geen setschroeven.

----------------------------------------
File: DOM/PM/Maatwerk/Calc/HF/impl/TAOR/Builder/Generated/HFSetSchroevenv000.cls.xml
Revision Comment by Jo Claes on 21 February 2017, 13:44 defect
http://subversion02:8060/cru/CR-91#c230

  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

  Reply by Kurt Callebaut on 22 February 2017, 14:33
  > 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.

Revision Comment by Jo Claes on 21 February 2017, 13:42 defect
http://subversion02:8060/cru/CR-91#c229

  HeeftSpecifiekeProductieWijze lijkt mij toch een betere
  naamkeuze..

  Reply by Kurt Callebaut on 22 February 2017, 14:32
  > SpecifiekeProductieWijze wordt nergens in de code gebruikt,
  > dan zouden we de rest ook moeten refactoren naar
  > specifieke...


---

ID: CR-91 http://subversion02:8060/cru/CR-91

Title: [UST1797] TAX set schroeven: fase 1: ook bodemondersteuningschroeven meeleveren in de set
Statement of Objectives:


State: Closed

Summary: 

Author: Kurt Callebaut
Moderator: Kurt Callebaut
Reviewers: (0 active, 1 completed*)
     Jo Claes   (*)