Autoguardar los adjuntos si cambia el perfil.

Fixes #124
INTA-Suelos · Jun 13, 2017 · 8d0b5be · 8d0b5be
1 parent 05561b6
commit 8d0b5be
Show file tree

Hide file tree

Showing 3 changed files with 15 additions and 2 deletions.
diff --git a/app/models/adjunto.rb b/app/models/adjunto.rb
@@ -1,4 +1,4 @@
-# encoding: utf-8
+# Recopila información sobre un archivo adjunto a un perfil
 class Adjunto < ActiveRecord::Base
   belongs_to :perfil, inverse_of: :adjuntos
   belongs_to :usuario, inverse_of: :adjuntos

diff --git a/app/models/perfil.rb b/app/models/perfil.rb
@@ -23,7 +23,7 @@ class Perfil < ActiveRecord::Base
 
   has_many :horizontes, -> { order('horizontes.profundidad_superior ASC, horizontes.id ASC') },
     dependent: :destroy, inverse_of: :perfil
-  has_many :adjuntos,     dependent: :destroy, inverse_of: :perfil
+  has_many :adjuntos,     dependent: :destroy, inverse_of: :perfil, autosave: true
   has_one :capacidad,     dependent: :destroy, inverse_of: :perfil
   has_one :ubicacion,     dependent: :destroy, inverse_of: :perfil
   has_one :paisaje,       dependent: :destroy, inverse_of: :perfil

diff --git a/test/models/adjunto_test.rb b/test/models/adjunto_test.rb
@@ -23,10 +23,23 @@
 
   describe '#publico' do
     it 'se sincroniza con su perfil al guardar' do
+      # Diferente visibilidad
       perfil.update_attribute :publico, !adjunto.publico
 
       adjunto.save && adjunto.publico.must_equal(perfil.publico)
     end
+
+    # Test de regresión por #124
+    # https://github.com/INTA-Suelos/SiSinta/issues/124
+    it 'al guardarse el perfil se sincroniza su adjunto' do
+      # Misma visibilidad
+      perfil.update_attribute :publico, adjunto.publico
+
+      perfil.publico = perfil.reload.publico
+      perfil.save
+
+      adjunto.reload.publico.must_equal(perfil.publico)
+    end
   end
 
   describe '#sincronizar_visibilidad_perfil' do