Fix #1757 (#1764)

sapl/sessao/views.py

@@ -154,9 +154,10 @@ def abrir_votacao(request, pk, spk):
         reverse('sapl.sessao:' + redirect_url, kwargs={'pk': spk}))
 
 
-def customize_link_materia(context, pk):
+def customize_link_materia(context, pk, has_permission, is_expediente):
     for i, row in enumerate(context['rows']):
         materia = context['object_list'][i].materia
+        obj = context['object_list'][i]
         url_materia = reverse('sapl.materia:materialegislativa_detail',
                               kwargs={'pk': materia.id})
         numeracao = materia.numeracao_set.first()
@@ -199,80 +200,32 @@ def customize_link_materia(context, pk):
         # Na linha abaixo, o segundo argumento é None para não colocar
         # url em toda a string de title_materia
         context['rows'][i][1] = (title_materia, None)
-    return context
-
-
-def get_presencas_generic(model, sessao, legislatura):
-    presencas = model.objects.filter(
-        sessao_plenaria=sessao)
-
-    presentes = [p.parlamentar for p in presencas]
-
-    mandato = Mandato.objects.filter(
-        legislatura=legislatura).order_by('parlamentar__nome_parlamentar')
-
-    for m in mandato:
-        if m.parlamentar in presentes:
-            yield (m.parlamentar, True)
-        else:
-            yield (m.parlamentar, False)
-
-
-class MateriaOrdemDiaCrud(MasterDetailCrud):
-    model = OrdemDia
-    parent_field = 'sessao_plenaria'
-    help_topic = 'sessao_plenaria_materias_ordem_dia'
-    public = [RP_LIST, RP_DETAIL]
-
-    class BaseMixin(MasterDetailCrud.BaseMixin):
-        list_field_names = ['numero_ordem', 'materia', 'materia__ementa',
-                            'resultado']
-
-    class CreateView(MasterDetailCrud.CreateView):
-        form_class = OrdemDiaForm
-
-        def get_initial(self):
-            self.initial['data_ordem'] = SessaoPlenaria.objects.get(
-                pk=self.kwargs['pk']).data_inicio.strftime('%d/%m/%Y')
-            max_numero_ordem = OrdemDia.objects.filter(
-                sessao_plenaria=self.kwargs['pk']).aggregate(
-                    Max('numero_ordem'))['numero_ordem__max']
-            self.initial['numero_ordem'] = (
-                max_numero_ordem if max_numero_ordem else 0) + 1
-            return self.initial
-
-        def get_success_url(self):
-            return reverse('sapl.sessao:ordemdia_list',
-                           kwargs={'pk': self.kwargs['pk']})
-
-    class UpdateView(MasterDetailCrud.UpdateView):
-        form_class = OrdemDiaForm
-
-        def get_initial(self):
-            self.initial['tipo_materia'] = self.object.materia.tipo.id
-            self.initial['numero_materia'] = self.object.materia.numero
-            self.initial['ano_materia'] = self.object.materia.ano
-            return self.initial
-
-    class DetailView(MasterDetailCrud.DetailView):
 
-        layout_key = 'OrdemDiaDetail'
-
-    class ListView(MasterDetailCrud.ListView):
-        paginate_by = None
-        ordering = ['numero_ordem', 'materia', 'resultado']
-
-        def get_context_data(self, **kwargs):
-            context = super().get_context_data(**kwargs)
-            return customize_link_materia(context, self.kwargs['pk'])
-
-        def get_rows(self, object_list):
-            for obj in object_list:
         exist_resultado = obj.registrovotacao_set.filter(
             materia=obj.materia).exists()
         if not exist_resultado:
             if obj.votacao_aberta:
                 url = ''
+                if is_expediente:
+                    if obj.tipo_votacao == 1:
+                        url = reverse('sapl.sessao:votacaosimbolicaexp',
+                                      kwargs={
+                                          'pk': obj.sessao_plenaria_id,
+                                          'oid': obj.pk,
+                                          'mid': obj.materia_id})
+                    elif obj.tipo_votacao == 2:
+                        url = reverse('sapl.sessao:votacaonominalexp',
+                                      kwargs={
+                                          'pk': obj.sessao_plenaria_id,
+                                          'oid': obj.pk,
+                                          'mid': obj.materia_id})
+                    elif obj.tipo_votacao == 3:
+                        url = reverse('sapl.sessao:votacaosecretaexp',
+                                      kwargs={
+                                          'pk': obj.sessao_plenaria_id,
+                                          'oid': obj.pk,
+                                          'mid': obj.materia_id})
+                else:
                     if obj.tipo_votacao == 1:
                         url = reverse('sapl.sessao:votacaosimbolica',
                                       kwargs={
@@ -291,38 +244,67 @@ class MateriaOrdemDiaCrud(MasterDetailCrud):
                                           'pk': obj.sessao_plenaria_id,
                                           'oid': obj.pk,
                                           'mid': obj.materia_id})
-                        if self.request.user.has_module_perms(AppConfig.label):
+                if has_permission:
                     btn_registrar = '''
-                            <a href="%s"
+                                    <form action="%s">
-                               class="btn btn-primary"
+                                     <input type="submit" class="btn btn-primary"
-                               role="button">Registrar Votação</a>''' % (
+                                      value="Registrar Votação" />
+                                 </form>''' % (
                         url)
-                            obj.resultado = btn_registrar
+
+                    resultado = btn_registrar
                 else:
-                            obj.resultado = '''Não há resultado'''
+                    resultado = '''Não há resultado'''
+            else:
+                if is_expediente:
+                    url = reverse('sapl.sessao:abrir_votacao', kwargs={
+                        'pk': obj.pk,
+                        'spk': obj.sessao_plenaria_id
+                    }) + '?tipo_materia=expediente'
                 else:
                     url = reverse('sapl.sessao:abrir_votacao', kwargs={
                         'pk': obj.pk,
                         'spk': obj.sessao_plenaria_id
                     }) + '?tipo_materia=ordem'
 
-                        if self.request.user.has_module_perms(AppConfig.label):
+                if has_permission:
                     btn_abrir = '''
                                         Matéria não votada<br />
                                         <a href="%s"
                                            class="btn btn-primary"
                                            role="button">Abrir Votação</a>''' % (url)
-                            obj.resultado = btn_abrir
+                    resultado = btn_abrir
                 else:
-                            obj.resultado = '''Não há resultado'''
+                    resultado = '''Não há resultado'''
         else:
             resultado = obj.registrovotacao_set.get(
                 materia_id=obj.materia_id)
             resultado_descricao = resultado.tipo_resultado_votacao.nome
             resultado_observacao = resultado.observacao
 
-                    if self.request.user.has_module_perms(AppConfig.label):
+            if has_permission:
                 url = ''
+                if is_expediente:
+                    if obj.tipo_votacao == 1:
+                        url = reverse(
+                            'sapl.sessao:votacaosimbolicaexpedit',
+                            kwargs={
+                                'pk': obj.sessao_plenaria_id,
+                                'oid': obj.pk,
+                                'mid': obj.materia_id})
+                    elif obj.tipo_votacao == 2:
+                        url = reverse('sapl.sessao:votacaonominalexpedit',
+                                      kwargs={
+                                          'pk': obj.sessao_plenaria_id,
+                                          'oid': obj.pk,
+                                          'mid': obj.materia_id})
+                    elif obj.tipo_votacao == 3:
+                        url = reverse('sapl.sessao:votacaosecretaexpedit',
+                                      kwargs={
+                                          'pk': obj.sessao_plenaria_id,
+                                          'oid': obj.pk,
+                                          'mid': obj.materia_id})
+                else:
                     if obj.tipo_votacao == 1:
                         url = reverse('sapl.sessao:votacaosimbolicaedit',
                                       kwargs={
@@ -341,12 +323,22 @@ class MateriaOrdemDiaCrud(MasterDetailCrud):
                                           'pk': obj.sessao_plenaria_id,
                                           'oid': obj.pk,
                                           'mid': obj.materia_id})
-                        obj.resultado = ('<a href="%s">%s<br/>%s</a>' %
+
+                resultado = ('<a href="%s">%s<br/>%s</a>' %
                                  (url,
                                   resultado_descricao,
                                   resultado_observacao))
             else:
                 if obj.tipo_votacao == 2:
+                    if is_expediente:
+                        url = reverse(
+                            'sapl.sessao:votacao_nominal_transparencia',
+                            kwargs={
+                                'pk': obj.sessao_plenaria_id,
+                                'oid': obj.pk,
+                                'mid': obj.materia_id}) + \
+                              '?&materia=expediente'
+                    else:
                         url = reverse(
                             'sapl.sessao:votacao_nominal_transparencia',
                             kwargs={
@@ -354,17 +346,82 @@ class MateriaOrdemDiaCrud(MasterDetailCrud):
                                 'oid': obj.pk,
                                 'mid': obj.materia_id}) + \
                               '?&materia=ordem'
-                            obj.resultado = ('<a href="%s">%s<br/>%s</a>' %
+
+                    resultado = ('<a href="%s">%s<br/>%s</a>' %
                                      (url,
                                       resultado_descricao,
                                       resultado_observacao))
                 else:
-                            obj.resultado = ('%s<br/>%s' %
+                    resultado = ('%s<br/>%s' %
                                      (resultado_descricao,
                                       resultado_observacao))
+        context['rows'][i][3] = (resultado, None)
+    return context
 
-            return [self._as_row(obj) for obj in object_list]
 
+def get_presencas_generic(model, sessao, legislatura):
+    presencas = model.objects.filter(
+        sessao_plenaria=sessao)
+
+    presentes = [p.parlamentar for p in presencas]
+
+    mandato = Mandato.objects.filter(
+        legislatura=legislatura).order_by('parlamentar__nome_parlamentar')
+
+    for m in mandato:
+        if m.parlamentar in presentes:
+            yield (m.parlamentar, True)
+        else:
+            yield (m.parlamentar, False)
+
+
+class MateriaOrdemDiaCrud(MasterDetailCrud):
+    model = OrdemDia
+    parent_field = 'sessao_plenaria'
+    help_topic = 'sessao_plenaria_materias_ordem_dia'
+    public = [RP_LIST, RP_DETAIL]
+
+    class BaseMixin(MasterDetailCrud.BaseMixin):
+        list_field_names = ['numero_ordem', 'materia', 'materia__ementa',
+                            'resultado']
+
+    class CreateView(MasterDetailCrud.CreateView):
+        form_class = OrdemDiaForm
+
+        def get_initial(self):
+            self.initial['data_ordem'] = SessaoPlenaria.objects.get(
+                pk=self.kwargs['pk']).data_inicio.strftime('%d/%m/%Y')
+            max_numero_ordem = OrdemDia.objects.filter(
+                sessao_plenaria=self.kwargs['pk']).aggregate(
+                    Max('numero_ordem'))['numero_ordem__max']
+            self.initial['numero_ordem'] = (
+                max_numero_ordem if max_numero_ordem else 0) + 1
+            return self.initial
+
+        def get_success_url(self):
+            return reverse('sapl.sessao:ordemdia_list',
+                           kwargs={'pk': self.kwargs['pk']})
+
+    class UpdateView(MasterDetailCrud.UpdateView):
+        form_class = OrdemDiaForm
+
+        def get_initial(self):
+            self.initial['tipo_materia'] = self.object.materia.tipo.id
+            self.initial['numero_materia'] = self.object.materia.numero
+            self.initial['ano_materia'] = self.object.materia.ano
+            return self.initial
+
+    class DetailView(MasterDetailCrud.DetailView):
+
+        layout_key = 'OrdemDiaDetail'
+
+    class ListView(MasterDetailCrud.ListView):
+        paginate_by = None
+        ordering = ['numero_ordem', 'materia', 'resultado']
+        def get_context_data(self, **kwargs):
+            context = super().get_context_data(**kwargs)
+            has_permition = self.request.user.has_module_perms(AppConfig.label)
+            return customize_link_materia(context, self.kwargs['pk'], has_permition, False)
 
 def recuperar_materia(request):
     tipo = TipoMateriaLegislativa.objects.get(pk=request.GET['tipo_materia'])
@@ -399,106 +456,8 @@ class ExpedienteMateriaCrud(MasterDetailCrud):
 
         def get_context_data(self, **kwargs):
             context = super().get_context_data(**kwargs)
-
+            has_permition = self.request.user.has_module_perms(AppConfig.label)
-            return customize_link_materia(context, self.kwargs['pk'])
+            return customize_link_materia(context, self.kwargs['pk'], has_permition, True)
-
-        def get_rows(self, object_list):
-            for obj in object_list:
-                exist_resultado = obj.registrovotacao_set.filter(
-                    materia=obj.materia).exists()
-
-                if not exist_resultado:
-                    if obj.votacao_aberta:
-                        url = ''
-                        if obj.tipo_votacao == 1:
-                            url = reverse('sapl.sessao:votacaosimbolicaexp',
-                                          kwargs={
-                                              'pk': obj.sessao_plenaria_id,
-                                              'oid': obj.pk,
-                                              'mid': obj.materia_id})
-                        elif obj.tipo_votacao == 2:
-                            url = reverse('sapl.sessao:votacaonominalexp',
-                                          kwargs={
-                                              'pk': obj.sessao_plenaria_id,
-                                              'oid': obj.pk,
-                                              'mid': obj.materia_id})
-                        elif obj.tipo_votacao == 3:
-                            url = reverse('sapl.sessao:votacaosecretaexp',
-                                          kwargs={
-                                              'pk': obj.sessao_plenaria_id,
-                                              'oid': obj.pk,
-                                              'mid': obj.materia_id})
-
-                        if self.request.user.has_module_perms(AppConfig.label):
-                            btn_registrar = '''
-                                <form action="%s">
-                                    <input type="submit" class="btn btn-primary"
-                                     value="Registrar Votação" />
-                                </form>
-                                ''' % (url)
-                            obj.resultado = btn_registrar
-                    else:
-                        url = reverse('sapl.sessao:abrir_votacao', kwargs={
-                            'pk': obj.pk,
-                            'spk': obj.sessao_plenaria_id
-                        }) + '?tipo_materia=expediente'
-                        btn_abrir = '''Matéria não votada<br />'''
-
-                        if self.request.user.has_module_perms(AppConfig.label):
-                            btn_abrir += '''
-                            <a href="%s"
-                               class="btn btn-primary"
-                               role="button">Abrir Votação</a>''' % (url)
-
-                        obj.resultado = btn_abrir
-                else:
-                    url = ''
-                    resultado = obj.registrovotacao_set.get(
-                        materia_id=obj.materia_id)
-                    resultado_descricao = resultado.tipo_resultado_votacao.nome
-                    resultado_observacao = resultado.observacao
-                    if self.request.user.has_module_perms(AppConfig.label):
-                        if obj.tipo_votacao == 1:
-                            url = reverse(
-                                'sapl.sessao:votacaosimbolicaexpedit',
-                                kwargs={
-                                    'pk': obj.sessao_plenaria_id,
-                                    'oid': obj.pk,
-                                    'mid': obj.materia_id})
-                        elif obj.tipo_votacao == 2:
-                            url = reverse('sapl.sessao:votacaonominalexpedit',
-                                          kwargs={
-                                              'pk': obj.sessao_plenaria_id,
-                                              'oid': obj.pk,
-                                              'mid': obj.materia_id})
-                        elif obj.tipo_votacao == 3:
-                            url = reverse('sapl.sessao:votacaosecretaexpedit',
-                                          kwargs={
-                                              'pk': obj.sessao_plenaria_id,
-                                              'oid': obj.pk,
-                                              'mid': obj.materia_id})
-                        obj.resultado = ('<a href="%s">%s<br/>%s</a>' %
-                                         (url,
-                                          resultado_descricao,
-                                          resultado_observacao))
-                    else:
-                        if obj.tipo_votacao == 2:
-                            url = reverse(
-                                'sapl.sessao:votacao_nominal_transparencia',
-                                kwargs={
-                                    'pk': obj.sessao_plenaria_id,
-                                    'oid': obj.pk,
-                                    'mid': obj.materia_id}) +\
-                                '?&materia=expediente'
-                            obj.resultado = ('<a href="%s">%s<br/>%s</a>' %
-                                             (url,
-                                              resultado_descricao,
-                                              resultado_observacao))
-                        else:
-                            obj.resultado = ('%s<br/>%s' %
-                                             (resultado_descricao,
-                                              resultado_observacao))
-            return [self._as_row(obj) for obj in object_list]
 
     class CreateView(MasterDetailCrud.CreateView):
         form_class = ExpedienteMateriaForm

sapl/templates/sessao/ordemdia_list.html

@@ -45,6 +45,10 @@
             setTimeout(function(){ window.location.reload(true) }, 500);
         }
      });
+    $(window).on('beforeunload', function () {
+            $('tbody').sortable('disable');
+            $("input[type=submit], input[type=button]").prop("disabled", "disabled");
+     });
     </script>
 
 {% endblock %}