Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
[ICT-2634] [rvPVR] Automatische matjessnijder:
[ICT-2634] [rvPVR] Automatische matjessnijder:
De method under test : ToeleveringService.GeefLijnReferentie() kan ook in de Catch{} terecht komen. Dat codepad kan dan best ook ge-unittest worden. In dit geval zal de werkwijze wellicht iets ande...

De method under test : ToeleveringService.GeefLijnReferentie() kan ook in de Catch{} terecht komen.
Dat codepad kan dan best ook ge-unittest worden.
In dit geval zal de werkwijze wellicht iets anders zijn dan bij de gewone "ZetVerwachteExceptie()"
Bij twijfel, vraag gerus.

Bij deze exceptie-omschrijving staat bijzonder weinig info (geen eigenlijk) waardoor de persoon die dit moet troubleshooten sowieso extra tijd zal nodig hebben om uit te pluizen over welke Batch/kl...

Bij deze exceptie-omschrijving staat bijzonder weinig info (geen eigenlijk) waardoor de persoon die dit moet troubleshooten sowieso extra tijd zal nodig hebben om uit te pluizen over welke Batch/klant/palletID/... dit hier gaat. M.a.w. tracht bij een exceptie steeds zoveel mogelijk context te scheppen. En indien dit niet mogelijk is in deze stacklevel, dan zeker checken dat er een Try-Catch op een hoger niveau gebeurt, waar wel meer context aan de exceptie toegevoegd wordt.

Deze opmerking hoort wellicht niet tot de review van deze story, dus feel free om door te geven aan de vorige owner

Over gecombineerde condities (i.e. conditie 1 en conditie 2 hebben een geheel andere context) en waarvan één conditie klantspecifieke filtering is, wil ik je nog een inzicht delen. Uitleg ca. 5 à 1...

Over gecombineerde condities (i.e. conditie 1 en conditie 2 hebben een geheel andere context) en waarvan één conditie klantspecifieke filtering is, wil ik je nog een inzicht delen.
Uitleg ca. 5 à 10 minuten, dus graag effe samenzitten.

[ICT-2954] De Decker - barcodes op etiketten van productie
[ICT-2954] De Decker - barcodes op etiketten van productie
Enkele opmerkingen hierbij: 1) Het etiket zelf is: \\bartender\Bartender\Templates\ProboxLijnReferentieBarcode.btw 2) De DnaCodes in de batch zijn gegroepeerd op DossierCode en LijnReferentie. Vana...

Enkele opmerkingen hierbij:
1) Het etiket zelf is: \\bartender\Bartender\Templates\ProboxLijnReferentieBarcode.btw
2) De DnaCodes in de batch zijn gegroepeerd op DossierCode en LijnReferentie. Vanaf 1 van deze verschillend is wordt een nieuwe 'groep' gemaakt. Echter gekozen voor een sorteeralgoritme omdat dit al bestond en ook wel op een logische manier werkt.
3) De LijnReferentieBepaler is verhuisd naar de ToeleveringService: er waren al 2 exacte dezelfde klasses plus tests hiervoor, een derde zou wat teveel worden. Daarom gekozen om het op een centralere plaats te zetten

detail, maar hier kan je gewoon Quit ##class(TECH.ListIterator).%New(DnaCodeList) gebruiken. Het dimmen in een aparte variabele heeft niet echt meerwaarde omdat de dim bij cache enkel zorgt voor co...

detail, maar hier kan je gewoon Quit ##class(TECH.ListIterator).%New(DnaCodeList) gebruiken. Het dimmen in een aparte variabele heeft niet echt meerwaarde omdat de dim bij cache enkel zorgt voor code-completion

[ICT-2520] [rvTVE] Sommige referenties van ladetype ontbreken soms op ProboxinhoudEtiket
[ICT-2520] [rvTVE] Sommige referenties van ladetype ontbreken soms op ProboxinhoudEtiket
[ICT-2520] [rvTVE] Sommige referenties van ladetype ontbreken soms op ProboxinhoudEtiket
[ICT-2520] [rvTVE] Sommige referenties van ladetype ontbreken soms op ProboxinhoudEtiket
En eigenlijk moet die in de constructor genewed worden. Ik weet het, aan de ene kant "deze klasse gebruikt die manier van werken nog niet" en aan de andere kant het "boyscout principe". Your call

En eigenlijk moet die in de constructor genewed worden. Ik weet het, aan de ene kant "deze klasse gebruikt die manier van werken nog niet" en aan de andere kant het "boyscout principe". Your call

Done!

Done!

= Code van Wim. Wellicht zijn er teveel verschillende mogelijkheden om een al te specifieke te gebruiken hé.

= Code van Wim. Wellicht zijn er teveel verschillende mogelijkheden om een al te specifieke te gebruiken hé.

Goed gezien :-s Bij deze aangepast!

Goed gezien :-s Bij deze aangepast!

ik veronderstel dat dit een copy paste van lbx is, maar misschien toch de ' door $$$Not vervangen

ik veronderstel dat dit een copy paste van lbx is, maar misschien toch de ' door $$$Not vervangen

Als er ooit vhossen bij mvx komen, gaat ge het probleem hebben dat er geen vhossetiketten worden afgedrukt. Dat is een uitbreiding die dan nog moet toegevoegd worden op de KlantEtiket.PrinterFactory

Als er ooit vhossen bij mvx komen, gaat ge het probleem hebben dat er geen vhossetiketten worden afgedrukt. Dat is een uitbreiding die dan nog moet toegevoegd worden op de KlantEtiket.PrinterFactory

is het niet beter om een specifiekere exception te gebruiken? CommunicationErrorException ofzo

is het niet beter om een specifiekere exception te gebruiken? CommunicationErrorException ofzo

moet hier ook geen ...Merivobox voorzien worden (zelfs als die gelijk is aan de Legrabox)? Kwestie van consequent namen te gebruiken

moet hier ook geen ...Merivobox voorzien worden (zelfs als die gelijk is aan de Legrabox)? Kwestie van consequent namen te gebruiken

groter dan wat? (550, dat zie ik wel door naar de code te kijken, maar als je een comment toevoegt doe je dat om niet naar de code te moeten kijken http://subversion02.vanhoecke.be/static/ogdo0b/2s...

groter dan wat? (550, dat zie ik wel door naar de code te kijken, maar als je een comment toevoegt doe je dat om niet naar de code te moeten kijken ) (meerdere keren)

'> ?? gebruik beter <= (meerdere keren)

'> ?? gebruik beter <= (meerdere keren)

detail, maar in principe gebruiken we die domeincontext niet meer (zie WV en MVI voor details) en kan die gewoon genewed worden. (staat op meerdere plaatsen, ga dat niet overal schrijven http://sub...

detail, maar in principe gebruiken we die domeincontext niet meer (zie WV en MVI voor details) en kan die gewoon genewed worden. (staat op meerdere plaatsen, ga dat niet overal schrijven )