Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
[ICT-3839] Aanpassen stickers onderdelendozen -> Ook batchnummer toevoegen
[ICT-3839] Aanpassen stickers onderdelendozen -> Ook batchnummer toevoegen
[ICT-3839] Aanpassen stickers onderdelendozen -> Ook batchnummer toevoegen

- Property toegevoegd aan etiket-klasse

- Property ingevuld in geval van "onderdelen" (anders gewoon leeg)

    • -1
    • +1
    ./impl/OnderdelenEtiketPrinterBijLade.cls.xml
    • -1
    • +2
    ./impl/OnderdelenEtiketPrinterBase.cls.xml
  1. … 1 more file in changeset.
[XS-19] 24. Add logo on labels on packaging & delivery notes

- In klasse OnderdelenEtiketPrinter: OnderdelendoosLogoBepaler vervangen door LabelLogoBepaler

    • -1
    • +1
    ./impl/OnderdelenEtiketPrinterGekoppeldeIK.cls.xml
    • -1
    • +1
    ./impl/OnderdelenEtiketPrinterBijLade.cls.xml
    • -4
    • +4
    ./impl/OnderdelenEtiketPrinterBase.cls.xml
  1. … 1 more file in changeset.
[rvTVE] PTL - Altijd onderdelen etiket (behalve als er geen onderdelen zijn)

- Er moet niet 2x geprint worden ;)

    • -21
    • +24
    ./impl/OnderdelenEtiketPrinterBijLade.cls.xml
[rvTVE] PTL - Altijd onderdelen etiket (behalve als er geen onderdelen zijn)

- Enkel printen naar bartender als e rook effectief iets te printen valt

    • -1
    • +6
    ./impl/OnderdelenEtiketPrinterBijLade.cls.xml
[rvTVE] PTL - Altijd onderdelen etiket (behalve als er geen onderdelen zijn)

- Ook etiketten genereren wanneer er geen onderdelendoos nodig is maar wel onderdelenpicking. Dit om een sticker te bekomen voor toeleveringen zonder onderdelendoos voor de pick to light (PTL). Wanneeer het bestandje voor PTl wordt aangemaakt zal in een global worden bijgehouden dat voor die toelevering ook minstens 1 sticker nodig is.

    • -2
    • +30
    ./impl/OnderdelenEtiketPrinterBijLade.cls.xml
  1. … 1 more file in changeset.
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-1797] [rvWV] Wijzigen etiketten bestellingen MP Van Raemdonck

-Review opmerkingen verwerkt

-$Get("global") =1 gebruikt ipv $data()

-Extra testen toegevoegd voor klant Van Hoecke en een klant die deze global niet heeft

    • -1
    • +1
    ./impl/OnderdelendoosLogoBepaler.cls.xml
  1. … 2 more files in changeset.
Ja dat was iets van Jo, maar is nu ook rood, is aangepast in revision 63265 door Jo.

Ja dat was iets van Jo, maar is nu ook rood, is aangepast in revision 63265 door Jo.

TIP : Misschien nog iets meer uitleg in de commit message : waarom tests in commentaar gezet? Is dit tijdelijk? Moeten de code eerst terug gefixed worden? Moeten ze later alsnog aangepast worden? M...

TIP :
Misschien nog iets meer uitleg in de commit message : waarom tests in commentaar gezet? Is dit tijdelijk? Moeten de code eerst terug gefixed worden? Moeten ze later alsnog aangepast worden? Mogen ze weg, maar twijfel je nog?
Feel free om dit soort zaken in de commit message toe te voegen, want nu lees ik enkel dat er iets veranderd is in de LogoBepaler.

Nog 2 testmethods toevoegen : *voor klantnummer 4682 *voor klantummer "UT_klantummerNietinIPCom" --> result is logoVH (alé, dat verwacht ik toch http://subversion02.vanhoecke.be/static/ogdo0b/2st...

Nog 2 testmethods toevoegen :

  • voor klantnummer 4682
  • voor klantummer "UT_klantummerNietinIPCom" --> result is logoVH (alé, dat verwacht ik toch )
$D() of $Data() geeft aan of de subnode aanwezig is in de global, ongeacht de waarde die er ingevuld is. Dus het lijkt me veiliger om gebruik te maken van : $Get(...) = 1 want stel dat iemand ooit ...

$D() of $Data() geeft aan of de subnode aanwezig is in de global, ongeacht de waarde die er ingevuld is.
Dus het lijkt me veiliger om gebruik te maken van : $Get(...) = 1
want stel dat iemand ooit de waarde in de global wijzigt van 1 naar 0 , dan zal $Data() nog steeds true zijn.

Eigenaardige if-clause hierboven : Klant is NIET haefeleBE en toch projectSetting voor HaefeleBEnaarEindklant checken. is dat niet heel raar?

Eigenaardige if-clause hierboven :
Klant is NIET haefeleBE en toch projectSetting voor HaefeleBEnaarEindklant checken.
is dat niet heel raar?