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...
[ICT-4755] VHLOG MES M1 - Triggeren van 'Batch naar OBJT' voor een specifieke toelevering ipv hele batch
  1. … 9 more files in changeset.
[ICT-1589] Implementatie productieaansturing SpaceStep

- Telling voor geefAantalRelevanteProducten enkel laten uitvoeren bij productiegroep SpaceStep

  1. … 1 more file in changeset.
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
[ICT-1808] Uitgegrijsde playbutton bij voormontage bij productiegroep SST waar wel documenten zijn
  1. … 1 more file in changeset.
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
[IC-1732] Implementatie SpaceStep Klantetiket

- Klantetiketten voor spacestep kunnen nu afgedrukt worden nadat die van de lade al zijn afgedrukt. Vroeger kon dit niet omdat een klantetiket slechts 1x kon worden afgedrukt maar voor spacestep dient het eigenlijk 2x afgedrukt te worden, 1x voor lade en 1x voor spacestep

  1. … 5 more files in changeset.
[IC-1732] Implementatie SpaceStep Klantetiket

- Originele aansturingactie klantetiketten enkel de etiketten laten afdrukken van de lades, niet die van spacestep, dat is voor de aansturingactie klantetiketten SpaceStep

  1. … 1 more file in changeset.
[IC-1732] Implementatie SpaceStep Klantetiket

- Implementatie in commentaar plaatsen zolang deze nog niet volledig klaar is zodat testen niet falen

[IC-1732] Implementatie SpaceStep Klantetiket

- Toevoegen code zodat enkel spacestepproducten overblijven in de uitvoeringinfolijst

[IC-1732] Implementatie SpaceStep Klantetiket

- Implementatie in commentaar plaatsen zolang deze nog niet volledig klaar is zodat testen niet falen

[IC-1732] Implementatie SpaceStep Klantetiket

- Met deze commit doet aansturingsactie klantetiketten spacestep hetzelfde als de gewone aansturingsactie voor klantetiketten, later zal deze lijst aangepast worden

  1. … 3 more files in changeset.
[IC-1732] Implementatie SpaceStep Klantetiket

- Geslaagd Resultaat geven zodat testen slagen

[IC-1732] Implementatie SpaceStep Klantetiket

- Aanmaken aansturingsactie 'Klantetiketten SpaceStep' in groep voormontage

    • -0
    • +36
    ./KlantEtiketSpaceStep.cls.xml
  1. … 3 more files in changeset.