From 7122b44d0d68d22e30dd65682156f492d871dca5 Mon Sep 17 00:00:00 2001
From: LeandroRoberto <leandroroberto.br@gmail.com>
Date: Thu, 15 Sep 2016 15:29:05 -0300
Subject: [PATCH] =?UTF-8?q?Def=20tipo=20de=20bot=20de=20exclus=C3=A3o=20p?=
 =?UTF-8?q?=20blocos=20de=20altera=C3=A7=C3=A3o?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 sapl/compilacao/forms.py                      | 27 +++++++++++--------
 .../migrations/0052_auto_20160915_1327.py     | 19 +++++++++++++
 sapl/compilacao/models.py                     |  1 +
 .../templatetags/compilacao_filters.py        |  2 +-
 sapl/compilacao/views.py                      | 11 ++++++++
 .../templates/compilacao/text_edit_bloco.html |  6 +++--
 6 files changed, 52 insertions(+), 14 deletions(-)
 create mode 100644 sapl/compilacao/migrations/0052_auto_20160915_1327.py

diff --git a/sapl/compilacao/forms.py b/sapl/compilacao/forms.py
index 88e847690..417ced537 100644
--- a/sapl/compilacao/forms.py
+++ b/sapl/compilacao/forms.py
@@ -636,16 +636,20 @@ class DispositivoEdicaoBasicaForm(ModelForm):
 
         btns_excluir = []
 
-        btns_excluir = [
-            HTML('<a class="btn btn-danger btn-action btn-excluir" '
-                 'action="json_delete_item_dispositivo" '
-                 'title="%s" '
-                 'pk="%s" '
-                 '>%s</a>' % (_('Excluir apenas este dispositivo.'),
-                              inst.pk,
-                              _('Excluir Dispositivo')))]
-
-        if inst.dispositivos_filhos_set.exists():
+        if not inst.tipo_dispositivo.dispositivo_de_alteracao and \
+                not inst.tipo_dispositivo.dispositivo_de_articulacao:
+            btns_excluir = [
+                HTML('<a class="btn btn-danger btn-action btn-excluir" '
+                     'action="json_delete_item_dispositivo" '
+                     'title="%s" '
+                     'pk="%s" '
+                     '>%s</a>' % (_('Excluir apenas este dispositivo.'),
+                                  inst.pk,
+                                  _('Excluir Dispositivo')))]
+
+        if inst.dispositivos_filhos_set.exists() or (
+                inst.tipo_dispositivo.dispositivo_de_alteracao and
+                inst.tipo_dispositivo.dispositivo_de_articulacao):
             btns_excluir.append(
                 HTML(
                     '<a class="btn btn-danger btn-action btn-excluir" '
@@ -1250,7 +1254,8 @@ class DispositivoRegistroAlteracaoForm(Form):
             data_type_selection='radio',
             template="compilacao/layout/dispositivo_radio.html")
 
-        layout.append(Fieldset(_('Registro de Alteração - Seleção do Dispositivo a ser alterado'),
+        layout.append(Fieldset(_('Registro de Alteração - '
+                                 'Seleção do Dispositivo a ser alterado'),
                                row_dispositivo,
                                css_class="col-md-12"))
         layout.append(Field('dispositivo_search_form'))
diff --git a/sapl/compilacao/migrations/0052_auto_20160915_1327.py b/sapl/compilacao/migrations/0052_auto_20160915_1327.py
new file mode 100644
index 000000000..5cab6191f
--- /dev/null
+++ b/sapl/compilacao/migrations/0052_auto_20160915_1327.py
@@ -0,0 +1,19 @@
+# -*- coding: utf-8 -*-
+# Generated by Django 1.9.7 on 2016-09-15 16:27
+from __future__ import unicode_literals
+
+from django.db import migrations
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ('compilacao', '0051_dispositivo_auto_inserido'),
+    ]
+
+    operations = [
+        migrations.AlterUniqueTogether(
+            name='dispositivo',
+            unique_together=set([('ta', 'dispositivo0', 'dispositivo1', 'dispositivo2', 'dispositivo3', 'dispositivo4', 'dispositivo5', 'tipo_dispositivo', 'dispositivo_pai', 'dispositivo_atualizador', 'ta_publicado', 'publicacao'), ('ta', 'ordem')]),
+        ),
+    ]
diff --git a/sapl/compilacao/models.py b/sapl/compilacao/models.py
index b3c00acfa..7ddd3a00c 100644
--- a/sapl/compilacao/models.py
+++ b/sapl/compilacao/models.py
@@ -650,6 +650,7 @@ class Dispositivo(BaseModel, TimestampedMixin):
              'dispositivo5',
              'tipo_dispositivo',
              'dispositivo_pai',
+             'dispositivo_atualizador',
              'ta_publicado',
              'publicacao',),
         )
diff --git a/sapl/compilacao/templatetags/compilacao_filters.py b/sapl/compilacao/templatetags/compilacao_filters.py
index 2d4d512fd..4846e71be 100644
--- a/sapl/compilacao/templatetags/compilacao_filters.py
+++ b/sapl/compilacao/templatetags/compilacao_filters.py
@@ -212,7 +212,7 @@ def heranca(request, d, ignore_ultimo=0, ignore_primeiro=0):
     ta_id = str(d.ta_id)
     d_pk = str(d.pk)
     if ta_id not in ta_dpts_parents or d_pk not in ta_dpts_parents[ta_id]:
-        #print('recarregando estrutura temporaria de heranças')
+        print('recarregando estrutura temporaria de heranças')
         dpts_parents = {}
         ta_dpts_parents[ta_id] = dpts_parents
         update_dispositivos_parents(dpts_parents, ta_id)
diff --git a/sapl/compilacao/views.py b/sapl/compilacao/views.py
index f513f2e0e..5938c4f68 100644
--- a/sapl/compilacao/views.py
+++ b/sapl/compilacao/views.py
@@ -2070,6 +2070,11 @@ class ActionsEditMixin(ActionDragAndMoveDispositivoAlteradoMixin,
 
         4 - Se é um dispositivo articulado com subsequente
             - não deveria ter filhos locais
+
+        5 - Alterações em dispositivo articulado só são relevantes para
+            alteração de rótulo. O editor dinâmico não possibilita essa
+            mudança, porém, após registro de alteração, a mudança de rótulo
+            pode ser feita no editor avançado.
         """
 
         data = {}
@@ -2079,6 +2084,12 @@ class ActionsEditMixin(ActionDragAndMoveDispositivoAlteradoMixin,
         history = dispositivo_a_alterar.history()
 
         for d in history:
+            """FIXME: A comparação "<" deverá ser mudada para
+                "<=" caso um seja necessário permitir duas alterações
+                com mesmo inicio_vigencia no mesmo dispositivo. Neste Caso,
+                a sequencia correta ficará a cargo dos reposicionamentos entre
+                dispositivos de mesmo nível,
+            """
             if d.inicio_vigencia < bloco_alteracao.inicio_vigencia:
                 dispositivo_a_alterar = d
                 break
diff --git a/sapl/templates/compilacao/text_edit_bloco.html b/sapl/templates/compilacao/text_edit_bloco.html
index 41dda2b6d..de5a36060 100644
--- a/sapl/templates/compilacao/text_edit_bloco.html
+++ b/sapl/templates/compilacao/text_edit_bloco.html
@@ -7,9 +7,11 @@
       <div class="dpt-actions"></div>
       <div class="dpt-actions-fixed">
         <a class="btn-action btn-dpt-edit btn btn-default btn-sm" pk="{{node.dpt.pk}}" title=""><span class="deactivate">{% trans 'Editar'%}</span><span class="activate">{% trans 'Em Edição'%}:</span> {{ node.td.nome }} {{ node.dpt.rotulo }}</a>
-        <a class="btn-action btn-dpt-edit btn btn-warning btn-sm activate" pk="{{node.dpt.pk}}" action="editor-help" data-toggle="modal" data-target="#modal-help"><span class="deactivate"></span><span class="activate"><i class="fa fa-question" aria-hidden="true"></i></span></a>
         <div class="btn-group">
-        <a class="btn-action btn-dpt-edit btn btn-info btn-sm activate" pk="{{node.dpt.pk}}" action="editor-close" title="{% trans 'Fechar Edição' %}"><span class="deactivate"></span><span class="activate"><i class="fa fa-times" aria-hidden="true"></i></span></a>
+          <a class="btn-action btn-dpt-edit btn btn-warning btn-sm activate" pk="{{node.dpt.pk}}" action="editor-help" data-toggle="modal" data-target="#modal-help"><span class="deactivate"></span><span class="activate"><i class="fa fa-question" aria-hidden="true"></i></span></a>
+        </div>
+        <div class="btn-group">
+          <a class="btn-action btn-dpt-edit btn btn-info btn-sm activate" pk="{{node.dpt.pk}}" action="editor-close" title="{% trans 'Fechar Edição' %}"><span class="deactivate"></span><span class="activate"><i class="fa fa-times" aria-hidden="true"></i></span></a>
         </div>
         {% if not node.dpt.rotulo and not node.dpt.texto and node.td.dispositivo_de_articulacao%}
         {% endif %}