Define operations on deletion of elements

Author: Gogs

app/controllers/states_controller.rb

@@ -20,13 +20,13 @@ def show
 
   # PUT /states/:id
   def update
-    @tag.update(state_params)
+    @state.update(state_params)
     head :no_content
   end
 
   # DELETE /states/:id
   def destroy
-    @tag.destroy
+    @state.destroy
     head :no_content
   end
 

app/models/document.rb

@@ -1,8 +1,8 @@
 class Document < ActiveRecord::Base
-  belongs_to :folder
+  belongs_to :folder, optional: true
   belongs_to :user
-  belongs_to :state
-  belongs_to :person
+  belongs_to :state, optional: true
+  belongs_to :person, optional: true
 
   has_many :documenttags
   has_many :tags, through: :documenttags

app/models/folder.rb

@@ -2,8 +2,8 @@ class Folder < ActiveRecord::Base
   belongs_to :folder
   belongs_to :user
 
-  has_many :documents
-  has_many :folders
+  has_many :documents, dependent: :destroy
+  has_many :folders, dependent: :destroy
 
 	def as_json(_options = {})
   	super include: {

app/models/person.rb

@@ -1,7 +1,7 @@
 class Person < ActiveRecord::Base
   belongs_to :user
 
-  has_many :documents
+  has_many :documents, dependent: :nullify
 
   validates_presence_of :name
 end

app/models/tag.rb

@@ -1,7 +1,7 @@
 class Tag < ActiveRecord::Base
   belongs_to :user
 
-  has_many :documenttags
+  has_many :documenttags, dependent: :nullify
   has_many :documents, through: :documenttags
 
   validates_presence_of :name

db/migrate/20190528071704_create_documents.rb

@@ -6,10 +6,10 @@ def change
       t.date :document_date
       t.string :document_url
       t.decimal :version
-      t.references :folder, index: true, foreign_key: true
-      t.references :user, index: true, foreign_key: true
-      t.references :state, index: true, foreign_key: true
-      t.references :person, index: true, foreign_key: true
+      t.references :folder, index: true, foreign_key: {on_delete: :nullify}
+      t.references :user, index: true, foreign_key: {on_delete: :cascade}
+      t.references :state, index: true, foreign_key: {on_delete: :nullify}
+      t.references :person, index: true, foreign_key: {on_delete: :nullify}
 
       t.timestamps null: false
     end

db/schema.rb

@@ -12,7 +12,7 @@
 
 ActiveRecord::Schema.define(version: 2019_05_28_071907) do
 
-  create_table "documents", id: :integer, options: "ENGINE=InnoDB DEFAULT CHARSET=latin1", force: :cascade do |t|
+  create_table "documents", id: :integer, options: "ENGINE=InnoDB DEFAULT CHARSET=utf8", force: :cascade do |t|
     t.string "title"
     t.text "description"
     t.date "document_date"
@@ -30,7 +30,7 @@
     t.index ["user_id"], name: "index_documents_on_user_id"
   end
 
-  create_table "documenttags", id: :integer, options: "ENGINE=InnoDB DEFAULT CHARSET=latin1", force: :cascade do |t|
+  create_table "documenttags", id: :integer, options: "ENGINE=InnoDB DEFAULT CHARSET=utf8", force: :cascade do |t|
     t.integer "document_id"
     t.integer "tag_id"
     t.datetime "created_at", null: false
@@ -39,7 +39,7 @@
     t.index ["tag_id"], name: "index_documenttags_on_tag_id"
   end
 
-  create_table "folders", id: :integer, options: "ENGINE=InnoDB DEFAULT CHARSET=latin1", force: :cascade do |t|
+  create_table "folders", id: :integer, options: "ENGINE=InnoDB DEFAULT CHARSET=utf8", force: :cascade do |t|
     t.string "name"
     t.integer "folder_id"
     t.integer "user_id"
@@ -50,30 +50,30 @@
     t.index ["user_id"], name: "index_folders_on_user_id"
   end
 
-  create_table "people", id: :integer, options: "ENGINE=InnoDB DEFAULT CHARSET=latin1", force: :cascade do |t|
+  create_table "people", id: :integer, options: "ENGINE=InnoDB DEFAULT CHARSET=utf8", force: :cascade do |t|
     t.string "name"
     t.integer "user_id"
     t.datetime "created_at", null: false
     t.datetime "updated_at", null: false
   end
 
-  create_table "states", id: :integer, options: "ENGINE=InnoDB DEFAULT CHARSET=latin1", force: :cascade do |t|
+  create_table "states", id: :integer, options: "ENGINE=InnoDB DEFAULT CHARSET=utf8", force: :cascade do |t|
     t.string "name"
     t.integer "user_id"
     t.datetime "created_at", null: false
     t.datetime "updated_at", null: false
     t.index ["name"], name: "index_states_on_name", unique: true
   end
 
-  create_table "tags", id: :integer, options: "ENGINE=InnoDB DEFAULT CHARSET=latin1", force: :cascade do |t|
+  create_table "tags", id: :integer, options: "ENGINE=InnoDB DEFAULT CHARSET=utf8", force: :cascade do |t|
     t.string "name"
     t.integer "user_id"
     t.string "color"
     t.datetime "created_at", null: false
     t.datetime "updated_at", null: false
   end
 
-  create_table "users", id: :integer, options: "ENGINE=InnoDB DEFAULT CHARSET=latin1", force: :cascade do |t|
+  create_table "users", id: :integer, options: "ENGINE=InnoDB DEFAULT CHARSET=utf8", force: :cascade do |t|
     t.string "name"
     t.string "password_digest"
     t.string "email"
@@ -82,10 +82,10 @@
     t.index ["email"], name: "index_users_on_email", unique: true
   end
 
-  add_foreign_key "documents", "folders"
-  add_foreign_key "documents", "people"
-  add_foreign_key "documents", "states"
-  add_foreign_key "documents", "users"
+  add_foreign_key "documents", "folders", on_delete: :nullify
+  add_foreign_key "documents", "people", on_delete: :nullify
+  add_foreign_key "documents", "states", on_delete: :nullify
+  add_foreign_key "documents", "users", on_delete: :cascade
   add_foreign_key "documenttags", "documents"
   add_foreign_key "documenttags", "tags"
   add_foreign_key "folders", "folders"