Fix #2187 (#2189)

* Fix #2187

* update

* refactoring ifs

* melhoramento da msg

* more pythonic

* update

* Update views.py

sapl/materia/views.py

@@ -530,7 +530,7 @@ class RetornarProposicao(UpdateView):
         if p.autor.user != request.user:
              messages.error(
                  request,
-                 'Usuário sem acesso a esta opção.' % 
+                 'Usuário sem acesso a esta opção.' %
                      request.user)
              return redirect('/')
 
@@ -1805,6 +1805,7 @@ class PrimeiraTramitacaoEmLoteView(PermissionRequiredMixin, FilterView):
 
         return context
 
+
     def post(self, request, *args, **kwargs):
         marcadas = request.POST.getlist('materia_id')
 
@@ -1814,38 +1815,20 @@ class PrimeiraTramitacaoEmLoteView(PermissionRequiredMixin, FilterView):
             msg = _('Nenhuma máteria foi selecionada.')
             messages.add_message(request, messages.ERROR, msg)
             return self.get(request, self.kwargs)
+        obrigatorios = {'data_tramitacao':'Data da Tramitação',
+                        'unidade_tramitacao_local':'Unidade Local',
+                        'unidade_tramitacao_destino':'Unidade Destino',
+                        'status':'Status',
+                        'urgente':'Urgente',
+                        'texto':'Texto da Ação'}
+        for field,nome in obrigatorios.items():
+            if not request.POST[field]:
+                msg = _('Campo {} deve ser preenchido.'.format(nome))
+                messages.add_message(request, messages.ERROR, msg)
+                return self.get(request, self.kwargs)
 
-        if request.POST['status'] == '':
-            msg = _('Campo Status deve ser preenchido.')
-            messages.add_message(request, messages.ERROR, msg)
-            return self.get(request, self.kwargs)
-
-        if request.POST['unidade_tramitacao_local'] == '':
-            msg = _('Campo Unidade Local deve ser preenchido.')
-            messages.add_message(request, messages.ERROR, msg)
-            return self.get(request, self.kwargs)
-
-        if request.POST['data_tramitacao'] == '':
-            msg = _('Campo Data da Tramitação deve ser preenchido.')
-            messages.add_message(request, messages.ERROR, msg)
-            return self.get(request, self.kwargs)
-
-        if request.POST['unidade_tramitacao_destino'] == '':
-            msg = _('Campo Unidade Destino deve ser preenchido.')
-            messages.add_message(request, messages.ERROR, msg)
-            return self.get(request, self.kwargs)
-
-        if request.POST['urgente'] == '':
-            msg = _('Campo Urgente deve ser preenchido.')
-            messages.add_message(request, messages.ERROR, msg)
-            return self.get(request, self.kwargs)
-
-        if request.POST['texto'] == '':
-            msg = _('Campo Texto da Ação deve ser preenchido.')
-            messages.add_message(request, messages.ERROR, msg)
-            return self.get(request, self.kwargs)
 
-        if request.POST['data_encaminhamento'] == '':
+        if not request.POST['data_encaminhamento']:
             data_encaminhamento = None
         else:
             data_encaminhamento = tz.localize(datetime.strptime(
@@ -1878,6 +1861,9 @@ class PrimeiraTramitacaoEmLoteView(PermissionRequiredMixin, FilterView):
                 texto=request.POST['texto']
             )
             t.save()
+            tramitacao_signal.send(sender=Tramitacao,
+                                   post=t,
+                                   request=self.request)
 
         status = StatusTramitacao.objects.get(id=request.POST['status'])