From da40234bf508736d66d58a7753a15ac49fac7b9e Mon Sep 17 00:00:00 2001
From: cristian-longhi <cristian-longhi@camaranh.rs.gov.br>
Date: Fri, 10 Aug 2018 13:20:22 -0300
Subject: [PATCH] =?UTF-8?q?Ajusta=20agrupamento=20nos=20relat=C3=B3rios=20?=
 =?UTF-8?q?'Mat=C3=A9rias=20por=20autor'=20e=20'Mat=C3=A9rias=20em=20trami?=
 =?UTF-8?q?ta=C3=A7=C3=A3o'=20(#2094)?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

* Acerta agrupamento nos relatórios de matéria por autor e em tramitação

* Acerta agrupamento nos relatórios de matéria por autor e em tramitação

* Ajusta agrupamento nos relatórios 'Matérias por autor' e 'Matérias em tramitação'

* Ajusta agrupamento nos relatórios 'Matérias por autor' e 'Matérias em tramitação'
---
 sapl/base/forms.py                            |  7 +-
 sapl/base/views.py                            |  2 +-
 .../RelatorioMateriasPorAutor_filter.html     | 73 +++++++++----------
 3 files changed, 43 insertions(+), 39 deletions(-)

diff --git a/sapl/base/forms.py b/sapl/base/forms.py
index 974068b20..b71d161da 100644
--- a/sapl/base/forms.py
+++ b/sapl/base/forms.py
@@ -671,6 +671,11 @@ class RelatorioMateriasTramitacaoilterSet(django_filters.FilterSet):
                                       label='Ano da Matéria',
                                       choices=RANGE_ANOS)
 
+    @property
+    def qs(self):
+        parent = super(RelatorioMateriasTramitacaoilterSet, self).qs
+        return parent.distinct().order_by('-ano', 'tipo', '-numero')
+
     class Meta:
         model = MateriaLegislativa
         fields = ['ano', 'tipo', 'tramitacao__unidade_tramitacao_local',
@@ -736,7 +741,7 @@ class RelatorioMateriasPorAutorFilterSet(django_filters.FilterSet):
     @property
     def qs(self):
         parent = super(RelatorioMateriasPorAutorFilterSet, self).qs
-        return parent.distinct().order_by('-ano', '-numero')
+        return parent.distinct().filter(autoria__primeiro_autor=True).order_by('autoria__autor', '-autoria__primeiro_autor', 'tipo', '-ano', '-numero')
 
     class Meta:
         model = MateriaLegislativa
diff --git a/sapl/base/views.py b/sapl/base/views.py
index b41492f4e..870fbbaa5 100644
--- a/sapl/base/views.py
+++ b/sapl/base/views.py
@@ -393,7 +393,7 @@ class RelatorioMateriasTramitacaoView(FilterView):
         context = super(RelatorioMateriasTramitacaoView,
                         self).get_context_data(**kwargs)
 
-        context['title'] = _('Matérias por Ano, Autor e Tipo')
+        context['title'] = _('Matérias em Tramitação')
 
         qs = context['object_list']
         qs = qs.filter(em_tramitacao=True)
diff --git a/sapl/templates/base/RelatorioMateriasPorAutor_filter.html b/sapl/templates/base/RelatorioMateriasPorAutor_filter.html
index ec26ecd63..bf0b7702f 100644
--- a/sapl/templates/base/RelatorioMateriasPorAutor_filter.html
+++ b/sapl/templates/base/RelatorioMateriasPorAutor_filter.html
@@ -15,56 +15,55 @@
 
     <table class="table table-bordered table-hover">
       <thead class="thead-default" >
-        <tr class="active"><th colspan="2" class="text-center">QUADRO GERAL</th></tr>
+        <tr class="active"><th colspan="3" class="text-center">QUADRO GERAL</th></tr>
         <tr class="active">
-          <th>Tipo Matéria</th>
+          <th colspan="2">Tipo Matéria</th>
           <th>Quantidade</th>
         </tr>
       </thead>
       <tbody>
         {% for key, value in qtdes.items %}
           <tr>
-            <td>{{key.sigla}} - {{key}}</td>
+            <td colspan="2">{{key.sigla}} - {{key}}</td>
             <td>{{value}}</td>
           </tr>
         {% endfor %}
       </tbody>
-    </table>
 
-    <table class="table table-bordered table-hover">
-      <thead class="thead-default" >
-        <tr class="active">
-          <th>Matéria</th>
-          <th>Ementa</th>
-          <th>Autor</th>
-          <th>Coautor(es)</th>
-        </tr>
-      </thead>
-      <tbody>
-        {% for materia in object_list %}
-          <tr>
-            <td><a href="{% url 'sapl.materia:materialegislativa_detail' materia.pk %}">
-              {{materia.tipo.sigla}} {{materia.numero}}/{{materia.ano}}
-            </a></td>
-            <td>{{materia.ementa}}</td>
-            <td>
-              {% for autor in materia.autoria_set.all %}
-                  {% if autor.primeiro_autor %}
-                {{autor.autor}}<br />
-                  {% endif %}
-              {% endfor %}
-            </td>
-            <td>
-              {% for autor in materia.autoria_set.all %}
-                  {% if not autor.primeiro_autor %}
-                {{autor.autor}}<br />
-                  {% endif %}
-              {% endfor %}
-            </td>
-          </tr>
-        {% endfor %}
-      </tbody>
+    {% for materia in object_list %}
+        {% ifchanged materia.autoria_set.first.autor %}
+
+             <thead class="thead-default" >
+               <tr style="border-left: hidden; border-right: hidden;"><th colspan="3"></th></tr>
+               <tr class="active"><th colspan="3" class="text-center">Autor: {{ materia.autoria_set.first.autor }}</th></tr>
+               <tr class="active">
+                 <th width="10%">Matéria</th>
+                 <th>Ementa</th>
+                 <th width="20%">Coautor(es)</th>
+               </tr>
+             </thead>
+        {% endifchanged %}
+             <tbody>
+                 <tr>
+                   <td><a href="{% url 'sapl.materia:materialegislativa_detail' materia.pk %}">
+                     {{materia.tipo.sigla}} {{materia.numero}}/{{materia.ano}}
+                   </a></td>
+                   <td>{% autoescape off %}{{materia.ementa}}{% endautoescape %}</td>
+                   <td>
+                      {% if materia.autoria_set.first != materia.autoria_set.last %}
+                        {% for autor in materia.autoria_set.all %}
+                          {% if not autor.primeiro_autor %}
+                            {{ autor.autor }}<br />
+                          {% endif %}
+                        {% endfor %}
+                      {% endif %}
+                   </td>
+                 </tr>
+             </tbody>
+
+    {% endfor %}
     </table>
+
   {% endif %}
 
 {% endblock base_content %}