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

Review Summary: No summary


----------------------------------------
File: vhConsistency/Checks/TerminalNavigatieTaken.cls.xml
Revision Comment by Tommy Hebb on 26 January 2021, 11:31 defect
http://subversion02:8060/cru/CR-2247#c3617

  Je zit hier met een If/Else met een $$$Not in de If. Hoewel de
  logica correct is, leest dit net iets minder makkelijk dan
  zonder de $$$Not, dus als je de logica in de If en de Else
  omdraait, dan kun je de $$$Not weglaten. 
  Flow-gewijs is het ook corrrecter in dit geval... Als de
  computernaam huppeldepup is, (If...) dan gaat het over data
  waar we niets aan hebben, dus verwijderen we die. (Else...) In
  het andere geval moet het wel gemeld worden.

Revision Comment by Tommy Hebb on 26 January 2021, 11:27 defect
http://subversion02:8060/cru/CR-2247#c3615

  Aangezien het object al eens eerder is geopend en toen in een
  lokale variabele is gestoken (TerminalNavigatieTaak), kun je
  die beter hier gebruiken. Het heeft niet veel nut om de data 2
  keer na elkaar op te halen, het zit dan toch al in memory. Dus:
  
  Do ..VoegMeldingToe("[ Openstaande taak
  ]"_$C(13,10)_TerminalNavigatieTaak.ToString())


---

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

Title: [ICT-1899] [rvTHB] Consistencycheck TerminalNavigatieTaken failed on [CACHE01]
Statement of Objectives:


State: Closed

Summary: 

Author: Jelle Baele
Moderator: Jelle Baele
Reviewers: (0 active, 2 completed*)
     Tom Vermeulen   (*)
     Tommy Hebb   (*)