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

Review Summary: No summary


----------------------------------------
File: APPS/Halux/common/impl/ProductieGroepBepalerLBX.cls.xml
Revision Comment by Wim Vermeulen on 26 September 2019, 10:50 
http://subversion02:8060/cru/CR-1830#c2730

  Klein detail : deze lijn onder de ..IngegevenKenmerkenTypeApi
  plaatsen (volgorde zelfde als de args erboven)

Revision Comment by Wim Vermeulen on 26 September 2019, 11:37 defect
http://subversion02:8060/cru/CR-1830#c2737

  Quit $$$True
    (want ik zie dat de UT-resultaten gewijzigd zijn, en dat was
  niet de bedoeling :-) )

  Reply by Kurt Callebaut on 27 September 2019, 07:51
  > De stat juist, Heb ik bewust gewijzigd. Anders krijgen we
  > standaard het gedrag dat spoelbak en smalle lade altijd in
  > spoelbakgroep terecht komen en default gedrag is dat dit niet
  > gebeurd. daarom heb ik ook de altijdsplitser moeten
  > toevoegen.
  > maw  indien true, faalde de bestaande testen op gewone
  > aantallen als er geen splitser meegegeven wordt..
  > 
  > (Ik vind het nog steeds een gevaarlijk iets om te beslissen
  > dat de groepering per kenmerk vervangen wordt door groepering
  > op productiegroep, terwijl het in dit geval toevallig
  > hetzelfde is.)

----------------------------------------
File: vhUnitTest/APPS/Halux/common/impl/ProductieGroepBepalerLBX/GeefProductieGroep/Test.cls.xml
Revision Comment by Wim Vermeulen on 26 September 2019, 12:29 
http://subversion02:8060/cru/CR-1830#c2742

  ToeleveringSplitserMock.Verifieer()

Revision Comment by Wim Vermeulen on 26 September 2019, 13:10 
http://subversion02:8060/cru/CR-1830#c2743

  Ook testmethod maken met 
  VerwachtMethodCall("IsNodigTeSplitsen" --> dan Return False

----------------------------------------
File: vhDeploy/Deploy20190919xProjectSettingsAantalVoorProductiegroep.cls.xml
Revision Comment by Wim Vermeulen on 26 September 2019, 11:11 
http://subversion02:8060/cru/CR-1830#c2735

  Deze klasse mag weg, indien geen Config-item gebruikt wordt.
  (zie opmerking bij ToeleveringSplitser)

----------------------------------------
File: APPS/Halux/common/impl/AantallenInToeleveringTeller.cls.xml
Revision Comment by Wim Vermeulen on 26 September 2019, 11:25 
http://subversion02:8060/cru/CR-1830#c2736

  AltijdSplitser kan best in de constructor, want nu krijg je een
  nieuwe instantie bij iedere "VoegToe"-call.

----------------------------------------
File: APPS/Halux/common/impl/ToeleveringSplitser.cls.xml
Revision Comment by Wim Vermeulen on 26 September 2019, 11:10 
http://subversion02:8060/cru/CR-1830#c2734

  Kritische vraag (misschien om in de groep te gooien) :
  Is dit een project-setting?  --> volgens mij enkel bedoeld om
  iets nieuw te kunnen "infaseren". Nadien wordt een
  project-setting opgekuist (in theorie althans :-) )
  Is dit dan eerder een gewone ConfigItem?  --> is eigenlijk
  bedoeld om onderscheid te kunnen maken tussen de verschillende
  omgevingen (bvb. andere waarde op PROD dan op DEV)
  Is het voldoende om deze gewoon als Class Parameter te
  definieren?  --> volgens mij wel. Dit is een
  single-responsibility klasse, dus een wijziging van de waarde
  committen en mergen, kan supersnel.
  
  Willen we dit toch wegtrekken uit onze code, dan misschien toch
  een aparte data-structuur voor maken.   Functioneel is
  config-item hiervoor ook wel geschikt, maar ik vraag me af of
  we den boel dan niet te veel gaan vervuilen?

----------------------------------------
File: APPS/Halux/common/AantalLadesBepaler.cls.xml
Revision Comment by Wim Vermeulen on 26 September 2019, 10:53 
http://subversion02:8060/cru/CR-1830#c2731

  Deze klasse is niet meer nodig, denk.

----------------------------------------
File: APPS/Halux/common/LadeVoorBewerkingsCenterBepaler.cls.xml
Revision Comment by Wim Vermeulen on 26 September 2019, 10:53 
http://subversion02:8060/cru/CR-1830#c2732

  Deze klasse is niet meer nodig, denk.


---

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

Title: ICT-817: Productiegroep Specials PM voor meer dan 10 LBX smalle lades
Statement of Objectives:
Indien er 10 of meer LBX Smalle lades in een order zitten, dan mogen deze niet mee over de lijnen gaan. Bij meer dan 10 LSL, worden deze in Special PM gestoken. 						

Dit is dezelfde werkwijze als bij Sp+Si- lades (bij meer dan 10 Sp+Si worden deze ook omgezet naar Special PM).						


State: Closed

Summary: 

Author: Kurt Callebaut
Moderator: Kurt Callebaut
Reviewers: (0 active, 1 completed*)
     Wim Vermeulen   (*)