Fix scoping bug

We recently ran into a bug with this project in the causes app where
all calls to the Facebook API, which go through method proxy, were
broken for all versions of Internet Explorer. The behavior was
inconsistent in other browsers.

It turns out that this was a scoping problem. The `item` variable
happened to be referencing a global object, causing the unexpected
behavior. The fix for this is to declare the `item` variable in the
local scope.

Change-Id: Ie972fe2e0907182299b087057f406e5b8e718259
Reviewed-on: http://gerrit.causes.com/37016
Tested-by: Joe Lencioni <joe.lencioni@causes.com>
Reviewed-by: Hao Su <hao.su@causes.com>
Alphabetize .jshintrc

This was copypasta'd from another project. Before making adjustments, I
decided to give it some order. No other changes were made in this
commit.

Change-Id: I0c5d9bc9c5ce18e96cf0017ac02d3ee9abd87b34
Reviewed-on: http://gerrit.causes.com/37021
Tested-by: Joe Lencioni <joe.lencioni@causes.com>
Reviewed-by: Hao Su <hao.su@causes.com>