Comment 3 for bug 520632

Revision history for this message
Mike Rylander (mrylander) wrote :

Here are the files and counts within each where we're using for..in. I have not looked at them yet to make sure they're array accesses, but for anyone that wants to, converting to for(;;) should be fairly trivial.

miker@whirly:~/svn/head-ILS$ gack 'for \(\s*(?:var )?i in' --type=js --ignore-dir=chrome --ignore-dir=xul --ignore-dir=acq --ignore-dir=backend --ignore-dir=conify --ignore-dir=booking --ignore-dir=reports -c |grep -v 0$
Open-ILS/web/opac/common/js/opac_utils.js:1
Open-ILS/web/opac/common/js/jscalendar/calendar-setup.js:1
Open-ILS/web/opac/common/js/jscalendar/calendar.js:1
Open-ILS/web/opac/common/js/org_utils.js:4
Open-ILS/web/opac/skin/default/js/rdetail.js:1
Open-ILS/web/opac/skin/default/js/result_common.js:2
Open-ILS/web/js/dojo/MARC/Field.js:2
Open-ILS/web/js/dojo/MARC/Record.js:2
Open-ILS/web/js/dojo/openils/widget/TranslatorPopup.js:1
Open-ILS/web/js/dojo/openils/I18N.js:1
Open-ILS/web/js/dojo/fieldmapper/dojoData.js:9
Open-ILS/web/js/dojo/fieldmapper/OrgUtils.js:7
Open-ILS/web/js/dojo/fieldmapper/hash.js:2
miker@whirly:~/svn/head-ILS$

I'll take the Dojo stuff ... anyone want to look at the opac proper?

--miker