This is a list of all comments for CR-2154.

Review Summary: No summary


----------------------------------------
File: BL/Flow/Levering/Label.cls.xml
Revision Comment by Wim Vermeulen on 15 October 2020, 17:43 
http://subversion02:8060/cru/CR-2154#c3513

  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.

Revision Comment by Wim Vermeulen on 15 January 2021, 18:31 defect
http://subversion02:8060/cru/CR-2154#c3590

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

  Reply by Sam Van Hoey on 15 January 2021, 20:08
  > Ja dat was iets van Jo, maar is nu ook rood, is aangepast in
  > revision 63265 door Jo.

----------------------------------------
File: BL/PPS/common/impl/OnderdelendoosLogoBepaler.cls.xml
Revision Comment by Wim Vermeulen on 15 January 2021, 18:38 
http://subversion02:8060/cru/CR-2154#c3591

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

----------------------------------------
File: vhUnitTest/BL/Flow/Levering/LabelLogoBepaler/GeefLogo/Test.cls.xml
Revision Comment by Wim Vermeulen on 15 January 2021, 18:46 defect
http://subversion02:8060/cru/CR-2154#c3592

  Nog 2 testmethods toevoegen :
   - voor klantnummer 4682
   - voor klantummer "UT_klantummerNietinIPCom"  --> result is
  logoVH (alé, dat verwacht ik toch :-) )

----------------------------------------
File: vhUnitTest/BL/PPS/TBX/DocsV1/ProboxInhoudEtiketGenerator/GenereerProboxInhoudEtiket/Test.cls.xml
Revision Comment by Wim Vermeulen on 15 January 2021, 18:50 
http://subversion02:8060/cru/CR-2154#c3593

  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.


---

ID: CR-2154 http://subversion02:8060/cru/CR-2154

Title: [ICT-1797] Wijzigen etiketten bestellingen MP Van Raemdonck
Statement of Objectives:


State: Closed

Summary: 

Author: Sam Van Hoey
Moderator: Sam Van Hoey
Reviewers: (0 active, 1 completed*)
     Wim Vermeulen   (*)