EtiketBuilder.cls.xml

Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Voor mij oke! Ik zou misschien hebben gekozen voor een andere variabele naam dan 'IsSpecialeLade' want dit geeft niet meteen aan waarvoor het zal worden gebruikt.

Voor mij oke! Ik zou misschien hebben gekozen voor een andere variabele naam dan 'IsSpecialeLade' want dit geeft niet meteen aan waarvoor het zal worden gebruikt.

[ICT-3849] [rvPVR] Koppelen orgalux sticker aan lade code in het bewerkingscenter:
[ICT-3849] [rvPVR] Koppelen orgalux sticker aan lade code in het bewerkingscenter:
dit is idd één van de gevallekes waar opgelet gaat moeten worden

dit is idd één van de gevallekes waar opgelet gaat moeten worden

Kan het dat deze ook al aangepast was in ICT-2171? Geen idee of dit voor conflicten kan zorgen?

Kan het dat deze ook al aangepast was in ICT-2171? Geen idee of dit voor conflicten kan zorgen?

[ICT-2146] pa-ub-215: vhossetiketten volgorde dnacodes
[ICT-2146] pa-ub-215: vhossetiketten volgorde dnacodes
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-1804] PA-UB-150 - Vhosskasten krijgen niet altijd een SequentieNummer in de bijhorende DnaCodes...
[ICT-1804] PA-UB-150 - Vhosskasten krijgen niet altijd een SequentieNummer in de bijhorende DnaCodes...
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
Is het interessant om als het om een herafdruk ga te checken of er een batchid werd meegeven? Om zo een nauwkeurigere error log te hebben. Of wordt misschien al gedaan in de ZoekViaBatchIdEnSequent...

Is het interessant om als het om een herafdruk ga te checken of er een batchid werd meegeven? Om zo een nauwkeurigere error log te hebben. Of wordt misschien al gedaan in de ZoekViaBatchIdEnSequentienummer!?

juist een bedenking. Moet deze test ook voor klantetiketten?

juist een bedenking. Moet deze test ook voor klantetiketten?

[ICT-1702] PA-UB-139: Planscan: TAOR etiketten afdrukken
[ICT-1702] PA-UB-139: Planscan: TAOR etiketten afdrukken
Batch gaat hier altijd de Batch van de laatste DnaCode in de Iterator geven. Waarom is de whileloop nodig?

Batch gaat hier altijd de Batch van de laatste DnaCode in de Iterator geven. Waarom is de whileloop nodig?

ook hier beter een exceptie raisen ipv een wlipke

ook hier beter een exceptie raisen ipv een wlipke

BatchService injecten en testen voorzien die het gedrag van een true of false antwoord op BestaatBatch illustreren, dan had het gedoe hieronder niet naar boven gekomen http://subversion02.vanhoecke...

BatchService injecten en testen voorzien die het gedrag van een true of false antwoord op BestaatBatch illustreren, dan had het gedoe hieronder niet naar boven gekomen

bovenstaand stukje is wreed tricky. In principe gebruiken we hoe dan ook geen early quits, want die maken het lezen/volgen van code moeilijker. Wat gebeurt er met ErrorMessage? Niets http://subvers...

bovenstaand stukje is wreed tricky.
In principe gebruiken we hoe dan ook geen early quits, want die maken het lezen/volgen van code moeilijker.
Wat gebeurt er met ErrorMessage? Niets
In mijn testgeval gaf BestaatBatch blijkbaar een false en er is niets in resultaat/exception dat daarop wijst. De code geeft zelfs een 200 terug op hoger niveau

wlipkes galore! (zie lager)

wlipkes galore! (zie lager)

als je dus een exceptie smijt in de lagere stukken, dan kan deze reutemeteut ook weg http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif

als je dus een exceptie smijt in de lagere stukken, dan kan deze reutemeteut ook weg