Support multiple vacancies

+++ b/election_fptp.module
@@ -110,18 +141,26 @@ function election_fptp_preprocess_election_vote_confirm(&$variables) {
+    $insert_id = db_insert('election_vote')

Could you use a single multiple-value insert query instead? See this answer: http://drupal.stackexchange.com/a/53319

1. +++ b/election_fptp.module
   @@ -35,9 +35,15 @@ function election_fptp_vote_form(&$form, &$form_state) {
   +  if ((int) $post->vacancy_count === 1) {
   

   Casting a property to an integer and then comparing it to 1 has the same effect as using the == operator.

2. +++ b/election_fptp.module
   @@ -75,11 +91,30 @@ function election_fptp_vote_form(&$form, &$form_state) {
   +  $vacancy_count = (int) $form_state['post']->vacancy_count;
   

   It doesn't seem right to me to cast the type of vacancy_count, you have this in 3 places already - is there a particular reason?

3. +++ b/election_fptp.module
   @@ -91,13 +126,15 @@ function election_fptp_preprocess_election_vote_confirm(&$variables) {
   +      '#title' => (count($voted_for) > 1) ? t('Your chosen candidates:') : t('Your chosen candidate:'),
   

   This should use format_plural().

Just discovered an issue:

In the foreach loop in election_fptp_preprocess_election_vote_confirm(), the $candidate_id is used as the index for the $form_state['candidates'] array, but that array uses incremental indices starting at 0. So the names never appear on the vote confirmation page because candidate objects aren't getting passed into election_candidate_get_name() unless the $candidate_id matches one of the indices.