From a19a8c0b1b0fbdc8d4dfed3c94a082a49e84c1a8 Mon Sep 17 00:00:00 2001
From: Leandro Roberto <leandroroberto.br@gmail.com>
Date: Sun, 6 Jan 2019 14:08:36 -0200
Subject: [PATCH] Refatora form de pesquisa de Documentos Administrativos

---
 sapl/protocoloadm/forms.py | 51 +++++++++++++++++++++-----------------
 sapl/utils.py              |  5 ++++
 2 files changed, 33 insertions(+), 23 deletions(-)

diff --git a/sapl/protocoloadm/forms.py b/sapl/protocoloadm/forms.py
index 90fb7c7ce..2b43e7ab3 100644
--- a/sapl/protocoloadm/forms.py
+++ b/sapl/protocoloadm/forms.py
@@ -20,7 +20,8 @@ from sapl.materia.models import (MateriaLegislativa, TipoMateriaLegislativa,
                                  UnidadeTramitacao)
 from sapl.utils import (RANGE_ANOS, YES_NO_CHOICES, AnoNumeroOrderingFilter,
                         RangeWidgetOverride, autor_label, autor_modal,
-                        choice_anos_com_protocolo)
+                        choice_anos_com_protocolo,
+                        choice_anos_com_documentoadministrativo)
 
 from .models import (AcompanhamentoDocumento, DocumentoAcessorioAdministrativo,
                      DocumentoAdministrativo,
@@ -37,9 +38,7 @@ NATUREZA_PROCESSO = [('0', 'Administrativo'),
                      ('1', 'Legislativo')]
 
 
-EM_TRAMITACAO = [('', '---------'),
-                 (0, 'Sim'),
-                 (1, 'Não')]
+EM_TRAMITACAO = [(0, 'Sim'), (1, 'Não')]
 
 
 class AcompanhamentoDocumentoForm(ModelForm):
@@ -155,19 +154,24 @@ class ProtocoloFilterSet(django_filters.FilterSet):
 
 class DocumentoAdministrativoFilterSet(django_filters.FilterSet):
 
-    ano = django_filters.ChoiceFilter(required=False,
-                                      label='Ano',
-                                      choices=RANGE_ANOS)
+    ano = django_filters.ChoiceFilter(
+        required=False,
+        label='Ano',
+        choices=choice_anos_com_documentoadministrativo)
 
     tramitacao = django_filters.ChoiceFilter(required=False,
                                              label='Em Tramitação?',
-                                             choices=EM_TRAMITACAO)
+                                             choices=YES_NO_CHOICES)
 
-    assunto = django_filters.CharFilter(lookup_expr='icontains')
+    assunto = django_filters.CharFilter(
+        label=_('Assunto'),
+        lookup_expr='icontains')
 
-    interessado = django_filters.CharFilter(lookup_expr='icontains')
+    interessado = django_filters.CharFilter(
+        label=_('Interessado'),
+        lookup_expr='icontains')
 
-    o = AnoNumeroOrderingFilter()
+    o = AnoNumeroOrderingFilter(help_text='')
 
     class Meta:
         filter_overrides = {models.DateField: {
@@ -190,37 +194,38 @@ class DocumentoAdministrativoFilterSet(django_filters.FilterSet):
 
         local_atual = 'tramitacaoadministrativo__unidade_tramitacao_destino'
         self.filters['tipo'].label = 'Tipo de Documento'
+        self.filters['protocolo__numero'].label = 'Núm. Protocolo'
         self.filters['tramitacaoadministrativo__status'].label = 'Situação'
         self.filters[local_atual].label = 'Localização Atual'
 
         row1 = to_row(
-            [('tipo', 6),
-             ('numero', 6)])
+            [('tipo', 8),
+             ('o', 4), ])
 
         row2 = to_row(
-            [('ano', 4),
+            [('numero', 2),
+             ('ano', 2),
              ('protocolo__numero', 2),
              ('numero_externo', 2),
              ('data', 4)])
 
         row3 = to_row(
-            [('interessado', 4),
-             ('assunto', 4),
-             ('tramitacao', 4)])
+            [('interessado', 6),
+             ('assunto', 6)])
 
         row4 = to_row(
-            [('tramitacaoadministrativo__unidade_tramitacao_destino', 6),
-             ('tramitacaoadministrativo__status', 6)])
-
-        row5 = to_row(
-            [('o', 12)])
+            [
+                ('tramitacao', 2),
+                ('tramitacaoadministrativo__status', 5),
+                ('tramitacaoadministrativo__unidade_tramitacao_destino', 5),
+            ])
 
         self.form.helper = FormHelper()
         self.form.helper.form_method = 'GET'
         self.form.helper.layout = Layout(
             Fieldset(_('Pesquisar Documento'),
                      row1, row2,
-                     row3, row4, row5,
+                     row3, row4,
                      form_actions(label='Pesquisar'))
         )
 
diff --git a/sapl/utils.py b/sapl/utils.py
index c812fdcb9..b1970c46b 100644
--- a/sapl/utils.py
+++ b/sapl/utils.py
@@ -369,6 +369,11 @@ def choice_anos_com_protocolo():
     return choice_anos(Protocolo)
 
 
+def choice_anos_com_documentoadministrativo():
+    from sapl.protocoloadm.models import DocumentoAdministrativo
+    return choice_anos(DocumentoAdministrativo)
+
+
 def choice_anos_com_sessaoplenaria():
     try:
         from sapl.sessao.models import SessaoPlenaria