actor and object are allowed to be nil

manyfold3d · Nov 27, 2024 · 2f2be1c · 2f2be1c
1 parent 98df134
commit 2f2be1c
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 7 deletions.
diff --git a/app/models/federails/moderation/report.rb b/app/models/federails/moderation/report.rb
@@ -1,6 +1,6 @@
 class Federails::Moderation::Report < ApplicationRecord
-  belongs_to :federails_actor, class_name: "Federails::Actor"
-  belongs_to :object, polymorphic: true
+  belongs_to :federails_actor, class_name: "Federails::Actor", optional: true
+  belongs_to :object, polymorphic: true, optional: true
 
   def resolve!
     update!(resolved_at: DateTime.now, resolution: "resolved")

diff --git a/db/migrate/20241127105043_create_federails_moderation_reports.rb b/db/migrate/20241127105043_create_federails_moderation_reports.rb
@@ -2,9 +2,9 @@ class CreateFederailsModerationReports < ActiveRecord::Migration[8.0]
   def change
     create_table :federails_moderation_reports do |t|
       t.string :federated_url
-      t.references :federails_actor, null: false, foreign_key: true
+      t.references :federails_actor, foreign_key: true
       t.string :content
-      t.references :object, polymorphic: true, null: false
+      t.references :object, polymorphic: true
       t.datetime :resolved_at
       t.string :resolution
       t.timestamps

diff --git a/spec/dummy/db/schema.rb b/spec/dummy/db/schema.rb
@@ -62,10 +62,10 @@
 
   create_table "federails_moderation_reports", force: :cascade do |t|
     t.string "federated_url"
-    t.integer "federails_actor_id", null: false
+    t.integer "federails_actor_id"
     t.string "content"
-    t.string "object_type", null: false
-    t.integer "object_id", null: false
+    t.string "object_type"
+    t.integer "object_id"
     t.datetime "resolved_at"
     t.string "resolution"
     t.datetime "created_at", null: false