CV2-3916: log archived update (#1848)

meedan · Apr 3, 2024 · 008b540 · 008b540
1 parent 587bee6
commit 008b540
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 4 deletions.
diff --git a/app/models/project_media.rb b/app/models/project_media.rb
@@ -6,7 +6,7 @@ class ProjectMedia < ApplicationRecord
 
   accepts_nested_attributes_for :media, :claim_description
 
-  has_paper_trail on: [:create, :update, :destroy], only: [:source_id], if: proc { |_x| User.current.present? }, versions: { class_name: 'Version' }
+  has_paper_trail on: [:create, :update, :destroy], only: [:source_id, :archived], if: proc { |_x| User.current.present? }, versions: { class_name: 'Version' }
 
   include ProjectAssociation
   include ProjectMediaAssociations
@@ -382,8 +382,9 @@ def get_project_media_sources
     sources.to_json
   end
 
-  def version_metadata(_changes)
-    meta = { source_name: self.source&.name }
+  def version_metadata(changes)
+    changes = begin JSON.parse(changes) rescue {} end
+    meta = changes.keys.include?('source_id') ? { source_name: self.source&.name } : {}
     meta.to_json
   end
 

diff --git a/test/models/project_media_5_test.rb b/test/models/project_media_5_test.rb
@@ -641,7 +641,11 @@ def setup
       assert_difference 'PaperTrail::Version.count', 2 do
         pm = create_project_media team: t, media: m, user: u, skip_autocreate_source: false
       end
-      assert_equal 2, pm.versions.count
+      assert_difference 'PaperTrail::Version.count' do
+        pm.archived = CheckArchivedFlags::FlagCodes::SPAM
+        pm.save!
+      end
+      assert_equal 3, Version.from_partition(t.id).where(item_type: 'ProjectMedia', item_id: pm.id).count
       pm.destroy!
       v = Version.from_partition(t.id).where(item_type: 'ProjectMedia', item_id: pm.id, event: 'destroy').last
       assert_not_nil v