fix #2107

sapl/audiencia/forms.py

@@ -16,18 +16,21 @@ class AudienciaForm(forms.ModelForm):
 
     tipo_materia = forms.ModelChoiceField(
         label=_('Tipo Matéria'),
-        required=True,
+        required=False,
         queryset=TipoMateriaLegislativa.objects.all(),
         empty_label='Selecione',
     )
 
     numero_materia = forms.CharField(
-        label='Número Matéria', required=True)
+        label='Número Matéria', required=False)
 
-    ano_materia = forms.CharField(
+    ano_materia = forms.ChoiceField(
         label='Ano Matéria',
-        initial=int(data_atual.year),
-        required=True)
+        required=False)
+
+    materia = forms.ModelChoiceField(required=False,
+                                     widget=forms.HiddenInput(),
+                                     queryset=MateriaLegislativa.objects.all())
 
     class Meta:
         model = AudienciaPublica
@@ -36,7 +39,7 @@ class AudienciaForm(forms.ModelForm):
                   'observacao', 'audiencia_cancelada', 'url_audio',
                   'url_video', 'upload_pauta', 'upload_ata',
                   'upload_anexo', 'tipo_materia', 'numero_materia',
-                  'ano_materia']
+                  'ano_materia', 'materia']
 
 
     def __init__(self, **kwargs):
@@ -59,11 +62,16 @@ class AudienciaForm(forms.ModelForm):
         if not self.is_valid():
             return cleaned_data
 
+        materia = cleaned_data['numero_materia']
+        ano_materia = cleaned_data['ano_materia']
+        tipo_materia = cleaned_data['tipo_materia']
+
+        if materia and ano_materia and tipo_materia:
             try:
                 materia = MateriaLegislativa.objects.get(
-                numero=self.cleaned_data['numero_materia'],
-                ano=self.cleaned_data['ano_materia'],
-                tipo=self.cleaned_data['tipo_materia'])
+                    numero=materia,
+                    ano=ano_materia,
+                    tipo=tipo_materia)
             except ObjectDoesNotExist:
                 msg = _('A matéria a ser inclusa não existe no cadastro'
                         ' de matérias legislativas.')
@@ -71,6 +79,22 @@ class AudienciaForm(forms.ModelForm):
             else:
                 cleaned_data['materia'] = materia
 
+        elif ((materia and not (ano_materia or tipo_materia)) or
+              (ano_materia and not (tipo_materia or materia)) or
+              (tipo_materia and not (materia or ano_materia))):
+            msg = _('Preencha todos os campos relacionados à Matéria Legislativa')
+            raise ValidationError(msg)
+
+        if not cleaned_data['numero']:
+            try:
+                ultima_audiencia = AudienciaPublica.objects.filter().order_by('numero').last()
+                cleaned_data['numero'] = ultima_audiencia.numero + 1
+
+            except ObjectDoesNotExist:
+                cleaned_data['numero'] = 1
+
+
+
         if self.cleaned_data['hora_inicio'] and self.cleaned_data['hora_fim']:
             if (self.cleaned_data['hora_fim'] <
                 self.cleaned_data['hora_inicio']):
@@ -78,10 +102,3 @@ class AudienciaForm(forms.ModelForm):
                     raise ValidationError(msg)
 
         return cleaned_data
-
-    @transaction.atomic()
-    def save(self, commit=True):
-        audiencia = super(AudienciaForm, self).save(False)
-        audiencia.materia = self.cleaned_data['materia']
-        audiencia.save()
-        return audiencia

sapl/audiencia/migrations/0005_auto_20180806_1236.py

@@ -0,0 +1,20 @@
+# -*- coding: utf-8 -*-
+# Generated by Django 1.9.13 on 2018-08-06 15:36
+from __future__ import unicode_literals
+
+from django.db import migrations, models
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ('audiencia', '0004_auto_20180305_1006'),
+    ]
+
+    operations = [
+        migrations.AlterField(
+            model_name='audienciapublica',
+            name='hora_fim',
+            field=models.CharField(blank=True, max_length=5, null=True, verbose_name='Horário Fim(hh:mm)'),
+        ),
+    ]

sapl/audiencia/models.py

@@ -71,7 +71,7 @@ class AudienciaPublica(models.Model):
     hora_inicio = models.CharField(
         max_length=5, verbose_name=_('Horário Início(hh:mm)'))
     hora_fim = models.CharField(
-        max_length=5, verbose_name=_('Horário Fim(hh:mm)'))
+        max_length=5, blank=True, null=True, verbose_name=_('Horário Fim(hh:mm)'))
     observacao = models.TextField(
         max_length=500, blank=True, verbose_name=_('Observação'))
     audiencia_cancelada = models.BooleanField(

sapl/audiencia/views.py

@@ -6,15 +6,17 @@ from sapl.crud.base import RP_DETAIL, RP_LIST, Crud
 from .forms import AudienciaForm
 from .models import AudienciaPublica
 
+
 def index(request):
     return HttpResponse("Audiência  Pública")
 
+
 class AudienciaCrud(Crud):
     model = AudienciaPublica
     public = [RP_LIST, RP_DETAIL, ]
 
     class BaseMixin(Crud.BaseMixin):
-        list_field_names = ['materia', 'tipo', 'numero', 'nome',
+        list_field_names = ['numero', 'nome', 'tipo', 'materia',
                             'data']
         ordering = 'nome', 'numero', 'tipo', 'data'