Fixing controllers and views with validations

app/controllers/mats_controller.rb

@@ -18,14 +18,13 @@ class MatsController < ApplicationController
   def new
     @mat = Mat.new
     if params[:tournament]
-      @tournament_field = params[:tournament]
       @tournament = Tournament.find(params[:tournament])
     end
   end
 
   # GET /mats/1/edit
   def edit
-    @tournament_field = @mat.tournament_id
+    @tournament = Tournament.find(@mat.tournament_id)
   end
 
   # POST /mats
@@ -82,17 +81,17 @@ class MatsController < ApplicationController
     end
 
     def check_access
-	if params[:tournament]
+      if params[:tournament]
-	   @tournament = Tournament.find(params[:tournament])
+    	   @tournament = Tournament.find(params[:tournament])
-	elsif params[:mat]
+    	elsif params[:mat]
-	   @mat = Mat.new(mat_params) 
+    	   @mat = Mat.new(mat_params)
-	   @tournament = Tournament.find(@mat.tournament_id)
+    	   @tournament = Tournament.find(@mat.tournament_id)
-	elsif @mat
+    	elsif @mat
-	   @tournament = @mat.tournament
+    	   @tournament = @mat.tournament
-	end
+    	end
-	if current_user != @tournament.user
+    	if current_user != @tournament.user
-	  redirect_to '/static_pages/not_allowed'
+    	  redirect_to '/static_pages/not_allowed'
-	end
+    	end
     end
     
     

app/controllers/schools_controller.rb

@@ -14,14 +14,12 @@ class SchoolsController < ApplicationController
   def new
     @school = School.new
     if params[:tournament]
-      @tournament_field = params[:tournament]
       @tournament = Tournament.find(params[:tournament])
     end
   end
 
   # GET /schools/1/edit
   def edit
-    @tournament_field = @school.tournament_id
     @tournament = @school.tournament
   end
 
@@ -79,17 +77,17 @@ class SchoolsController < ApplicationController
     end
 
     def check_access
-	if params[:tournament]
+    	if params[:tournament]
-	   @tournament = Tournament.find(params[:tournament])
+    	   @tournament = Tournament.find(params[:tournament])
-	elsif params[:school]
+    	elsif params[:school]
-	   @school = School.new(school_params)
+    	   @school = School.new(school_params)
-	   @tournament = Tournament.find(@school.tournament_id)
+    	   @tournament = Tournament.find(@school.tournament_id)
-	elsif @school
+    	elsif @school
-	   @tournament = @school.tournament
+    	   @tournament = @school.tournament
-	end
+    	end
-	if current_user != @tournament.user
+    	if current_user != @tournament.user
-	  redirect_to '/static_pages/not_allowed'
+    	  redirect_to '/static_pages/not_allowed'
-	end
+    	end
     end
 
 end

app/controllers/weights_controller.rb

@@ -18,14 +18,12 @@ class WeightsController < ApplicationController
   def new
     @weight = Weight.new
     if params[:tournament]
-      @tournament_field = params[:tournament]
       @tournament = Tournament.find(params[:tournament])
     end
   end
 
   # GET /weights/1/edit
   def edit
-    @tournament_field = @weight.tournament_id
     @tournament = @weight.tournament
     @mats = @tournament.mats
   end

app/controllers/wrestlers_controller.rb

@@ -14,7 +14,6 @@ class WrestlersController < ApplicationController
   def new
     @wrestler = Wrestler.new
     if params[:school]
-      @school_field = params[:school]
       @school = School.find(params[:school])
     end
     if @school
@@ -28,7 +27,6 @@ class WrestlersController < ApplicationController
 
   # GET /wrestlers/1/edit
   def edit
-    @school_field = @wrestler.school_id
     @school = @wrestler.school
     @tournament = @wrestler.tournament
     @weight = @wrestler.weight
@@ -40,6 +38,7 @@ class WrestlersController < ApplicationController
   def create
     @wrestler = Wrestler.new(wrestler_params)
     @school = School.find(wrestler_params[:school_id])
+    @weights = @school.tournament.weights
     respond_to do |format|
       if @wrestler.save
         format.html { redirect_to @school, notice: 'Wrestler was successfully created.' }

app/views/mats/_form.html.erb

@@ -16,8 +16,8 @@
     <%= f.text_field :name %>
   </div>
 
-<% if @tournament_field %>
+<% if @tournament %>
-      <%= f.hidden_field :tournament_id, :value => @tournament_field %>
+      <%= f.hidden_field :tournament_id, :value => @tournament.id %>
 <% end %>
 <br>
   <div class="actions">

app/views/schools/_form.html.erb

@@ -17,7 +17,7 @@
   </div>
   
 <% if @school %>
-      <%= f.hidden_field :tournament_id, :value => @tournament_field %>
+      <%= f.hidden_field :tournament_id, :value => @tournament.id %>
   <% end %>
 <br>
 

app/views/weights/_form.html.erb

@@ -17,7 +17,7 @@
   </div>
 
   <% if @weight %>
-      <%= f.hidden_field :tournament_id, :value => @tournament_field %>
+      <%= f.hidden_field :tournament_id, :value => @tournament.id %>
   <% end %>
   <br>
   <div class="actions">

app/views/wrestlers/_form.html.erb

@@ -18,7 +18,7 @@
   </div>
 
   <% if @school %>
-      <%= f.hidden_field :school_id, :value => @school_field %>
+      <%= f.hidden_field :school_id, :value => @school.id %>
   <% else %>
       <div class="field">
         <%= f.label 'School' %><br>