Put set_positions task in Rakefile. Fix #15

scripts/setpositions.rb deleted as now redundant. New rake task works for all elections (full/byelections).

Rakefile

@@ -0,0 +1,35 @@
+require './models'
+
+desc "Set the candidate positions for an election (will prompt you for election ID)."
+task :set_positions do
+    Election.all.each do |e|
+      puts "%d %s %s" % [ e.id, e.d, e.body.name ]
+    end
+
+    puts "Which election ID?"
+
+    STDOUT.flush
+    id = STDIN.gets.chomp.to_i
+    
+    if e = Election.get(id)
+      puts "%d %s %s" % [ e.id, e.d,e.body.name ]
+      e.polls.each do |poll|
+        poll.set_positions
+        
+        separator = '-' * poll.district.name.size
+        puts
+        puts separator
+        puts poll.district.name
+        puts separator
+        
+        ccys = Candidacy.all(:conditions => { :district_id => poll.district_id, :election_id => e.id }, :order => [:votes.desc])
+
+        ccys.each do |cand|
+          puts "%-25s %-25s %-40s %5d %2d %s" % [ cand.candidate.surname, cand.candidate.forenames, cand.party.name, cand.votes, cand.position, cand.seats == 1 ? 'elected' : '' ]
+        end
+      end
+    else
+      puts "Election ID #{id} not found."
+      return 1
+    end
+end

scripts/setpositions.rb

@@ -1,42 +0,0 @@
-require_relative '../models'
-require 'pp'
-
-# Set position and elected for each candidacy
-# WARNING - This will only work in full council elections not byelections (as the number of seats being elected in that eleciton is probably less than the total number of seats in the ward)
-# This will break all the byelection data currently in the database
-
-election = Election.first(:d => ARGV.shift)
-
-unless election
-  puts "Election not found. Usage example: $ %s 2006-05-04" % __FILE__
-  exit 1
-end
-
-puts "Setting candidacy positions for %s %s %s" % [ election.body.name, election.kind, election.d.to_s ]
-if election.candidacies.size < 30
-  puts "This script only works for full council elections. Your election looks like a byelection, so quitting."
-  exit 1
-end
-puts "%d candidacies in this election" % election.candidacies.size
-
-election.body.districts.each do |district|
-  cands = Candidacy.all(:conditions => { :district_id => district.id, :election_id => election.id }, :order => [:votes.desc])
-  # pp cands
-  puts
-  pos = 1
-  cands.each do |cand|
-    if pos == 1
-      puts "-" * 62
-      puts district.name
-      puts "-" * 62
-    end
-
-    # If this candidate's position was in the top district.seats positions, they won a seat
-    pos <= district.seats ? seats = 1 : seats = 0
-    puts "%-25s %-25s %5d %2d %d" % [ cand.candidate.surname, cand.candidate.forenames, cand.votes, pos, seats ]
-    cand.position = pos
-    cand.seats = seats
-    cand.save
-    pos += 1
-  end
-end