Fix #2351 (#2352)

sapl/protocoloadm/views.py

@@ -563,19 +563,21 @@ class ProtocoloMostrarView(PermissionRequiredMixin, TemplateView):
         username = self.request.user.username
 
         if protocolo.tipo_materia:
-            try:
+            self.logger.debug(
-                self.logger.debug("user=" + username + ". Tentando obter objeto MateriaLegislativa com numero_protocolo={} e ano={}."
+                "user=" + username + ". Tentando obter objeto MateriaLegislativa com numero_protocolo={} e ano={}."
-                                .format(protocolo.numero, protocolo.ano))
+                .format(protocolo.numero, protocolo.ano))
-                materia = MateriaLegislativa.objects.get(
+            materia = MateriaLegislativa.objects.filter(
-                    numero_protocolo=protocolo.numero, ano=protocolo.ano)
+                numero_protocolo=protocolo.numero, ano=protocolo.ano)
-            except ObjectDoesNotExist:
+            context['materia'] = materia
-                self.logger.error("user=" + username + ". Objeto MateriaLegislativa com numero_protocolo={} e ano={} não encontrado."
+            if len(materia) > 1:
-                                  " Definido como None.".format(protocolo.numero, protocolo.ano))
+                msg = _('Foi encontrada mais de uma matéria com o mesmo número de protocolo e ano.'
-                context['materia'] = None
+                        ' Isso é um erro de uso!'
-            else:
+                        ' Reporte isso ao suporte para que seja corrigido.')
-                self.logger.info("user=" + username + ". Objeto MateriaLegislativa com numero_protocolo={} e ano={} encontrado"
+                messages.add_message(self.request, messages.ERROR, msg)
-                                 "com sucesso.".format(protocolo.numero, protocolo.ano))
+                self.logger.error(
-                context['materia'] = materia
+                    "user=" + username + ". Objeto MateriaLegislativa com numero_protocolo={} e ano={}"
+                                         " encontrado mais de um registro."
+                                         " Erro relatado ao usuário.".format(protocolo.numero, protocolo.ano))
 
         if protocolo.tipo_documento:
             context[

sapl/templates/protocoloadm/protocolo_mostrar.html

@@ -38,9 +38,10 @@
 			<br />
 			{% if not protocolo.anulado %} <a href="{% url 'sapl.protocoloadm:criar_documento' protocolo.pk %}" class="btn btn-primary">Criar Documento</a>{% endif %}
 	{% elif protocolo.tipo_materia %}
-		{% if materia %}
+		{% for m in materia%}
-			<a href="{% url 'sapl.materia:materialegislativa_detail' materia.pk %}"> {{materia}} </a>
+            &nbsp;&nbsp;&nbsp;&nbsp;
-		{% endif %}
+			<a href="{% url 'sapl.materia:materialegislativa_detail' m.pk %}"> {{m}} </a></br>
+		{% endfor %}
 			</br>
 			{% if not protocolo.anulado%}{% if not materia %}<a href="{% url 'sapl.materia:materia_create_simplificado' protocolo.pk %}" class="btn btn-primary">Criar Matéria</a>&nbsp;&nbsp;&nbsp;&nbsp;{% endif %}{% endif %}
 	{% endif %}