Adiciona correções

sapl/comissoes/forms.py

@@ -81,7 +81,7 @@ class ComposicaoForm(forms.ModelForm):
 
 def choices_posicao():
     posicoes = list(range(1, len(CargoComissao.objects.all())+1))
-    choices = [(posicao, f"{posicao}º") for posicao in posicoes]
+    choices = [(posicao, str(posicao)+"º") for posicao in posicoes]
     choices.insert(0, (0, "Não definida"))
     return choices
 
@@ -130,7 +130,7 @@ class CargosComissaoOrdenacaoForm(forms.Form):
         posicao = cleaned_data['posicao']
 
         if posicao == '0':
-            cargo.id_ordenacao = "Não definido"
+            cargo.id_ordenacao = None
         else:
             if CargoComissao.objects.filter(id_ordenacao=posicao).exists():
                 outro_cargo = CargoComissao.objects.get(id_ordenacao=posicao)

sapl/comissoes/migrations/0024_auto_20191204_2004.py → sapl/comissoes/migrations/0023_auto_20191210_1532.py

@@ -1,14 +1,14 @@
 # -*- coding: utf-8 -*-
-# Generated by Django 1.11.20 on 2019-12-04 23:04
+# Generated by Django 1.11.20 on 2019-12-10 18:32
 from __future__ import unicode_literals
 
-from django.db import migrations
+from django.db import migrations, models
 
 
 class Migration(migrations.Migration):
 
     dependencies = [
-        ('comissoes', '0023_cargocomissao_id_ordenacao'),
+        ('comissoes', '0022_auto_20191120_1440'),
     ]
 
     operations = [
@@ -20,4 +20,9 @@ class Migration(migrations.Migration):
             name='participacao',
             options={'ordering': ['-titular', 'cargo__id_ordenacao'], 'verbose_name': 'Participação em Comissão', 'verbose_name_plural': 'Participações em Comissão'},
         ),
+        migrations.AddField(
+            model_name='cargocomissao',
+            name='id_ordenacao',
+            field=models.PositiveIntegerField(blank=True, null=True, verbose_name='Id de Ordenacao'),
+        ),
     ]

sapl/comissoes/migrations/0023_cargocomissao_id_ordenacao.py

@@ -1,20 +0,0 @@
-# -*- coding: utf-8 -*-
-# Generated by Django 1.11.20 on 2019-12-02 22:46
-from __future__ import unicode_literals
-
-from django.db import migrations, models
-
-
-class Migration(migrations.Migration):
-
-    dependencies = [
-        ('comissoes', '0022_auto_20191120_1440'),
-    ]
-
-    operations = [
-        migrations.AddField(
-            model_name='cargocomissao',
-            name='id_ordenacao',
-            field=models.CharField(default='Não definido', max_length=50, verbose_name='Id de Ordenacao'),
-        ),
-    ]

sapl/comissoes/models.py

@@ -121,12 +121,8 @@ class Periodo(models.Model):  # PeriodoCompComissao
 
 @reversion.register()
 class CargoComissao(models.Model):
-    # O CharField é utilizado para evitar definir um valor máximo de
-    # cargos que podem ser criados e sempre apresentar os cargos não
-    # ordenados por último.
-    id_ordenacao = models.CharField(
-        max_length=50, verbose_name=_('Id de Ordenacao'),
-        default='Não definido'
+    id_ordenacao = models.PositiveIntegerField(
+        blank=True, null=True, verbose_name=_('Id de Ordenacao'),
     )
     nome = models.CharField(max_length=50, verbose_name=_('Cargo'))
     unico = models.BooleanField(

sapl/comissoes/views.py

@@ -117,7 +117,7 @@ class CargosComissaoOrdenacaoView(PermissionRequiredMixin, FormView):
             else:
                 tabela_ordenacao.append((posicao, ""))
 
-        cargos_desordenados = cargos.filter(id_ordenacao="Não definido")
+        cargos_desordenados = cargos.filter(id_ordenacao=None)
         for cargo_desordenado in cargos_desordenados:
             tabela_ordenacao.append(("Não definida", cargo_desordenado))