Merge ~twom/launchpad:fix-subscription-js-test into launchpad:master
Proposed by
Tom Wardill
Status: | Merged |
---|---|
Approved by: | Tom Wardill |
Approved revision: | 82740768c74e3691728da7e0dbc33d240fa9deeb |
Merge reported by: | Otto Co-Pilot |
Merged at revision: | not available |
Proposed branch: | ~twom/launchpad:fix-subscription-js-test |
Merge into: | launchpad:master |
Diff against target: |
20 lines (+2/-2) 1 file modified
lib/lp/bugs/javascript/subscription.js (+2/-2) |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Colin Watson (community) | Approve | ||
Review via email: mp+407018@code.launchpad.net |
Commit message
Guard using the iterable, rather than Object
Description of the change
We have an Array here, so we need to use the iterable's hasOwnProperty, rather than Object.
Following the 'Difference between for..of and for..in' section of https:/
To post a comment you must log in.