diff --git a/app/controllers/kalibro_configurations_controller.rb b/app/controllers/kalibro_configurations_controller.rb index b45dfa7..e3fbdfd 100644 --- a/app/controllers/kalibro_configurations_controller.rb +++ b/app/controllers/kalibro_configurations_controller.rb @@ -3,6 +3,7 @@ include OwnershipAuthentication class KalibroConfigurationsController < ApplicationController before_action :authenticate_user!, except: [:index, :show] before_action :kalibro_configuration_owner?, only: [:edit, :update, :destroy] + before_action :set_kalibro_configuration, only: [:show, :edit, :update, :destroy] # GET /kalibro_configurations/new def new @@ -27,7 +28,6 @@ class KalibroConfigurationsController < ApplicationController # GET /kalibro_configurations/1 # GET /kalibro_configurations/1.json def show - set_kalibro_configuration Rails.cache.fetch("#{@kalibro_configuration.id}_metric_configurations") do @kalibro_configuration.metric_configurations end @@ -35,15 +35,10 @@ class KalibroConfigurationsController < ApplicationController # GET /kalibro_configurations/1/edit # GET /kalibro_configurations/1/edit.json - def edit - set_kalibro_configuration - @attributes = @kalibro_configuration.attributes - end + def edit; end def update - set_kalibro_configuration if @kalibro_configuration.update(kalibro_configuration_params) - @kalibro_configuration.attributes.update(public: attributes_params) redirect_to(kalibro_configuration_path(@kalibro_configuration.id)) else render "edit" @@ -53,8 +48,6 @@ class KalibroConfigurationsController < ApplicationController # DELETE /kalibro_configurations/1 # DELETE /kalibro_configurations/1.json def destroy - set_kalibro_configuration - @kalibro_configuration.destroy respond_to do |format| @@ -77,14 +70,10 @@ class KalibroConfigurationsController < ApplicationController params[:kalibro_configuration] end - def attributes_params - params[:attributes][:public] == "1" - end - # Extracted code from create action def create_and_redir(format) if @kalibro_configuration.save - current_user.kalibro_configuration_attributes.create(kalibro_configuration_id: @kalibro_configuration.id, public: attributes_params) + current_user.kalibro_configuration_attributes.create(kalibro_configuration_id: @kalibro_configuration.id) format.html { redirect_to kalibro_configuration_path(@kalibro_configuration.id), notice: t('successfully_created', :record => @kalibro_configuration.model_name.human) } format.json { render action: 'show', status: :created, location: @kalibro_configuration } diff --git a/app/controllers/reading_groups_controller.rb b/app/controllers/reading_groups_controller.rb index ecc7252..231d829 100644 --- a/app/controllers/reading_groups_controller.rb +++ b/app/controllers/reading_groups_controller.rb @@ -8,7 +8,6 @@ class ReadingGroupsController < ApplicationController # GET /reading_groups/new def new @reading_group = ReadingGroup.new - @attributes = @reading_group.attributes end # GET /reading_groups @@ -32,13 +31,10 @@ class ReadingGroupsController < ApplicationController # GET /reading_groups/1/edit # GET /reading_groups/1/edit.json - def edit - @attributes = @reading_group.attributes - end + def edit; end def update if @reading_group.update(reading_group_params) - @reading_group.attributes.update(public: attributes_params) redirect_to(reading_group_path(@reading_group.id)) else render "edit" @@ -67,15 +63,11 @@ class ReadingGroupsController < ApplicationController params[:reading_group][:name].strip! params[:reading_group] end - - def attributes_params - params[:attributes][:public] == "1" - end # Extracted code from create action def create_and_redir(format) if @reading_group.save - current_user.reading_group_attributes.create(reading_group_id: @reading_group.id, public: attributes_params) + current_user.reading_group_attributes.create(reading_group_id: @reading_group.id) format.html { redirect_to reading_group_path(@reading_group.id), notice: t('successfully_created', :record => t(@reading_group.class)) } format.json { render action: 'show', status: :created, location: @reading_group } diff --git a/app/views/kalibro_configurations/_form.html.erb b/app/views/kalibro_configurations/_form.html.erb index 0571b58..86772d7 100644 --- a/app/views/kalibro_configurations/_form.html.erb +++ b/app/views/kalibro_configurations/_form.html.erb @@ -17,14 +17,6 @@ <%= f.text_area :description, class: 'text-area form-control' %> - -