We had to do a vote with many options, and it became apparent that the admin "Internal values, for debugging purposes only" might be useful to people not admins. So I rewrote the table so that the layout was bette, and used theme_table instead of a custom table.

This patch reflects this effort.

CommentFileSizeAuthor
ranking.inc.patch1.58 KBWetherMan

Comments

anarcat’s picture

anarcat commented

I've imported a modified version of the patch in decisions, in commit #46005. I've protected it by "inspect all votes" instead of "administer decisions", which should also answer that request. Nice patch!

ChrisKennedy’s picture

ChrisKennedy commented
Priority: Minor » Normal
Status: Needs review » Fixed

Great idea :) Man, you should have shown it in action at http://theforvm.org/polls/what-should-our-slogan-be earlier! I stumbled upon that tonight while looking to see if anyone was using Advanced Poll, and it was so beautiful that I had to get the code working tonight. IRV is finally not ugly as sin - it actually makes me want to use it on occasion.

The reason I didn't commit this earlier is that you were using an old version that didn't cache the results. It took a bit of effort to cache each round's data, but it should work. However, I haven't tested it much so let me know if you find any bugs.

http://drupal.org/cvs?commit=49361 (5.0)
http://drupal.org/cvs?commit=49363 (4.7)

Anonymous’s picture

(not verified) commented
Status: Fixed » Closed (fixed)