Fix up a few bugs in the js code.

Review Request #428 - Created Feb. 24, 2017 and submitted

guest8860
Review Board
c94eccd...
la
This change fixes a few issues that SonarQube surfaced:

- In a couple places, we were iterating over object fields without testing
  `hasOwnProperty`.
- The calls to `testCloseDescription` were passing too many arguments.
- We were testing whether `indexOf` returned a value greater than zero, but
  zero is a valid return value.

Reviewed at https://reviews.reviewboard.org/r/8733/

ok

guest8860
guest8860
Review request changed

Status: Closed (submitted)

guest8860
  1. Ship It!
  2. 
      
Loading...