rubocop -a.

app/controllers/book_masters_controller.rb

@@ -61,6 +61,6 @@ def set_book_master
 
     # Only allow a list of trusted parameters through.
     def book_master_params
-      params.require(:book_master).permit(:isbn, :title, :publication_date, :ndc_category_id, {author_ids: []})
+      params.require(:book_master).permit(:isbn, :title, :publication_date, :ndc_category_id, { author_ids: [] })
     end
 end

app/controllers/book_stock_statuses_controller.rb

@@ -47,7 +47,8 @@ def update
   # DELETE /book_stock_statuses/1
   def destroy
     @book_stock_status.destroy!
-    redirect_to book_stock_statuses_url, notice: t("controller.destroy.success", model: BookStockStatus.model_name.human)
+    redirect_to book_stock_statuses_url,
+                notice: t("controller.destroy.success", model: BookStockStatus.model_name.human)
   end
 
   private

app/controllers/lending_sets_controller.rb

@@ -63,6 +63,7 @@ def set_lending_set
 
     # Only allow a list of trusted parameters through.
     def lending_set_params
-      params.require(:lending_set).permit(:customer_id, :lending_status_id, :lend_start_date, :return_deadline_date, :return_date, :memo, {book_stock_ids: []})
+      params.require(:lending_set).permit(:customer_id, :lending_status_id, :lend_start_date, :return_deadline_date,
+                                          :return_date, :memo, { book_stock_ids: [] })
     end
 end

app/models/author.rb

@@ -3,7 +3,7 @@ def self.ransackable_attributes(_auth_object = nil)
     %w[name id created_at updated_at]
   end
 
-  validates :name, presence: true, uniqueness:true
+  validates :name, presence: true, uniqueness: true
 
   has_many :book_author_relationship
   has_many :book_masters, through: :book_author_relationship

app/models/book_master.rb

@@ -2,8 +2,9 @@ class BookMaster < ApplicationRecord
   def self.ransackable_attributes(_auth_object = nil)
     %w[isbn title publication_date ndc_category_id author_id id created_at updated_at]
   end
-  def self.ransackable_associations(auth_object = nil)
-    ["authors", "book_author_relationship", "ndc_category"]
+
+  def self.ransackable_associations(_auth_object = nil)
+    %w[authors book_author_relationship ndc_category]
   end
 
   # TODO: ちゃんとした判定ロジックを作る

app/models/lending.rb

@@ -2,8 +2,9 @@ class Lending < ApplicationRecord
   def self.ransackable_attributes(_auth_object = nil)
     %w[lending_set_id book_stock_id id created_at updated_at]
   end
-  def self.ransackable_associations(auth_object = nil)
-    ["lending_set_id", "book_stock_id", "id", "created_at", "updated_at"]
+
+  def self.ransackable_associations(_auth_object = nil)
+    %w[lending_set_id book_stock_id id created_at updated_at]
   end
   belongs_to :lending_set
   belongs_to :book_stock

app/models/lending_set.rb

@@ -1,22 +1,24 @@
 class LendingSet < ApplicationRecord
   def self.ransackable_attributes(_auth_object = nil)
-    %w[customer_id book_stock_id lending_status_id lend_start_date return_deadline_date return_date memo id created_at updated_at]
+    %w[customer_id book_stock_id lending_status_id lend_start_date return_deadline_date return_date memo id created_at
+       updated_at]
   end
-  def self.ransackable_associations(auth_object = nil)
-    ["book_stocks", "customer", "lending", "lending_status"]
+
+  def self.ransackable_associations(_auth_object = nil)
+    %w[book_stocks customer lending lending_status]
   end
 
   after_save do
-    if self.lending_status.name === "貸出中"
-      self.book_stocks.each {|bs|
-          bs.book_stock_status_id = 2
-          bs.save
-      }
-    elsif self.lending_status.name === "返却済"
-      self.book_stocks.each {|bs|
-          bs.book_stock_status_id = 1
-          bs.save
-      }
+    if lending_status.name === "貸出中"
+      book_stocks.each do |bs|
+        bs.book_stock_status_id = 2
+        bs.save
+      end
+    elsif lending_status.name === "返却済"
+      book_stocks.each do |bs|
+        bs.book_stock_status_id = 1
+        bs.save
+      end
     end
   end
 

app/views/lending_sets/_lending_set.json.jbuilder

@@ -1,2 +1,3 @@
-json.extract! lending_set, :id, :customer_id, :lending_status_id, :lend_start_date, :return_deadline_date, :return_date, :memo, :created_at, :updated_at
+json.extract! lending_set, :id, :customer_id, :lending_status_id, :lend_start_date, :return_deadline_date,
+              :return_date, :memo, :created_at, :updated_at
 json.url lending_set_url(lending_set, format: :json)

db/migrate/20240905113117_create_ndc_categories.rb

@@ -1,8 +1,8 @@
 class CreateNdcCategories < ActiveRecord::Migration[7.2]
   def change
     create_table :ndc_categories do |t|
-      t.string :name, :null => false
-      t.integer :number, :null => false
+      t.string :name, null: false
+      t.integer :number, null: false
 
       t.timestamps
     end

db/migrate/20240905114943_create_book_masters.rb

@@ -1,9 +1,9 @@
 class CreateBookMasters < ActiveRecord::Migration[7.2]
   def change
     create_table :book_masters do |t|
-      t.string :isbn, :null => false
-      t.string :title, :null => false
-      t.date :publication_date, :null => false
+      t.string :isbn, null: false
+      t.string :title, null: false
+      t.date :publication_date, null: false
       t.references :ndc_category, null: false, foreign_key: true
 
       t.timestamps

db/migrate/20240906035618_create_authors.rb

@@ -1,7 +1,7 @@
 class CreateAuthors < ActiveRecord::Migration[7.2]
   def change
     create_table :authors do |t|
-      t.string :name, :null => false
+      t.string :name, null: false
 
       t.timestamps
     end

db/migrate/20240907021517_create_book_stock_statuses.rb

@@ -1,7 +1,7 @@
 class CreateBookStockStatuses < ActiveRecord::Migration[7.2]
   def change
     create_table :book_stock_statuses do |t|
-      t.string :name, :null => false
+      t.string :name, null: false
 
       t.timestamps
     end

db/migrate/20240907053532_create_customers.rb

@@ -1,8 +1,8 @@
 class CreateCustomers < ActiveRecord::Migration[7.2]
   def change
     create_table :customers do |t|
-      t.string :name, :null => false
-      t.string :email_address, :null => false
+      t.string :name, null: false
+      t.string :email_address, null: false
 
       t.timestamps
     end

db/migrate/20240907053536_create_lending_statuses.rb

@@ -1,7 +1,7 @@
 class CreateLendingStatuses < ActiveRecord::Migration[7.2]
   def change
     create_table :lending_statuses do |t|
-      t.string :name, :null => false
+      t.string :name, null: false
 
       t.timestamps
     end

db/migrate/20240907053810_create_lending_sets.rb

@@ -3,8 +3,8 @@ def change
     create_table :lending_sets do |t|
       t.references :customer, null: false, foreign_key: true
       t.references :lending_status, null: false, foreign_key: true
-      t.date :lend_start_date, :null => false
-      t.date :return_deadline_date, :null => false
+      t.date :lend_start_date, null: false
+      t.date :return_deadline_date, null: false
       t.date :return_date
       t.text :memo