15275: Don't create snapshots whenever is_trashed is true.

Arvados-DCO-1.1-Signed-off-by: Lucas Di Pentima <ldipentima@veritasgenetics.com>

diff --git a/services/api/app/models/collection.rb b/services/api/app/models/collection.rb
index 8dca6fc9fbada7411e4ef2f4dd45c0effb7c71bb..8ec4761df61ee2848e3c3ea0eebc3df5e46e7124 100644 (file)
--- a/services/api/app/models/collection.rb
+++ b/services/api/app/models/collection.rb
@@ -338,7 +338,7 @@ class Collection < ArvadosModel
   def should_preserve_version?
     return false unless (Rails.configuration.Collections.CollectionVersioning && versionable_updates?(self.changes.keys))
 
-    return false if self.changes.keys.include?('is_trashed') && self.is_trashed_was == false
+    return false if self.is_trashed
 
     idle_threshold = Rails.configuration.Collections.PreserveVersionIfIdle
     if !self.preserve_version_was &&