Activity CacheAdminA_trunk2010/BL/Flow/Levering/LabelLogoBepaler.cls.xml

12:51 pm

Joannes Lemmens

More
CR-2863
started review

[ICT-5448] BCZ Distributeurs aanmaken:

8:20 am

Joannes Lemmens

More
74485
changed 3 files
root: CR-2863

[ICT-5448] BCZ Distributeurs aanmaken:

- LabelLogoBepaler uitgebreid voor andere Blum personalities

- Deployklasse om logo's te mappen

- Unit test uitgebreid

- -2
- +18
./LabelLogoBepaler.cls.xml
- History
- Source
- Diff
./LabelLogoBepaler.cls.xml
… 2 more files in changeset.

12:46 pm

Tom Vermeulen

More
Re: CR-2642
commented on review

Dag Joannes, Moet deze wijziging ook niet in de OnderdelendoosLogoBepaler voorzien worden? Die wordt gebruikt voor de onderdelendozen (duh), maar ook voor de proboxinhoudetiketten. Je kan je ook d...

Dag Joannes,

Moet deze wijziging ook niet in de OnderdelendoosLogoBepaler voorzien worden? Die wordt gebruikt voor de onderdelendozen (duh), maar ook voor de proboxinhoudetiketten. Je kan je ook de vraag stellen waarom we alsnog die OnderdelendoosLogoBepaler gebruiken (die is 99% hetzelfde als de LabelLogoBepaler). Kan heel die reutemeteut niet weggesmeten worden en geredirect langs de LabelLogoBepaler?

1:32 pm

Tom Vermeulen

More
CR-2642
started review

[XS-19] 24. Add logo on labels on packaging & delivery notes

2:16 pm

Joannes Lemmens

More
71662
changed 2 files
root: CR-2642

[XS-19] 24. Add logo on labels on packaging & delivery notes

- LabelLogoBepaler aangepast: werkt nu met oorsprongketting (of KlantNummer als die ketting niet nuttig is)

- -8
- +45
./LabelLogoBepaler.cls.xml
- History
- Source
- Diff
./LabelLogoBepaler.cls.xml
… 1 more file in changeset.

10:09 am

Wim Vermeulen

More
CR-2154
Closed

12:37 pm

Wim Vermeulen

More
CR-2154
finished reviewing

8:16 pm

SamV

More
63406
changed 3 files
root: CR-2154

[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
./LabelLogoBepaler.cls.xml
- History
- Source
- Diff
./LabelLogoBepaler.cls.xml
… 2 more files in changeset.

8:08 pm

Sam Van Hoey

More
Re: CR-2154
replied to Wim Vermeulen

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

6:55 pm

Wim Vermeulen

More
Re: CR-2154
commented on review

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.

6:48 pm

Wim Vermeulen

More
Re: CR-2154
commented on review

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 )

6:42 pm

Wim Vermeulen

More
Re: CR-2154
commented on review

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

6:34 pm

Wim Vermeulen

More
Re: CR-2154
commented on review

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?

8:53 am

Wim Vermeulen

More
Re: CR-2154
commented on review

Bij klant OptimaPack zie ik hetvolgende : *vorige versie geeft pxLabelKlant.Logo = "" *jou nieuwe versie (via LabelLogoBepaler + Data in ^IPCom) pxLabelKlant.Logo = "logo\Logo Optima 6279.svg" ...

Bij klant OptimaPack zie ik hetvolgende :

vorige versie geeft pxLabelKlant.Logo = ""
jou nieuwe versie (via LabelLogoBepaler + Data in ^IPCom) pxLabelKlant.Logo = "logo\Logo Optima 6279.svg"

Dat lijkt me niet hetzelfde eindresultaat te zullen geven. Misschien is dat oké, maar het is alvast mijn taak als kritische reviewer om dit op te merken.

10:31 am