EtiketAfdrukker.cls.xml

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-3054] [rvPVR] RVG aansturingactie: Klantetiketten
[ICT-3054] [rvPVR] RVG aansturingactie: Klantetiketten
[ICT-3054] [rvPVR] RVG aansturingactie: Klantetiketten

- Fout loggen in plaats van enkel catchen

[ICT-1732] Implementatie SpaceStep Klantetiket

- Reviewopmerkingen

  1. … 5 more files in changeset.
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-1732] Implementatie SpaceStep Klantetiket

- Opkuis ongebruikte code. Deze code werd gedeveloped voor wanneer spacesteps en lades toch in eenzelfde batch zouden moeten komen

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

- Verschil maken tussen klantetiketten voor lades en voor spacestep die via dezelfde code lopen

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

- Refactoring nodig om correcte lijst terug te krijgen

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