Repository: interlegis/sapl

Issue:
Migração de documentos
Foram retiradas do arquivo sapl/utils.py as duas funções abaixo no seguinte PR
https://github.com/interlegis/sapl/commit/71491c33dd3769b66bc98199e6d8027f6862e4e5#diff-248fb73d01307eb54567bfe0ba11d306

def save_texto(sender, instance, **kwargs):
    update_index = UpdateIndexCommand()
    update_index.start()


def delete_texto(sender, instance, **kwargs):
    update_index = UpdateIndexCommand()
    update_index.start()

No entanto o migrador de documentos precisa delas
https://github.com/interlegis/sapl/blob/master/sapl/legacy/migracao_documentos.py#L17


Discussion:
Precisa não.

O q o script tá fazendo é desligar os signals dessas funções. Como elas não são mais necessárias (nem devem ser utilizadas) basta remover toda e qualquer referência a essas funções do arquivo de migração. Dá pra resolver isso com um PR simples. O que NÃO DEVE SER FEITO é ret9rnar essas funções para o código (como não são mais utilizadas não tem nem porque desligar)
opa, vou testar, se eu conseguir fazer funcionar, faço o PR. Obrigado !!
resolvido!
https://github.com/interlegis/sapl/pull/1279

Resolving patch:
diff --git a/sapl/legacy/migracao_documentos.py b/sapl/legacy/migracao_documentos.py
--- a/sapl/legacy/migracao_documentos.py
+++ b/sapl/legacy/migracao_documentos.py
@@ -14,7 +14,6 @@
                                       DocumentoAdministrativo)
 from sapl.sessao.models import SessaoPlenaria
 from sapl.settings import MEDIA_ROOT
-from sapl.utils import delete_texto, save_texto
 
 # MIGRAÇÃO DE DOCUMENTOS  ###################################################
 EXTENSOES = {
@@ -192,29 +191,7 @@ def migrar_docs_por_ids(tipo):
                         tipo.__name__, id, destino))
 
 
-def desconecta_sinais_indexacao():
-    post_save.disconnect(save_texto, NormaJuridica)
-    post_save.disconnect(save_texto, DocumentoAcessorio)
-    post_save.disconnect(save_texto, MateriaLegislativa)
-    post_delete.disconnect(delete_texto, NormaJuridica)
-    post_delete.disconnect(delete_texto, DocumentoAcessorio)
-    post_delete.disconnect(delete_texto, MateriaLegislativa)
-
-
-def conecta_sinais_indexacao():
-    post_save.connect(save_texto, NormaJuridica)
-    post_save.connect(save_texto, DocumentoAcessorio)
-    post_save.connect(save_texto, MateriaLegislativa)
-    post_delete.connect(delete_texto, NormaJuridica)
-    post_delete.connect(delete_texto, DocumentoAcessorio)
-    post_delete.connect(delete_texto, MateriaLegislativa)
-
-
 def migrar_documentos():
-    # precisamos excluir os sinais de post_save e post_delete para não que o
-    # computador não trave com a criação de threads desnecessárias
-    desconecta_sinais_indexacao()
-
     # aqui supomos que uma pasta chamada sapl_documentos está em MEDIA_ROOT
     # com o conteúdo da pasta de mesmo nome do zope
     # Os arquivos da pasta serão movidos para a nova estrutura e a pasta será
@@ -241,6 +218,3 @@ def migrar_documentos():
                   len(sobrando)))
         for doc in sobrando:
             print('  {}'. format(doc))
-            #
-    # reconexão dos sinais desligados no inicio da migração de documentos
-    conecta_sinais_indexacao()


Test patch: