Thatoneguy123/SVDP — pull request #6 by bcat (2012-05-16 00:09:25+00:00)
Title: Contact list anchors

Your changes to fix the member contact list bug look good. I went ahead and changed a couple things, though. First off, I fixed up the HTML, removing the `a` tags and just putting `id` attributes on the table rows themselves. (Using `` is deprecated, and the new way is supported in all modern and almost all not-so-modern browsers.) Second, there was a bug in the old code---the `last` anchor logic didn't actually prevent duplicate anchors from being generated. Unfortunately, there's no good way to pass output out of a partial to the partial's caller. The fix is pretty simple; however, it required moving a bit more logic into the controllers. If you aren't cool with this, just let me know and we can come up with another approach.