From 8f46abe35ba6249cf4bcf7b5de03fa87a7dee4c5 Mon Sep 17 00:00:00 2001 From: Talitha Pumar Date: Tue, 4 Sep 2018 13:56:05 -0300 Subject: [PATCH] Fix #2187 (#2189) * Fix #2187 * update * refactoring ifs * melhoramento da msg * more pythonic * update * Update views.py --- sapl/materia/views.py | 48 +++++++++++++++---------------------------- 1 file changed, 17 insertions(+), 31 deletions(-) diff --git a/sapl/materia/views.py b/sapl/materia/views.py index 1e4638e73..fa48209e2 100644 --- a/sapl/materia/views.py +++ b/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'])