extracted wrestlers_for_pool method

removed unnecessary select

app/models/pool.rb

@@ -16,7 +16,7 @@ class Pool
 
 	def roundRobin
 		matches = []
-		wrestlers = @weight.wrestlers.select{|w| w.generatePoolNumber == @pool}.to_a
+		wrestlers = @weight.wrestlers_for_pool(@pool)
 		poolMatches = RoundRobinTournament.schedule(wrestlers).reverse
 		poolMatches.each_with_index do |b, index|
 			round = index + 1

app/models/tournament.rb

@@ -28,10 +28,10 @@ class Tournament < ActiveRecord::Base
 
 	def upcomingMatches
 		if matches.nil?
-			return matches
+			return nil
 		else
 			generateMatchups
-			return matches
+			matches
 		end
 	end
 

app/models/tournamentmatchgen.rb

@@ -28,8 +28,7 @@ class Tournamentmatchgen
   def buildTournamentWeights
     @tournament.weights.sort_by{|x|[x.max]}.each do |weight|
       matches = Pool.new(weight).generatePools()
-      weight_matches = matches.select{|m| m.weight_id == weight.id }
-      last_match = weight_matches.sort_by{|m| m.round}.last
+      last_match = matches.sort_by{|m| m.round}.last
       highest_round = last_match.round
       @matches += Poolbracket.new.generateBracketMatches(matches, weight, highest_round)
     end

app/models/weight.rb

@@ -8,6 +8,10 @@ class Weight < ActiveRecord::Base
 		self.tournament.destroyAllMatches
 	end
 
+	def wrestlers_for_pool(pool)
+		wrestlers.select{|w| w.generatePoolNumber == pool}.to_a
+	end
+
 	def pools
 		@wrestlers = self.wrestlers
 		if @wrestlers.size <= 6