Repository: interlegis/sapl

Issue:
Retirar a obrigatoriedade do CAMPO TEXTO DA AÇÃO, ao adicionar uma TRAMITAÇÃO
<!--- Forneça um resumo geral da _issue_ no título acima -->

## Comportamento Esperado
<!--- Se você está descrevendo um _bug_, conte-nos o que deveria acontecer. -->
<!--- Se você está sugerindo uma mudança/melhoria, conte-nos como deve funcionar. -->
Não ser o usuário OBRIGADO A DIGITAR  O TEXTO DA AÇÃO, ao adicionar uma tramitação, que isso fique facultativo

## Comportamento Atual
<!--- Se está descrevendo um bug, conte-nos o que acontece em vez do comportamento esperado. -->
<!--- Se está sugerindo uma mudança/melhoria, explique a diferença com o comportamento atual. -->
Hoje ao adicionar uma tramitação, existe a obrigatoriedade de preencher o campo TEXTO DA AÇÃO, se não preencher você não consegue cadastrar uma tramitação

## Possível Solução
<!--- Não é obrigatório, mas sugira uma possível correção/razão para o bug -->
<!--- ou ideias de como implementar a adição/mudança. -->
Retirar esse obrigatoriedade, pois na maioria dos STATUS que o usuário defini, já existe a ação que deve ser tomada, por exemplo: STATUS : Aguardando inclusão de matéria na ORDEM DO DIA, isso por si só já é o texto da AÇÃO, fica repetitivo e demorado para o usuário

## Passos para Reproduzir (para bugs)
<!--- Forneça um link para um exemplo, ou um conjunto de passos inequívocos -->
<!--- para reproduzir esse bug. Inclua código para reproduzir, se relevante. -->
1.
2.
3.
4.

## Contexto
<!--- Como esse problema o afeta? O que você está tentando realizar? -->
<!--- Fornecer o contexto nos ajuda a encontrar uma solução que seja mais útil no mundo real -->

## Imagens do Ocorrido
<!--- Representação visual em vídeo ou imagem do ocorrido -->
<!--- Se está descrevendo um bug poste imagens ou vídeos na repordução do bug citado, caso se aplique -->
![trammmmmmmmmmmmmmmmm](https://user-images.githubusercontent.com/2423695/65625416-2df15f00-dfa2-11e9-9f3d-bca9c27ee81f.png)

## Seu Ambiente
<!--- Inclua detalhes relevantes sobre o ambiente em que você presenciou/experienciou o bug. -->
* Versão usada (_Release_):
* Nome e versão do navegador:
* Nome e versão do Sistema Operacional (desktop ou mobile):
* Link para o seu projeto (Caso de fork deste projeto):



Discussion:
O que acha @LeandroJatai ? @DtiLegBG ? Vcs também estão de acordo?
Na minha opinião, não vejo problemas. Pode ser retirada.
Concorda @andreagudo ?

Resolving patch:
diff --git a/sapl/materia/migrations/0060_auto_20190930_1136.py b/sapl/materia/migrations/0060_auto_20190930_1136.py
new file mode 100644
--- /dev/null
+++ b/sapl/materia/migrations/0060_auto_20190930_1136.py
@@ -0,0 +1,20 @@
+# -*- coding: utf-8 -*-
+# Generated by Django 1.11.20 on 2019-09-30 14:36
+from __future__ import unicode_literals
+
+from django.db import migrations, models
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ('materia', '0059_merge_20191003_0854'),
+    ]
+
+    operations = [
+        migrations.AlterField(
+            model_name='tramitacao',
+            name='texto',
+            field=models.TextField(blank=True, verbose_name='Texto da Ação'),
+        ),
+    ]
diff --git a/sapl/materia/models.py b/sapl/materia/models.py
--- a/sapl/materia/models.py
+++ b/sapl/materia/models.py
@@ -1065,7 +1065,7 @@ class Tramitacao(models.Model):
     turno = models.CharField(
         max_length=1, blank=True, verbose_name=_('Turno'),
         choices=TURNO_CHOICES)
-    texto = models.TextField(verbose_name=_('Texto da Ação'))
+    texto = models.TextField(verbose_name=_('Texto da Ação'), blank=True)
     data_fim_prazo = models.DateField(
         blank=True, null=True, verbose_name=_('Data Fim Prazo'))
     user = models.ForeignKey(get_settings_auth_user_model(),


Test patch:
diff --git a/sapl/materia/tests/test_materia.py b/sapl/materia/tests/test_materia.py
--- a/sapl/materia/tests/test_materia.py
+++ b/sapl/materia/tests/test_materia.py
@@ -521,9 +521,7 @@ def test_form_errors_tramitacao(admin_client):
             ['Este campo é obrigatório.'])
     assert (response.context_data['form'].errors[
             'unidade_tramitacao_destino'] == ['Este campo é obrigatório.'])
-    assert (response.context_data['form'].errors['texto'] ==
-            ['Este campo é obrigatório.'])
-
+            
 
 @pytest.mark.django_db(transaction=False)
 def test_form_errors_relatoria(admin_client):
diff --git a/sapl/materia/tests/test_materia_form.py b/sapl/materia/tests/test_materia_form.py
--- a/sapl/materia/tests/test_materia_form.py
+++ b/sapl/materia/tests/test_materia_form.py
@@ -206,13 +206,12 @@ def test_valida_campos_obrigatorios_tramitacao_form():
     errors = form.errors
 
     assert errors['unidade_tramitacao_local'] == [_('Este campo é obrigatório.')]
-    assert errors['texto'] == [_('Este campo é obrigatório.')]
     assert errors['status'] == [_('Este campo é obrigatório.')]
     assert errors['data_tramitacao'] == [_('Este campo é obrigatório.')]
     assert errors['unidade_tramitacao_destino'] == [_('Este campo é obrigatório.')]
     assert errors['urgente'] == [_('Este campo é obrigatório.')]
 
-    assert len(errors) == 6
+    assert len(errors) == 5
 
 
 @pytest.mark.django_db(transaction=False)
@@ -224,13 +223,12 @@ def test_valida_campos_obrigatorios_tramitacao_update_form():
     errors = form.errors
 
     assert errors['unidade_tramitacao_local'] == [_('Este campo é obrigatório.')]
-    assert errors['texto'] == [_('Este campo é obrigatório.')]
     assert errors['status'] == [_('Este campo é obrigatório.')]
     assert errors['data_tramitacao'] == [_('Este campo é obrigatório.')]
     assert errors['unidade_tramitacao_destino'] == [_('Este campo é obrigatório.')]
     assert errors['urgente'] == [_('Este campo é obrigatório.')]
 
-    assert len(errors) == 6
+    assert len(errors) == 5
 
 
 @pytest.mark.django_db(transaction=False)