Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
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 )

[ICT-2026] HX: Planning: MVX: Probox overdoosetiketten:
[ICT-2026] HX: Planning: MVX: Probox overdoosetiketten:
[ICT-2026] HX: Planning: MVX: Probox overdoosetiketten:

- Uitbreiding met VerwerkMVXKenmerken => werpt NotImplementedException => geen MVX in Unishop of VHintra, anderzijds wel de IngegevenKenmerkenVisitor (waarop een VerwerkMVXKenmerken zit) als super van de base-klasse

  1. … 1 more file in changeset.
[ICT750] TAX: FPV1: Opkuis code flatpack Procom:
[ICT750] TAX: FPV1: Opkuis code flatpack Procom:
[ICT750] TAX: FPV1: Opkuis code flatpack Procom:

- Code

  1. … 45 more files in changeset.
[ICT218] [Kanban ICT490] - TAX: FPV1: Opkuis code flatpack V1:
[ICT218] [Kanban ICT490] - TAX: FPV1: Opkuis code flatpack V1:
[ICT218] [Kanban ICT490] - TAX: FPV1: Opkuis code flatpack V1:

- Aanpassing naamgeving => implementatie zelf wijzigt niet tussen oude en nieuwe FlatpackV1

Bijna juist. ik had zelf ook bijna dezelfde redeneringsfout gemaakt :-P LBX spoelbak zijn 2 smalle, losse lades (ca 158mm breed) die de klant aan zijn ladefront en corpus moet monteren. Dus voor V...

Bijna juist.
ik had zelf ook bijna dezelfde redeneringsfout gemaakt :-P

LBX spoelbak zijn 2 smalle, losse lades (ca 158mm breed) die de klant aan zijn ladefront en corpus moet monteren. Dus voor VH (prijs / productie / verpakking / ... ) maakt het ni uit hoe groot de "corpusbreedte" is.

Maar het geeft aan dat je voldoende kritisch reviewt

Hiermee negeer je de andere beperkingen, bv. spoelbak is echter ook mogelijk in vhoss verpakking, toch blijft de toegelaten breedte 1430. Volgens mij moet enkel de controle op rugwandmateriaal uitg...

Hiermee negeer je de andere beperkingen, bv. spoelbak is echter ook mogelijk in vhoss verpakking, toch blijft de toegelaten breedte 1430.
Volgens mij moet enkel de controle op rugwandmateriaal uitgebreid worden.

[VRB472] LBX Spoelbaklade: Maximum Breedte mag groter dan 1168mm
[VRB472] LBX Spoelbaklade: Maximum Breedte mag groter dan 1168mm
[VRB472] LBX Spoelbaklade: Maximum Breedte mag groter dan 1168mm
  1. … 1 more file in changeset.
[UST2006] TAX FPC v1: Corpusbreedte beperken

- Corpusbreedte beperken zoals bij Flatpack V1 (1168 mm). + extra testjes rond maximums

  1. … 3 more files in changeset.