Replaced Rails loop with JavaScript map to handle instructor merges

[jasonkhoe]

Followed advice from @seshness

[seshness]

🉑 👍 🌞

[richardxia]

Nitpicks about the indentation and lane breaks here:

1. success should be lined up with url.
2. Why is autocomplete on a separate line? It fits within 80 characters with the previous line.
3. result should be indented more.
4. Having three nested anonymous functions in the success handler is pretty confusing to read. I would create a new named function and just reference it here.
5. There is inconsistent spacing around function arguments. I would personally prefer no spaces before or after parentheses and a space after every comma.

[richardxia]

I realize you weren't responsible for most of the things I pointed out, but I figured we might as well clean it up while we're at it. Everything else LGTM.

[seshness]

Yeah...maybe this wasn't the best way of curing the problem. We might as well jQuery all the way.
Instead of using the ID selectors, a class applied to the inputs and the hidden fields might be a better idea. Say the fields currently id'd as iboxN had the class instructor-name. You could then write:

$('.instructor-name').each(function() {
  // ... your code here, where this refers to each of the elements with the class 'instructor-name'
});