Adiciona USER, IP e DATETIME da última edição em Proposição (#2908)

* Adicionar novos campos no model Proposicao

* Adicionar migrações

* Adicionar novos widgets em ProposicaoForm

* Salvar e apresentar dados - última edição

* Identificar e salvar timezone

* Salvar e apresentar dados - criação

sapl/materia/forms.py

@@ -1826,12 +1826,19 @@ class ProposicaoForm(FileFieldCheckMixin, forms.ModelForm):
                   'ano_materia',
                   'tipo_texto',
                   'hash_code',
-                  'numero_materia_futuro']
+                  'numero_materia_futuro',
+                  'user',
+                  'ip',
+                  'ultima_edicao']
 
         widgets = {
             'descricao': widgets.Textarea(attrs={'rows': 4}),
             'tipo': TipoProposicaoSelect(),
-            'hash_code': forms.HiddenInput(), }
+            'hash_code': forms.HiddenInput(),
+            'user': forms.HiddenInput(),
+            'ip': forms.HiddenInput(),
+            'ultima_edicao': forms.HiddenInput()
+        }
 
     def __init__(self, *args, **kwargs):
         self.texto_articulado_proposicao = AppConfig.attr(

sapl/materia/migrations/0052_auto_20190731_1554.py

@@ -0,0 +1,28 @@
+# -*- coding: utf-8 -*-
+# Generated by Django 1.11.20 on 2019-07-31 18:54
+from __future__ import unicode_literals
+
+from django.conf import settings
+from django.db import migrations, models
+import django.db.models.deletion
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        migrations.swappable_dependency(settings.AUTH_USER_MODEL),
+        ('materia', '0051_auto_20190703_1414'),
+    ]
+
+    operations = [
+        migrations.AddField(
+            model_name='proposicao',
+            name='ip',
+            field=models.CharField(blank=True, default='', max_length=30, verbose_name='IP'),
+        ),
+        migrations.AddField(
+            model_name='proposicao',
+            name='user',
+            field=models.ForeignKey(blank=True, null=True, on_delete=django.db.models.deletion.PROTECT, to=settings.AUTH_USER_MODEL, verbose_name='Usuário'),
+        ),
+    ]

sapl/materia/migrations/0053_proposicao_ultima_edicao.py

@@ -0,0 +1,20 @@
+# -*- coding: utf-8 -*-
+# Generated by Django 1.11.20 on 2019-07-31 22:26
+from __future__ import unicode_literals
+
+from django.db import migrations, models
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ('materia', '0052_auto_20190731_1554'),
+    ]
+
+    operations = [
+        migrations.AddField(
+            model_name='proposicao',
+            name='ultima_edicao',
+            field=models.DateTimeField(blank=True, null=True, verbose_name='Data e Hora da Edição'),
+        ),
+    ]

sapl/materia/models.py

@@ -833,6 +833,24 @@ class Proposicao(models.Model):
     documento_gerado = models.ForeignKey(
         DocumentoAcessorio, blank=True, null=True)"""
 
+    user = models.ForeignKey(
+        get_settings_auth_user_model(),
+        verbose_name=_('Usuário'),
+        on_delete=models.PROTECT,
+        null=True,
+        blank=True
+    )
+    ip = models.CharField(
+        verbose_name=_('IP'),
+        max_length=30,
+        blank=True,
+        default=''
+    )
+    ultima_edicao = models.DateTimeField(
+        verbose_name=_('Data e Hora da Edição'),
+        blank=True, null=True
+    )
+
     @property
     def perfis(self):
         return self.tipo.perfis.all()

sapl/materia/views.py

@@ -772,6 +772,11 @@ class ProposicaoCrud(Crud):
 
             context['title'] = '%s <small>(%s)</small>' % (
                 self.object, self.object.autor)
+
+            context['user'] = self.request.user
+            context['proposicao'] = Proposicao.objects.get(
+                pk=self.kwargs['pk']
+            )
             return context
 
         def get(self, request, *args, **kwargs):
@@ -940,6 +945,40 @@ class ProposicaoCrud(Crud):
     class UpdateView(BaseLocalMixin, Crud.UpdateView):
 
         logger = logging.getLogger(__name__)
+        form_class = ProposicaoForm
+
+        def form_valid(self, form):
+            tz = timezone.get_current_timezone()
+
+            objeto_antigo = Proposicao.objects.get(
+                pk=self.kwargs['pk']
+            )
+            dict_objeto_antigo = objeto_antigo.__dict__
+
+            tipo_texto = self.request.POST.get('tipo_texto', '')
+            if tipo_texto=='D' and objeto_antigo.texto_articulado.exists() or tipo_texto=='T' and not objeto_antigo.texto_articulado.exists():
+                self.object.user = self.request.user
+                self.object.ip = get_client_ip(self.request)
+                self.object.ultima_edicao = tz.localize(datetime.now())
+                self.object.save()
+
+            self.object = form.save()
+            dict_objeto_novo = self.object.__dict__
+
+            atributos = [
+                'tipo_id', 'descricao', 'observacao', 'texto_original',
+                'materia_de_vinculo_id'
+            ]
+
+            for atributo in atributos:
+                if dict_objeto_antigo[atributo] != dict_objeto_novo[atributo]:
+                    self.object.user = self.request.user
+                    self.object.ip = get_client_ip(self.request)
+                    self.object.ultima_edicao = tz.localize(datetime.now())
+                    self.object.save()
+                    break
+            
+            return super().form_valid(form)
 
         def _action_is_valid(self, request, *args, **kwargs):
 
@@ -994,6 +1033,17 @@ class ProposicaoCrud(Crud):
         form_class = ProposicaoForm
         layout_key = None
 
+        def get_initial(self):
+            initial = super().get_initial()
+
+            initial['user'] = self.request.user
+            initial['ip'] = get_client_ip(self.request)
+            
+            tz = timezone.get_current_timezone()
+            initial['ultima_edicao'] = tz.localize(datetime.now())
+
+            return initial
+
         def get_context_data(self, **kwargs):
             context = super().get_context_data(**kwargs)
             context['subnav_template_name'] = ''

sapl/templates/materia/proposicao_detail.html

@@ -1,8 +1,8 @@
 {% extends "crud/detail.html" %}
 {% load i18n common_tags %}
 {% load tz %}
-{% block sub_actions %}{{block.super}}
+{% block sub_actions %}
-
+  {{block.super}}
   <div class="actions btn-group btn-group-sm {%block sub_actions_pull%}{% endblock%}" role="group">
     {% if object.texto_articulado.exists %}
       <a class="btn btn-success" href="{% url 'sapl.materia:proposicao_ta' object.pk%}">{% trans "Texto Eletrônico" %}</a>
@@ -12,9 +12,7 @@
     {% endif %}
   </div>
 {% endblock sub_actions%}
-
 {% block editions %}
-
   {% if object.data_envio %}
     {% if user == object.autor.user  %}
       {% block editions_actions_return %}
@@ -26,161 +24,183 @@
         </div>
       {% endblock %}
     {% endif %}
-
   {% else %}
-
     {% block editions_actions_send %}
       <div class="actions btn-group" role="group">
         <a href="{{ view.detail_url }}?action=send" class="btn btn-primary">{% trans 'Enviar' %}</a>
       </div>
-
       <div class="actions btn-group" role="group">
         <a href="{{ view.update_url }}" class="btn btn-outline-primary">{% trans 'Editar' %}</a>
         <a href="{{ view.delete_url }}" class="btn btn-outline-primary btn-outline-danger">{% trans 'Excluir' %}</a>
       </div>
     {% endblock %}
   {% endif %}
-
 {% endblock editions %}
-
-
 {% block detail_content %}
-
-
   <h2 class="legend">{% model_verbose_name 'sapl.materia.models.Proposicao' %}</h2>
-
   <div class="row">
-
+    <div class="col-sm-3">
-      <div class="col-sm-3">
+      <div id="div_id_tipo" class="form-group">
-        <div id="div_id_tipo" class="form-group">
+        <p class="control-label">{%field_verbose_name object 'tipo'%}</p>
-          <p class="control-label">{%field_verbose_name object 'tipo'%}</p>
+        <div class="controls">
-          <div class="controls">
+          <div class="form-control-static">{{object.tipo}}</div>
-              <div class="form-control-static">{{object.tipo}}</div>
-          </div>
         </div>
       </div>
-      {% if object.data_devolucao %}
+    </div>
-
+    {% if object.data_devolucao %}
-        <div class="col-sm-9">
+      <div class="col-sm-9">
-          <div class="alert alert-danger alert-dismissible " role="alert">
+        <div class="alert alert-danger alert-dismissible " role="alert">
-            <strong>{% trans "Proposição devolvida em:" %} {{ object.data_devolucao}}</strong>
+          <strong>{% trans "Proposição devolvida em:" %} {{ object.data_devolucao}}</strong>
-            <div >{% trans "Justificativa:" %} {{object.justificativa_devolucao}}</div>
+          <div >{% trans "Justificativa:" %} {{object.justificativa_devolucao}}</div>
-          </div>
         </div>
-
+      </div>
-      {% else %}
+    {% else %}
-
+      {% if object.data_envio %}
-        {% if object.data_envio %}
+        <div class="col-sm-3">
-          <div class="col-sm-3">
+          <div id="div_id_data_envio" class="form-group">
-            <div id="div_id_data_envio" class="form-group">
             <p class="control-label">{%field_verbose_name object 'data_envio' %}</p>
-              <div class="controls">
+            <div class="controls">
-                  <div class="form-control-static">{{object.data_envio}}</div>
+              <div class="form-control-static">{{object.data_envio}}</div>
-              </div>
             </div>
           </div>
-        {% endif %}
+        </div>
-
+      {% endif %}
-        {% if object.data_recebimento %}
+      {% if object.data_recebimento %}
-          <div class="col-sm-3">
+        <div class="col-sm-3">
-            <div id="div_id_data_envio" class="form-group">
+          <div id="div_id_data_envio" class="form-group">
             <p class="control-label">{%field_verbose_name object 'data_recebimento'%}</p>
-              <div class="controls">
+            <div class="controls">
-                  <div class="form-control-static">{{object.data_recebimento}}</div>
+              <div class="form-control-static">{{object.data_recebimento}}</div>
-              </div>
             </div>
           </div>
-        {% elif object.data_envio %}
+        </div>
-          <div class="col-sm-6">
+      {% elif object.data_envio %}
-            <div class="alert alert-info alert-dismissible " role="alert">
+        <div class="col-sm-6">
-              <div >{% trans "Proposição aguardando recebimento" %}</div>
+          <div class="alert alert-info alert-dismissible " role="alert">
-            </div>
+            <div >{% trans "Proposição aguardando recebimento" %}</div>
           </div>
-        {% endif %}
+        </div>
       {% endif %}
+    {% endif %}
   </div>
-
   <div class="row">
-      <div class="col-sm-12">
+    <div class="col-sm-12">
-        <div id="div_id_descricao" class="form-group">
+      <div id="div_id_descricao" class="form-group">
-          <p class="control-label">{%field_verbose_name object 'descricao'%}</p>
+        <p class="control-label">{%field_verbose_name object 'descricao'%}</p>
-          <div class="controls">
+        <div class="controls">
-              <div class="form-control-static">{{object.descricao}}</div>
+          <div class="form-control-static">{{object.descricao}}</div>
-          </div>
         </div>
       </div>
+    </div>
   </div>
-
-
   {% if object.observacao %}
     <div class="row">
-        <div class="col-sm-12">
+      <div class="col-sm-12">
-          <div id="div_id_obseracao" class="form-group">
+        <div id="div_id_obseracao" class="form-group">
-            <p class="control-label">{%field_verbose_name object 'observacao'%}</p>
+          <p class="control-label">{%field_verbose_name object 'observacao'%}</p>
-            <div class="controls">
+          <div class="controls">
-                <div class="form-control-static">{{object.observacao}}</div>
+            <div class="form-control-static">{{object.observacao}}</div>
-            </div>
           </div>
         </div>
+      </div>
     </div>
   {% endif %}
-
   <div class="row">
     {% if object.conteudo_gerado_related %}
-        <div class="col-sm-6">
+      <div class="col-sm-6">
-          <h2 class="legend">{% trans "Conteúdo Gerado" %}</h2>
+        <h2 class="legend">{% trans "Conteúdo Gerado" %}</h2>
-          <div id="div_id_conteudo_gerado_related" class="form-group">
+        <div id="div_id_conteudo_gerado_related" class="form-group">
-            <div class="controls">
+          <div class="controls">
-                <a href="{% url object|urldetail_content_type:object.conteudo_gerado_related object.conteudo_gerado_related.id%}">
+            <a href="{% url object|urldetail_content_type:object.conteudo_gerado_related object.conteudo_gerado_related.id%}">
-                  {{object.conteudo_gerado_related}}
+              {{object.conteudo_gerado_related}}
-                </a>
+            </a>
-            </div>
           </div>
         </div>
+      </div>
     {% endif %}
     {% if object.materia_de_vinculo %}
-        <div class="col-sm-6">
+      <div class="col-sm-6">
-          <h2 class="legend">{% trans "Vínculo com a Matéria Legislativa" %}</h2>
+        <h2 class="legend">{% trans "Vínculo com a Matéria Legislativa" %}</h2>
-          <div id="div_id_materia_de_vinculo" class="form-group">
+        <div id="div_id_materia_de_vinculo" class="form-group">
-            <div class="controls">
+          <div class="controls">
-              Matéria</br>
+            Matéria</br>
-              &nbsp;&nbsp;&nbsp;&nbsp;
+            &nbsp;&nbsp;&nbsp;&nbsp;
-                <a href="{% url object.materia_de_vinculo|urldetail object.materia_de_vinculo.id%}">
+            <a href="{% url object.materia_de_vinculo|urldetail object.materia_de_vinculo.id%}">
-                  {{object.materia_de_vinculo}}
+              {{object.materia_de_vinculo}}
-                </a>
+            </a>
-              </br>
+            </br>
-                {% if object.materia_de_vinculo.autoria_set.all %}
+            {% if object.materia_de_vinculo.autoria_set.all %}
-            		Autores
+            	Autores
-            		{% for a in object.materia_de_vinculo.autoria_set.all %}
+            	{% for a in object.materia_de_vinculo.autoria_set.all %}
-              </br>&nbsp;&nbsp;&nbsp;&nbsp;{{a.autor}}
+                </br>&nbsp;&nbsp;&nbsp;&nbsp;{{a.autor}}
-                {% endfor %}
+              {% endfor %}
-                {% endif %}
+            {% endif %}
-              </br>
+            </br>
-                Texto Original
+            Texto Original
-              </br>
+            </br>
-              {% if object.materia_de_vinculo.texto_original %}
+            {% if object.materia_de_vinculo.texto_original %}
-                  <a href="{{object.materia_de_vinculo.texto_original.url}}">
+              <a href="{{object.materia_de_vinculo.texto_original.url}}">
-                    {{object.materia_de_vinculo.texto_original| to_str | split:"/" | get_last_item_from_list:-1}}
+                {{object.materia_de_vinculo.texto_original| to_str | split:"/" | get_last_item_from_list:-1}}
-                  </a>
+              </a>
-              {% endif %}
+            {% endif %}
-              </br>
+            </br>
           </div>
         </div>
       </div>
     {% endif %}
-
     {% if not AppConfig.receber_recibo_proposicao %}
-        {% if object.hash_code %}
+      {% if object.hash_code %}
-
+        <div class="col-sm-12">
-            <div class="col-sm-12">
+          <div id="div_id_hash_code" class="form-group">
-            <div id="div_id_hash_code" class="form-group">
+            <p class="control-label">{%field_verbose_name object 'hash_code'%}</p>
-              <p class="control-label">{%field_verbose_name object 'hash_code'%}</p>
+            <div class="controls">
-              <div class="controls">
+              <div class="form-control-static">{{object.hash_code}}</div>
-                  <div class="form-control-static">{{object.hash_code}}</div>
-              </div>
             </div>
           </div>
-        {% endif %}
+        </div>
+      {% endif %}
     {% endif %}
   </div>
+  {% if user.is_superuser %}
+    <div class="row">
+      {% if proposicao.user %}
+        <div class="col-sm-4">
+          <div id="div_id_user" class="form-group">
+            <p class="control-label">Usuário</p>
+            <div class="controls">
+              <div class="form-control-static">
+                <div class="dont-break-out">
+                  <a href="{% url 'sapl.base:user_edit' proposicao.user.pk %}">{{ proposicao.user }}</a>
+                </div>
+              </div>
+            </div>
+          </div>
+        </div>
+      {% endif %}
+      {% if proposicao.ip %}
+        <div class="col-sm-4">
+          <div id="div_ip_user" class="form-group">
+            <p class="control-label">IP</p>
+            <div class="controls">
+              <div class="form-control-static">
+                <div class="dont-break-out">{{ proposicao.ip }}</div>
+              </div>
+            </div>
+          </div>
+        </div>
+      {% endif %}
+      {% if proposicao.ultima_edicao %}
+        <div class="col-sm-4">
+          <div id="div_ud_user" class="form-group">
+            <p class="control-label">Data e Hora da Edição</p>
+            <div class="controls">
+              <div class="form-control-static">
+                <div class="dont-break-out">{{ proposicao.ultima_edicao }}</div>
+              </div>
+            </div>
+          </div>
+        </div>
+      {% endif %}
+    </div>
+  {% endif %}
 {% endblock detail_content %}