From c2aea79ff5a80ae1eb733c511344d503c1084c33 Mon Sep 17 00:00:00 2001
From: Edward Ribeiro <eribeiro@interlegis.leg.br>
Date: Thu, 3 May 2018 10:56:15 -0300
Subject: [PATCH] Fixes #1920

---
 .../migrations/0014_auto_20180503_1055.py     | 23 +++++++++++++++++++
 sapl/comissoes/models.py                      |  2 ++
 sapl/comissoes/views.py                       |  7 +++++-
 3 files changed, 31 insertions(+), 1 deletion(-)
 create mode 100644 sapl/comissoes/migrations/0014_auto_20180503_1055.py

diff --git a/sapl/comissoes/migrations/0014_auto_20180503_1055.py b/sapl/comissoes/migrations/0014_auto_20180503_1055.py
new file mode 100644
index 000000000..7a2524fed
--- /dev/null
+++ b/sapl/comissoes/migrations/0014_auto_20180503_1055.py
@@ -0,0 +1,23 @@
+# -*- coding: utf-8 -*-
+# Generated by Django 1.9.11 on 2018-05-03 13:55
+from __future__ import unicode_literals
+
+from django.db import migrations
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ('comissoes', '0013_auto_20180312_1533'),
+    ]
+
+    operations = [
+        migrations.AlterModelOptions(
+            name='composicao',
+            options={'ordering': ['periodo'], 'verbose_name': 'Composição de Comissão', 'verbose_name_plural': 'Composições de Comissão'},
+        ),
+        migrations.AlterModelOptions(
+            name='periodo',
+            options={'ordering': ['-data_inicio', '-data_fim'], 'verbose_name': 'Período de composição de Comissão', 'verbose_name_plural': 'Períodos de composição de Comissão'},
+        ),
+    ]
diff --git a/sapl/comissoes/models.py b/sapl/comissoes/models.py
index f0b1b8d8b..0c3240fc6 100644
--- a/sapl/comissoes/models.py
+++ b/sapl/comissoes/models.py
@@ -105,6 +105,7 @@ class Periodo(models.Model):  # PeriodoCompComissao
     class Meta:
         verbose_name = _('Período de composição de Comissão')
         verbose_name_plural = _('Períodos de composição de Comissão')
+        ordering = ['-data_inicio', '-data_fim']
 
     def __str__(self):
         if self.data_inicio and self.data_fim:
@@ -140,6 +141,7 @@ class Composicao(models.Model):  # IGNORE
     class Meta:
         verbose_name = _('Composição de Comissão')
         verbose_name_plural = _('Composições de Comissão')
+        ordering = ['periodo']
 
     def __str__(self):
         return '%s: %s' % (self.comissao.sigla, self.periodo)
diff --git a/sapl/comissoes/views.py b/sapl/comissoes/views.py
index 82c3a79c1..c5870389e 100644
--- a/sapl/comissoes/views.py
+++ b/sapl/comissoes/views.py
@@ -51,6 +51,9 @@ class PeriodoComposicaoCrud(CrudAux):
     class UpdateView(CrudAux.UpdateView):
         form_class = PeriodoForm
 
+    # class ListView(CrudAux.ListView):
+
+
 class ParticipacaoCrud(MasterDetailCrud):
     model = Participacao
     parent_field = 'composicao__comissao'
@@ -112,7 +115,9 @@ class ComposicaoCrud(MasterDetailCrud):
             composicao_pk = self.take_composicao_pk()
 
             if composicao_pk == 0:
-                ultima_composicao = context['composicao_list'].last()
+                # Composicao eh ordenada por Periodo, que por sua vez esta em
+                # ordem descrescente de data de inicio (issue #1920)
+                ultima_composicao = context['composicao_list'].first()
                 if ultima_composicao:
                     context['composicao_pk'] = ultima_composicao.pk
                 else: