diff --git a/app/controllers/admin/communication/website/categories_controller.rb b/app/controllers/admin/communication/website/categories_controller.rb index 15e7f5658c3164a2d8c6b350fda738c342afc6b4..57de36eac0d7b6bf1821018ba4419686c91a808a 100644 --- a/app/controllers/admin/communication/website/categories_controller.rb +++ b/app/controllers/admin/communication/website/categories_controller.rb @@ -9,8 +9,8 @@ class Admin::Communication::Website::CategoriesController < Admin::Communication end def reorder - parent_id = params['parentId'].blank? ? nil : params['parentId'] - ids = params['ids'] + parent_id = params[:parentId].blank? ? nil : params[:parentId] + ids = params[:ids] || [] ids.each.with_index do |id, index| category = @website.categories.find(id) category.update( diff --git a/app/controllers/admin/communication/website/menu/items_controller.rb b/app/controllers/admin/communication/website/menu/items_controller.rb index 13dee32be9f07737ec791725ad388ff778d42e2e..eb19f0bbbab5211bdcc0413bde02944dc9e0effe 100644 --- a/app/controllers/admin/communication/website/menu/items_controller.rb +++ b/app/controllers/admin/communication/website/menu/items_controller.rb @@ -3,8 +3,8 @@ class Admin::Communication::Website::Menu::ItemsController < Admin::Communicatio load_and_authorize_resource class: Communication::Website::Menu::Item, through: :menu def reorder - parent_id = params['parentId'].blank? ? nil : params['parentId'] - ids = params['ids'] + parent_id = params[:parentId].blank? ? nil : params[:parentId] + ids = params[:ids] || [] ids.each.with_index do |id, index| category = @menu.items.find(id) category.update( diff --git a/app/controllers/admin/communication/website/pages_controller.rb b/app/controllers/admin/communication/website/pages_controller.rb index 4371f769c2b5cff54d245701f2aa951ca2263801..1bb615d53c9ce3af4c0726d69f67848e0b01b865 100644 --- a/app/controllers/admin/communication/website/pages_controller.rb +++ b/app/controllers/admin/communication/website/pages_controller.rb @@ -8,8 +8,8 @@ class Admin::Communication::Website::PagesController < Admin::Communication::Web end def reorder - parent_id = params['parentId'].blank? ? nil : params['parentId'] - ids = params['ids'] + parent_id = params[:parentId].blank? ? nil : params[:parentId] + ids = params[:ids] || [] ids.each.with_index do |id, index| page = @website.pages.find(id) page.update( diff --git a/app/controllers/admin/education/programs_controller.rb b/app/controllers/admin/education/programs_controller.rb index 916ceae89a8074eb3857d2369611a4b1e14c7201..38d9e145c120db81e105f6969efbe43862c35741 100644 --- a/app/controllers/admin/education/programs_controller.rb +++ b/app/controllers/admin/education/programs_controller.rb @@ -7,8 +7,8 @@ class Admin::Education::ProgramsController < Admin::Education::ApplicationContro end def reorder - parent_id = params['parentId'].blank? ? nil : params['parentId'] - ids = params['ids'] + parent_id = params[:parentId].blank? ? nil : params[:parentId] + ids = params[:ids] || [] website_ids = [] ids.each.with_index do |id, index| program = current_university.education_programs.find(id) diff --git a/app/controllers/concerns/admin/reorderable.rb b/app/controllers/concerns/admin/reorderable.rb index 2ce07bf582985548f7f69d1f07f0363034e669e6..949be51e392a35cd6ce1f108ae68659425cc7ab2 100644 --- a/app/controllers/concerns/admin/reorderable.rb +++ b/app/controllers/concerns/admin/reorderable.rb @@ -3,7 +3,7 @@ module Admin::Reorderable included do def reorder - ids = params[:ids] + ids = params[:ids] || [] ids.each.with_index do |id, index| object = model.find_by(id: id) object.update_column(:position, index + 1) unless object.nil?