Fix #715

8 years ago · debc275fb2
7 changed files with 116 additions and 96 deletions
--- a/sapl/compilacao/compilacao_data_tables.sql
+++ b/sapl/compilacao/compilacao_data_tables.sql
@ -222,28 +222,28 @@ INSERT INTO compilacao_tipodispositivorelationship (filho_permitido_id, pai_id,
 INSERT INTO compilacao_tipodispositivorelationship (filho_permitido_id, pai_id, filho_de_insercao_automatica, perfil_id, quantidade_permitida, permitir_variacao) VALUES (120, 119, true, 1, 1, false);
 INSERT INTO compilacao_tipodispositivorelationship (filho_permitido_id, pai_id, filho_de_insercao_automatica, perfil_id, quantidade_permitida, permitir_variacao) VALUES (120, 119, true, 2, 1, true);

-INSERT INTO compilacao_tipodispositivorelationship (filho_permitido_id, pai_id, filho_de_insercao_automatica, perfil_id, quantidade_permitida, permitir_variacao) VALUES (125, 117, false, 1, 1, true);
-INSERT INTO compilacao_tipodispositivorelationship (filho_permitido_id, pai_id, filho_de_insercao_automatica, perfil_id, quantidade_permitida, permitir_variacao) VALUES (125, 117, false, 2, 1, true);
-INSERT INTO compilacao_tipodispositivorelationship (filho_permitido_id, pai_id, filho_de_insercao_automatica, perfil_id, quantidade_permitida, permitir_variacao) VALUES (125, 116, false, 1, 1, true);
-INSERT INTO compilacao_tipodispositivorelationship (filho_permitido_id, pai_id, filho_de_insercao_automatica, perfil_id, quantidade_permitida, permitir_variacao) VALUES (125, 116, false, 2, 1, true);
-INSERT INTO compilacao_tipodispositivorelationship (filho_permitido_id, pai_id, filho_de_insercao_automatica, perfil_id, quantidade_permitida, permitir_variacao) VALUES (125, 115, false, 1, 1, true);
-INSERT INTO compilacao_tipodispositivorelationship (filho_permitido_id, pai_id, filho_de_insercao_automatica, perfil_id, quantidade_permitida, permitir_variacao) VALUES (125, 115, false, 2, 1, true);
-INSERT INTO compilacao_tipodispositivorelationship (filho_permitido_id, pai_id, filho_de_insercao_automatica, perfil_id, quantidade_permitida, permitir_variacao) VALUES (125, 114, false, 1, 1, true);
-INSERT INTO compilacao_tipodispositivorelationship (filho_permitido_id, pai_id, filho_de_insercao_automatica, perfil_id, quantidade_permitida, permitir_variacao) VALUES (125, 114, false, 2, 1, true);
-INSERT INTO compilacao_tipodispositivorelationship (filho_permitido_id, pai_id, filho_de_insercao_automatica, perfil_id, quantidade_permitida, permitir_variacao) VALUES (125, 113, false, 1, 1, true);
-INSERT INTO compilacao_tipodispositivorelationship (filho_permitido_id, pai_id, filho_de_insercao_automatica, perfil_id, quantidade_permitida, permitir_variacao) VALUES (125, 113, false, 2, 1, true);
-INSERT INTO compilacao_tipodispositivorelationship (filho_permitido_id, pai_id, filho_de_insercao_automatica, perfil_id, quantidade_permitida, permitir_variacao) VALUES (125, 112, false, 1, 1, true);
-INSERT INTO compilacao_tipodispositivorelationship (filho_permitido_id, pai_id, filho_de_insercao_automatica, perfil_id, quantidade_permitida, permitir_variacao) VALUES (125, 112, false, 2, 1, true);
-INSERT INTO compilacao_tipodispositivorelationship (filho_permitido_id, pai_id, filho_de_insercao_automatica, perfil_id, quantidade_permitida, permitir_variacao) VALUES (125, 111, false, 1, 1, true);
-INSERT INTO compilacao_tipodispositivorelationship (filho_permitido_id, pai_id, filho_de_insercao_automatica, perfil_id, quantidade_permitida, permitir_variacao) VALUES (125, 111, false, 2, 1, true);
-INSERT INTO compilacao_tipodispositivorelationship (filho_permitido_id, pai_id, filho_de_insercao_automatica, perfil_id, quantidade_permitida, permitir_variacao) VALUES (125, 101, false, 1, 1, true);
-INSERT INTO compilacao_tipodispositivorelationship (filho_permitido_id, pai_id, filho_de_insercao_automatica, perfil_id, quantidade_permitida, permitir_variacao) VALUES (125, 101, false, 2, 1, true);
-INSERT INTO compilacao_tipodispositivorelationship (filho_permitido_id, pai_id, filho_de_insercao_automatica, perfil_id, quantidade_permitida, permitir_variacao) VALUES (125, 102, false, 1, 1, true);
-INSERT INTO compilacao_tipodispositivorelationship (filho_permitido_id, pai_id, filho_de_insercao_automatica, perfil_id, quantidade_permitida, permitir_variacao) VALUES (125, 102, false, 2, 1, true);
-INSERT INTO compilacao_tipodispositivorelationship (filho_permitido_id, pai_id, filho_de_insercao_automatica, perfil_id, quantidade_permitida, permitir_variacao) VALUES (125, 103, false, 1, 1, true);
-INSERT INTO compilacao_tipodispositivorelationship (filho_permitido_id, pai_id, filho_de_insercao_automatica, perfil_id, quantidade_permitida, permitir_variacao) VALUES (125, 103, false, 2, 1, true);
-INSERT INTO compilacao_tipodispositivorelationship (filho_permitido_id, pai_id, filho_de_insercao_automatica, perfil_id, quantidade_permitida, permitir_variacao) VALUES (125, 104, false, 1, 1, true);
-INSERT INTO compilacao_tipodispositivorelationship (filho_permitido_id, pai_id, filho_de_insercao_automatica, perfil_id, quantidade_permitida, permitir_variacao) VALUES (125, 104, false, 2, 1, true);
+INSERT INTO compilacao_tipodispositivorelationship (filho_permitido_id, pai_id, filho_de_insercao_automatica, perfil_id, quantidade_permitida, permitir_variacao) VALUES (125, 117, false, 1, -1, false);
+INSERT INTO compilacao_tipodispositivorelationship (filho_permitido_id, pai_id, filho_de_insercao_automatica, perfil_id, quantidade_permitida, permitir_variacao) VALUES (125, 117, false, 2, -1, false);
+INSERT INTO compilacao_tipodispositivorelationship (filho_permitido_id, pai_id, filho_de_insercao_automatica, perfil_id, quantidade_permitida, permitir_variacao) VALUES (125, 116, false, 1, -1, false);
+INSERT INTO compilacao_tipodispositivorelationship (filho_permitido_id, pai_id, filho_de_insercao_automatica, perfil_id, quantidade_permitida, permitir_variacao) VALUES (125, 116, false, 2, -1, false);
+INSERT INTO compilacao_tipodispositivorelationship (filho_permitido_id, pai_id, filho_de_insercao_automatica, perfil_id, quantidade_permitida, permitir_variacao) VALUES (125, 115, false, 1, -1, false);
+INSERT INTO compilacao_tipodispositivorelationship (filho_permitido_id, pai_id, filho_de_insercao_automatica, perfil_id, quantidade_permitida, permitir_variacao) VALUES (125, 115, false, 2, -1, false);
+INSERT INTO compilacao_tipodispositivorelationship (filho_permitido_id, pai_id, filho_de_insercao_automatica, perfil_id, quantidade_permitida, permitir_variacao) VALUES (125, 114, false, 1, -1, false);
+INSERT INTO compilacao_tipodispositivorelationship (filho_permitido_id, pai_id, filho_de_insercao_automatica, perfil_id, quantidade_permitida, permitir_variacao) VALUES (125, 114, false, 2, -1, false);
+INSERT INTO compilacao_tipodispositivorelationship (filho_permitido_id, pai_id, filho_de_insercao_automatica, perfil_id, quantidade_permitida, permitir_variacao) VALUES (125, 113, false, 1, -1, false);
+INSERT INTO compilacao_tipodispositivorelationship (filho_permitido_id, pai_id, filho_de_insercao_automatica, perfil_id, quantidade_permitida, permitir_variacao) VALUES (125, 113, false, 2, -1, false);
+INSERT INTO compilacao_tipodispositivorelationship (filho_permitido_id, pai_id, filho_de_insercao_automatica, perfil_id, quantidade_permitida, permitir_variacao) VALUES (125, 112, false, 1, -1, false);
+INSERT INTO compilacao_tipodispositivorelationship (filho_permitido_id, pai_id, filho_de_insercao_automatica, perfil_id, quantidade_permitida, permitir_variacao) VALUES (125, 112, false, 2, -1, false);
+INSERT INTO compilacao_tipodispositivorelationship (filho_permitido_id, pai_id, filho_de_insercao_automatica, perfil_id, quantidade_permitida, permitir_variacao) VALUES (125, 111, false, 1, -1, false);
+INSERT INTO compilacao_tipodispositivorelationship (filho_permitido_id, pai_id, filho_de_insercao_automatica, perfil_id, quantidade_permitida, permitir_variacao) VALUES (125, 111, false, 2, -1, false);
+INSERT INTO compilacao_tipodispositivorelationship (filho_permitido_id, pai_id, filho_de_insercao_automatica, perfil_id, quantidade_permitida, permitir_variacao) VALUES (125, 101, false, 1, -1, false);
+INSERT INTO compilacao_tipodispositivorelationship (filho_permitido_id, pai_id, filho_de_insercao_automatica, perfil_id, quantidade_permitida, permitir_variacao) VALUES (125, 101, false, 2, -1, false);
+INSERT INTO compilacao_tipodispositivorelationship (filho_permitido_id, pai_id, filho_de_insercao_automatica, perfil_id, quantidade_permitida, permitir_variacao) VALUES (125, 102, false, 1, -1, false);
+INSERT INTO compilacao_tipodispositivorelationship (filho_permitido_id, pai_id, filho_de_insercao_automatica, perfil_id, quantidade_permitida, permitir_variacao) VALUES (125, 102, false, 2, -1, false);
+INSERT INTO compilacao_tipodispositivorelationship (filho_permitido_id, pai_id, filho_de_insercao_automatica, perfil_id, quantidade_permitida, permitir_variacao) VALUES (125, 103, false, 1, -1, false);
+INSERT INTO compilacao_tipodispositivorelationship (filho_permitido_id, pai_id, filho_de_insercao_automatica, perfil_id, quantidade_permitida, permitir_variacao) VALUES (125, 103, false, 2, -1, false);
+INSERT INTO compilacao_tipodispositivorelationship (filho_permitido_id, pai_id, filho_de_insercao_automatica, perfil_id, quantidade_permitida, permitir_variacao) VALUES (125, 104, false, 1, -1, false);
+INSERT INTO compilacao_tipodispositivorelationship (filho_permitido_id, pai_id, filho_de_insercao_automatica, perfil_id, quantidade_permitida, permitir_variacao) VALUES (125, 104, false, 2, -1, false);



--- a/sapl/compilacao/models.py
+++ b/sapl/compilacao/models.py
@ -1246,7 +1246,7 @@ class Dispositivo(BaseModel, TimestampedMixin):
            if disps.exists():
                dispositivo.set_numero_completo(
                    disps[0].get_numero_completo())
-                dispositivo.transform_in_next()
+                # dispositivo.transform_in_next()
            else:
                dispositivo.set_numero_completo([1, 0, 0, 0, 0, 0, ])
        else:
--- a/sapl/compilacao/views.py
+++ b/sapl/compilacao/views.py
@ -1577,24 +1577,9 @@ class ActionDispositivoCreateMixin(ActionsCommonsMixin):
    def allowed_inserts(self, _base=None):
        request = self.request
        try:
-            if request and 'perfil_estrutural' not in request.session:
-                self.set_perfil_in_session(request)
-
-            perfil_pk = request.session['perfil_estrutural']
-
            base = Dispositivo.objects.get(
                pk=self.kwargs['dispositivo_id'] if not _base else _base)

-            prox_possivel = Dispositivo.objects.filter(
-                ordem__gt=base.ordem,
-                nivel__lte=base.nivel,
-                ta_id=base.ta_id)[:1]
-
-            if prox_possivel.exists():
-                prox_possivel = prox_possivel[0]
-            else:
-                prox_possivel = None
-
            result = [{'tipo_insert': force_text(string_concat(
                _('Inserir Após'),
                ' ',
@ -1612,10 +1597,24 @@ class ActionDispositivoCreateMixin(ActionsCommonsMixin):
                {'tipo_insert': force_text(_('Inserir Antes')),
                 'icone': '&#8630;&nbsp;',
                 'action': 'json_add_prior',
-                 'itens': []}
-            ]
+                 'itens': []}]

-            # Possíveis inserções sequenciais já existentes
+            if request and 'perfil_estrutural' not in request.session:
+                self.set_perfil_in_session(request)
+
+            perfil_pk = request.session['perfil_estrutural']
+
+            prox_possivel = Dispositivo.objects.filter(
+                ordem__gt=base.ordem,
+                nivel__lte=base.nivel,
+                ta_id=base.ta_id)[:1]
+
+            if prox_possivel.exists():
+                prox_possivel = prox_possivel[0]
+            else:
+                prox_possivel = None
+
+            # Possíveis inserções de nível já existentes
            parents = base.get_parents()
            parents.insert(0, base)
            nivel = sys.maxsize
@ -1769,6 +1768,41 @@ class ActionDispositivoCreateMixin(ActionsCommonsMixin):
                    else:
                        Dispositivo.set_numero_for_add_in(base, base, td)

+                    r = []
+
+                    flag_pv = td.permitido_variacao(tipb, perfil_pk=perfil_pk)
+
+                    if td.contagem_continua and flag_pv:
+                        flag_direcao = 1
+                        flag_variacao = 0
+                        while True:
+                            if base.dispositivo0 == 0:
+                                local_insert = 1
+                            else:
+                                local_insert = 0
+
+                            rt = base.transform_in_next(flag_direcao)
+                            if not rt[0]:
+                                break
+                            flag_variacao += rt[1]
+                            r.append({'class_css': td.class_css,
+                                      'tipo_pk': td.pk,
+                                      'variacao': flag_variacao,
+                                      'provavel': '%s <small>(%s)</small>' % (
+                                          base.rotulo_padrao(local_insert),
+                                          base.tipo_dispositivo.nome,),
+                                      'dispositivo_base': base.pk})
+
+                            flag_direcao = -1
+
+                        r.reverse()
+
+                        if len(r) > 0 and td.formato_variacao0 == \
+                                TipoDispositivo.FNCN:
+                            r = [r[0], ]
+                    else:
+                        if td.contagem_continua:
+                            base.transform_in_next()
                        r = [{'class_css': td.class_css,
                              'tipo_pk': td.pk,
                              'variacao': 0,
@ -1778,29 +1812,16 @@ class ActionDispositivoCreateMixin(ActionsCommonsMixin):
                              'dispositivo_base': base.pk}]

                    if paradentro == 1:
-                        """if (tipb.class_css == 'caput' and
-                                td.class_css == 'paragrafo'):
-                            result[0]['itens'].insert(0, r[0])
-                        else:"""
                        result[1]['itens'] += r
                    else:
                        result[2]['itens'] += r
                        result[0]['itens'] += r

-            # if len(result[0]['itens']) < len(result[1]['itens']):
-            #    r = result[0]
-            #    result.remove(result[0])
-            #    result.insert(1, r)
-
-            # remover temporariamente a opção inserir antes
-            # confirmar necessidade
+            # FIXME para liberar as opções de inserção antes,
+            # o método json_add_prior deve ser implementado
            if len(result) > 2:
                result.pop()

-            # if tipb.dispositivo_de_articulacao and\
-            #        tipb.dispositivo_de_alteracao:
-            #    result.pop()
-
            return result

        except Exception as e:
@ -1912,15 +1933,13 @@ class ActionDispositivoCreateMixin(ActionsCommonsMixin):
                        ordem__lte=base.ordem,
                        ordem__gte=parents[-1].ordem,
                        tipo_dispositivo_id=tipo.pk,
-                        ta_id=base.ta_id)[:1]
+                        ta_id=base.ta_id).first()

-                    if not ultimo_irmao.exists():
-                        dp.set_numero_completo([1, 0, 0, 0, 0, 0, ])
-                    else:
-                        ultimo_irmao = ultimo_irmao[0]
                    dp.set_numero_completo(
+                        [1, 0, 0, 0, 0, 0, ] if not ultimo_irmao else
                        ultimo_irmao.get_numero_completo())
-                        dp.transform_in_next()
+                    if ultimo_irmao:
+                        dp.transform_in_next(variacao)
                else:
                    if ';' in tipo.rotulo_prefixo_texto:
                        dp.set_numero_completo([0, 0, 0, 0, 0, 0, ])
@ -2198,7 +2217,6 @@ class ActionsEditMixin(ActionDragAndMoveDispositivoAlteradoMixin,
    def json_add_next_registra_inclusao(
            self, context, local_add='json_add_next'):

-        base = Dispositivo.objects.get(pk=self.kwargs['dispositivo_id'])
        bloco_alteracao = Dispositivo.objects.get(pk=context['pk_bloco'])

        data = {}
--- a/sapl/sessao/forms.py
+++ b/sapl/sessao/forms.py
@ -15,8 +15,7 @@ from sapl.parlamentares.models import Parlamentar
 from sapl.utils import RANGE_DIAS_MES, RANGE_MESES, autor_label, autor_modal

 from .models import (Bancada, ExpedienteMateria, Orador, OradorExpediente,
-                     OrdemDia, PresencaOrdemDia, SessaoPlenaria,
-                     SessaoPlenariaPresenca)
+                     OrdemDia, SessaoPlenaria, SessaoPlenariaPresenca)


 def recupera_anos():
@ -203,6 +202,7 @@ class SessaoPlenariaFilterSet(django_filters.FilterSet):


 class AdicionarVariasMateriasFilterSet(MateriaLegislativaFilterSet):
+
    class Meta:
        model = MateriaLegislativa
        fields = ['numero',
--- a/sapl/sessao/urls.py
+++ b/sapl/sessao/urls.py
@ -54,7 +54,8 @@ urlpatterns = [
        abrir_votacao_ordem_view,
        name="abrir_votacao"),
    url(r'^sessao/(?P<pk>\d+)/reordenar-expediente$',
-                   reordernar_materias_expediente, name="reordenar_expediente"),
+        reordernar_materias_expediente,
+        name="reordenar_expediente"),
    url(r'^sessao/(?P<pk>\d+)/reordenar-ordem$', reordernar_materias_ordem,
        name="reordenar_ordem"),
    url(r'^sessao/rest/', include(sessao_rest)),
@ -105,15 +106,18 @@ urlpatterns = [
        PesquisarSessaoPlenariaView.as_view(), name='pesquisar_sessao'),
    url(r'^sessao/(?P<pk>\d+)/matordemdia/votnom/(?P<oid>\d+)/(?P<mid>\d+)$',
        VotacaoNominalView.as_view(), name='votacaonominal'),
-    url(r'^sessao/(?P<pk>\d+)/matordemdia/votnom/edit/(?P<oid>\d+)/(?P<mid>\d+)$',
+    url(r'^sessao/(?P<pk>\d+)/matordemdia/votnom'
+        '/edit/(?P<oid>\d+)/(?P<mid>\d+)$',
        VotacaoNominalEditView.as_view(), name='votacaonominaledit'),
    url(r'^sessao/(?P<pk>\d+)/matordemdia/votsec/(?P<oid>\d+)/(?P<mid>\d+)$',
        VotacaoView.as_view(), name='votacaosecreta'),
-    url(r'^sessao/(?P<pk>\d+)/matordemdia/votsec/view/(?P<oid>\d+)/(?P<mid>\d+)$',
+    url(r'^sessao/(?P<pk>\d+)/matordemdia/votsec'
+        '/view/(?P<oid>\d+)/(?P<mid>\d+)$',
        VotacaoEditView.as_view(), name='votacaosecretaedit'),
    url(r'^sessao/(?P<pk>\d+)/matordemdia/votsimb/(?P<oid>\d+)/(?P<mid>\d+)$',
        VotacaoView.as_view(), name='votacaosimbolica'),
-    url(r'^sessao/(?P<pk>\d+)/matordemdia/votsimb/view/(?P<oid>\d+)/(?P<mid>\d+)$',
+    url(r'^sessao/(?P<pk>\d+)/matordemdia/votsimb'
+        '/view/(?P<oid>\d+)/(?P<mid>\d+)$',
        VotacaoEditView.as_view(), name='votacaosimbolicaedit'),
    url(r'^sessao/(?P<pk>\d+)/matexp/votnom/(?P<oid>\d+)/(?P<mid>\d+)$',
        VotacaoNominalExpedienteView.as_view(), name='votacaonominalexp'),
--- a/sapl/test_urls.py
+++ b/sapl/test_urls.py
@ -1,11 +1,11 @@
+import pytest
 from django.apps import apps
 from django.contrib.auth import get_user_model
 from django.contrib.auth.management import _get_all_permissions
 from django.contrib.auth.models import Permission
 from django.contrib.contenttypes.models import ContentType
-from django.utils.translation import string_concat
 from django.utils.translation import ugettext_lazy as _
-import pytest
+from django.utils.translation import string_concat

 from sapl.crud.base import PermissionRequiredForAppCrudMixin
 from scripts.inicializa_grupos_autorizacoes import cria_grupos_permissoes
@ -13,7 +13,6 @@ from scripts.lista_urls import lista_urls

 from .settings import SAPL_APPS

-
 pytestmark = pytest.mark.django_db

 sapl_appconfs = [apps.get_app_config(n[5:]) for n in SAPL_APPS]
@ -333,4 +332,4 @@ def em_construcao_crud_permissions_urls(urls_app, client):
                            url.startswith('/sistema/'):
                        assert btn_login not in str(rp.content)

-                logout = client.get('/logout/', follow=True)
+                client.get('/logout/', follow=True)
--- a/scripts/lista_urls.py
+++ b/scripts/lista_urls.py
@ -8,9 +8,8 @@ if __name__ == '__main__':
    django.setup()

 if True:
-    from django.apps import apps
    from sapl.urls import urlpatterns
-    from django.core.urlresolvers import RegexURLResolver, RegexURLPattern
+    from django.core.urlresolvers import RegexURLResolver


 class ListaUrls():