Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Misschien is het properder om de logica om de 'FilterToeleveringId' op te splitsen in een andere method?

Misschien is het properder om de logica om de 'FilterToeleveringId' op te splitsen in een andere method?

[ICT-4755] VHLOG MES M1 - Triggeren van 'Batch naar OBJT' voor een specifieke toelevering ipv hele...
[ICT-4755] VHLOG MES M1 - Triggeren van 'Batch naar OBJT' voor een specifieke toelevering ipv hele...
Deze klasse leidt af van een abstracte klasse en heeft daarom geen constructor :/

Deze klasse leidt af van een abstracte klasse en heeft daarom geen constructor :/

Ik weet dat het in deze 'stijl' of 'leeftijd' van de code niet de gewoonte was om alle %New's in de constructor te steken, maar ge moogt dat gerust doen (om de gewoonte te blijven hebben)

Ik weet dat het in deze 'stijl' of 'leeftijd' van de code niet de gewoonte was om alle %New's in de constructor te steken, maar ge moogt dat gerust doen (om de gewoonte te blijven hebben)

Ik denk dat dit veel extra werk zou opleveren en niet waard zou zijn kosten/baten

Ik denk dat dit veel extra werk zou opleveren en niet waard zou zijn kosten/baten

[ICT-1808] Uitgegrijsde playbutton bij voormontage bij productiegroep SST waar wel documenten zijn
[ICT-1808] Uitgegrijsde playbutton bij voormontage bij productiegroep SST waar wel documenten zijn
Door de bestaande testen aan te passen heb je het gedrag waarbij de laatste parameter de default is, weggemoffeld. Je moet de originele testen laten staan en een paar nieuwe testen maken die True a...

Door de bestaande testen aan te passen heb je het gedrag waarbij de laatste parameter de default is, weggemoffeld. Je moet de originele testen laten staan en een paar nieuwe testen maken die True als laatste parameter hebben.

{$$$False} ipv 0

{$$$False} ipv 0

Kan die nieuwe property eigenlijk niet mee op de UitvoeringInfo gestoken worden? In principe staan daar nog allerlei gegevens die nodig zijn om (in dit geval) etiketten af te drukken. Op die manier...

Kan die nieuwe property eigenlijk niet mee op de UitvoeringInfo gestoken worden? In principe staan daar nog allerlei gegevens die nodig zijn om (in dit geval) etiketten af te drukken. Op die manier houden we de interface(s) proper (zeker als er ooit nog andere parameters bij moeten komen).

Dit geldt ook voor sommige andere methods/klasses in deze review

comments weg

comments weg

[IC-1732] Implementatie SpaceStep Klantetiket
[IC-1732] Implementatie SpaceStep Klantetiket