launchpad-reviewers team mailing list archive

[Curtis Hovey <curtis.hovey@xxxxxxxxxxxxx>]

Thank you for extracting the code.

Consider using
    !Y.Lang.isArray(expanders[index])
for
    expanders[index] === undefined
The code needs an array. or just use
   !Y.Lang.isValue(expanders[index])
if you just want to be sure there is something.
-- 
https://code.launchpad.net/~dooferlad/launchpad/upcomingwork-expand-all/+merge/107083
Your team Launchpad code reviewers is subscribed to branch lp:launchpad.