Merge from jdong

Fix more DeprecationWarnings

Merge head

Merge from jdong, XMLRPC changes and testcases

Add several testcases for casting quickvotes with invalid information

* Add a few testcases for checking for nonexistent things
* Change helpers to be a bit more streamlined in the way they handle assertions failures vs unexpected exceptions.

Improve consistency of XMLRPC error reporting. Now, all error conditions will raise an exception. Before, there was an inconsistent mix between raising exceptions and returning nil objects. Nil objects are much less desirable because the client must explicitly check for errors and some errors might silently pass through and produce incorrect results

Add assert_cast_quickvote_succeeds/fails helper, and changed the fails helpers to instead check that the _succeeds  helper doesn't really succeed.

Also expose voter's Session ID

Added Sparklines controller and dependency, see README. Created method and table for margins of victory (i.e won by how much) and ties, in rubyvote, has already been svn'd.

Merge from jdong; all testcases should succeed at this point, and all deprecations should be gone

Fix validation of candidate names

Fixed all triggerable DeprecationWarnings. BE ON LOOKOUT FOR REGRESSIONS.

Add testcase for checking validity of vote results

Merged changes from jdong's branch into HEAD.

* Refactor checking proper quickvote creation to helper assert_create_quickvote_succeeds
* Remove a crufty assertion from XML checker -- it is redundant
* Add testcase for a description with binary characters

* Add one more assertion in testcase for dupe candidates
* Remove test_create_quickvote_description_htmlescape. As I thought earlier, I actually want the DB to properly store HTML elements. It's the job of the view to make sure it doesn't clobber itself displaying the element.
  - Change the test instead to a test for proper XML escaping. An appended </string> element should be stored properly and not trip hell

Correct a bunch of the testcases. Poor copy-and-paste led to description => nil where it shouldn't have been. Now a bunch more testcases FAIL. Yay me.

* Refactor testcase code -- create_quickvote_fails assertion -- was using same block of code repetivitely

Add more testcases for improper data
  * Passing in a nil ElectionStruct to quickvotes
  * Passing in only nil {name, description, candidates}
  * Passing in whitespace for name, description, candidates
  * Passing in duplicate candidates
  * Passing in dirty HTML in description field
    - IMO this may not be a "bug" -- we should rather assert that the view will not be injected with HTML. It may be desirable (a feature) to store HTML-ish text in the database.

* Validate that quickvote.name is not nil

Caught error from testcase! Quickvote creation errors weren't being properly returned.

Added more testcases:
 * Cast large # of votes
 * Create large # of quickvotes
 * Test creation of quickvote with a bad name

Add some testcases:
  * Create a new quickvote
  * Retrieve a quickvote
  * Casting a quickvote

Add a way to get the votes list from a quickvote votes XMLRPC call

Started implementing XMLRPC test cases

Removed crufty testcases that fail. Will rewrite them

Fix a host of testcase failure issues. Still not done, but this is the end of syntactical changes. (Rest of fixes will be behavioral)

Merged from John.

Import new svn rubyvote (revno 43)

Throw an ArgumentError if invalid election name is passed to casting

XMLRPC Vote Casting Validation:
  * Can only vote for candidate_id's within the current election
  * Can only rank each candidate once
  * Must rank all candidates

Update TODO list

Add XMLRPC interface for creating new quickvotes

Add XMLRPC API for getting an individual quickvote object; refactor getting all quickvotes with this new method

list_quickvotes XMLRPC returns candidate ID's instead of names

Add XMLRPC call for getting a list of all QuickVotes

Add Google Map of voters
 * Add rails plugin for google maps
 * Add rails plugin for IP to location mapping

Merge Justin

Merged from John's branch, includs a lot of IP lookup libraries.

The defeats lsit partial contains the display logic for the mehtod added to condorcet.rb taht helps print out a list of who defeated who and by how much. I also modified the tab;e in _pref_table.rhtml to use names as the table headers and notcandidate id's. Finally, results.rhtml has been updated to display them.

Add nil-guards to the graphs

Display IP address on unresolvable hosts

Add ability to parse RIPE WHOIS too

* Display DNS instead of IP address
* Convert old DNS field to WHOIS OrganizationName field.
* Btter handle invalid IP's, including proper display of XMLRPC voters

HTML escape description to prevent code injection onto page

Disallow identical candidates to be added to a quickvote. Differently cased identical strings are still allowed. This of course makes Asdf and asdf impossible to tell apart in the vote, which is kind of a bug. We should think of some way of differentiating those

Validate that quickvote name is not an integer -- causes URL clash

Added XMLRPC API to obtain list of voters

Use WHOIS to search up meaningful organizational data about voters. NOTE: Probably destroys performance when there's >= 100 voters.

Also a part of the previous commit

* Tighter validation, closed a number of crashes due to invalid data
* Security: Escape HTML to prevent injection of code onto the form
* Prevent empty candidates from passing validation
* Clearer, non Engrishy error messages on quickvote/create
* Prevent quickvote ident names from clashing with reserved controller actions

Fix crasher when nil is passed to ident_to_quickvote

Merge from jdong

Merge from Justin
Fix runtime errors in RubyVote introduced.

Modified graphs to ahve a 4 color scheme, but it isn't the full palette of selectricity yet
, that will be on next commit. LAso modified several voting methods within RubyVote to have
more useful instance variables and accessors available in the results class. the quickvote
results view has been updated to include a partial for displaying a condorcet vote table, w
hich is strangely appearing below the footer on the results page, it seems to be displaying
 the correct data however.

Display same in hostname and IP when IP lookup fails

Merge jdong

Record "XMLRPC Request" in ipadress field of XML RPC created votes.

Added IPAddr verification of IP Addresses

Record a junk IP address for XMLRPC'ers

Merge jdong

Add some quick info about XMLRPC API

Merge my branch, which properly implements casting quickvotes

Merge head

Merge another cpssd fix

Merge head

Pull in svn Rubyvote again

Merge from head

Import latest RubyVote svn to fix infinite loop bug

Commit partial implementation of casting quickvotes via xmlrpc. WARNING: Currently I think it creates inconsistent objects in the database that'll screw up voting.

XMLRPC: Add translator from candidate ID to names

Merge jdong to pull in new rubyvote

Import latest subversion RubyVote.

Merge plurality fix

Fix broken Plurality vote by removing a type check in RubyVote. TODO: Figure out why it was in there

Merge jdong branch

* Refactor candidate map over to its own API call

Merge from my devel branch

* Changed XMLRPC API to use arrays of ID's to report results
* Add a list of names and ID's to the results struct, for future translation purposes to canonical names

Refactor the ident_to_quickvote method as a classmethod of the model

Merge refactor

Refactored quickvotes result calculator from the controller to the model. TODO: Run testcases to make sure it still works

Merge my local branch, bring back xmlrpc

Introduce XMLRPC API files. It is not currently functional, but it doesn't break anything outside of itself.

Added a TODO list.

Changed quickvotes so that they take an identifier as opposed to just a
votename. This identifier may be either just a votename or an integer id.

fixed the optional/required snafu on the quickvote create page

Fixed choices_positions to use GruffGraff class. Initialize method now had a way of
dealign with multiple data arrays being passed in. Next need to compensate for giving each set of data it's own name, possible a hash with "name" => data as the key, value pairs...

Added the choices position and get_posititons_info to graph controller. Right now, since
the graph graphs a different set of data for each candidate, the data function has to be
called multiple times and I'm not sure how to do it/if it's possible within the class
definition. As such, I've defined everything within the choices_position method for now.
I'll be trying to figure out the kinks next. Overall, the graph is very  confusing and
might not be very useful in the end, but the data it uses is.

Fixed a bug in the graph library.

Fixed minor bug in the scruffy code.

Refactored code to move it to a more object oriented mode and slimmed
things down. Labels and such are still a little broken.

Removed the Gruff plugin from vendor/plugins. It seems to be working from the gem version now, which is v 0.2.8.

Added a new bar graph, that counts how many points the borda system of
voting has asssigned to every candidate, it shows the margins fo victory
and loss clearly. Also had to make modifications to the RubyVote libary.
There was an error in BordaVote that prevented it from working correctly,
which has been fixed. Also added an attr_reader to BordaResult for
ranked_candidates. About to delete the Gruff Plugin v 0.1.2 and replace it
with Gruff gem v 0.2.8.

Removed the two methods from Application controller which were redundant -
 the main version are in the graph controller. Started to add a borda bar
graph results method, when it came to our attention that the borda method
was most likely broken, counting only the most recent vote. Have added an attr_reader :election to Class Electionresults in rubyvote. About to attempt to fix the Borda method.

upgraded to latest version of rubyvote in svn

Updated the "votes_per_day" method in graph controller to work better and
read easier. Also, with Mako's insistence and help, wrote the "votes_per_interval" method (and helpers) in graph controller. The worst part of that
method is when I assign names to the labels_hash, ideas welcome, but it
works for now. Even though the graph uses fixed intervals and is named
"votes_per_interval" it actually shows votes over time (per interval),
so I may rename it in the near future (in fact I would've renamed it for
this commit, but I didn't think till I was writing this statement.)

* Renamed graphs_controller to graph_controller.
* Reviewed and fixed up the graph_controller code.

Added support for ruby-debug to replace breakpointers.

Renamed CSS and layout to move from the old name (HC) to the new name
(Selectricity).

Changed properties to remove unecessary exectables.