Skip to content

Commit

Permalink
Hotfix/1.33.12 (#1638)
Browse files Browse the repository at this point in the history
* fix(79686): Corrige layout consolidado dre

Corrige má distribuição das linhas, na coluna das despesas, quando não
há o recurso de livre aplicação

* fix(79837): Corrige lista de pcs para acompanhamento de PC

Informações de técnico, data de recebimento, data de última análise e
devolução ao tesouro não eram incluídas nas consultas de alguns status
de PC.

* fix(hf.79686): Corrige má distribuição de linhas na execução financeira

Remove labels de teste

* fix(1.3.12): Atualiza versão e hotfixes.md
  • Loading branch information
alcfernandes authored Nov 29, 2022
1 parent 363626f commit f058bc2
Show file tree
Hide file tree
Showing 9 changed files with 73 additions and 57 deletions.
4 changes: 4 additions & 0 deletions hotfixes.md
Original file line number Diff line number Diff line change
@@ -1,3 +1,7 @@
### 1.33.11 - 29/11/2022 - Hotfix - Soluções de bugs urgentes durante a sprint 54
* (79686 Consolidado PCs: Demonstrativo da execução financeira: má distribuição das linhas, na coluna das despesas, quando não há o recurso de livre aplicação
* (79837) Acompanhamento de PC: as informações das PCs nos painéis de síntese variam a depender dos filtros aplicados

### 1.33.11 - 23/11/2022 - Hotfix - Soluções de bugs urgentes durante a sprint 54
* (79496) Inclui tratamento de erro em prévia de relatório de acertos
* (79583) Configura integração Celery e Sentry
Expand Down
2 changes: 1 addition & 1 deletion sme_ptrf_apps/__init__.py
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
__version__ = "1.33.11"
__version__ = "1.33.12"


__version_info__ = tuple(
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -52,7 +52,7 @@ def get_tecnico_responsavel(self, obj):
return obj.tecnico_responsavel.nome if obj.tecnico_responsavel else ''

def get_devolucao_ao_tesouro(self, obj):
return _str_devolucao_ao_tesouro(obj)
return obj.total_devolucao_ao_tesouro_str

def get_unidade_tipo_unidade(self, obj):
return obj.associacao.unidade.tipo_unidade if obj.associacao and obj.associacao.unidade else ''
Expand Down Expand Up @@ -111,7 +111,7 @@ def get_processo_sei(self, obj):
return get_processo_sei_da_prestacao(prestacao_contas=obj)

def get_devolucao_ao_tesouro(self, obj):
return _str_devolucao_ao_tesouro(obj)
return obj.total_devolucao_ao_tesouro_str

def get_arquivos_referencia(self, prestacao_contas):
result = []
Expand Down Expand Up @@ -237,7 +237,3 @@ class Meta:
'publicada',
'referencia_consolidado_dre',
)


def _str_devolucao_ao_tesouro(obj):
return f'{obj.total_devolucao_ao_tesouro:.2f}'.replace('.', ',') if obj.devolucoes_ao_tesouro_da_prestacao.count() > 0 else 'Não'
4 changes: 4 additions & 0 deletions sme_ptrf_apps/core/models/prestacao_conta.py
Original file line number Diff line number Diff line change
Expand Up @@ -112,6 +112,10 @@ def tecnico_responsavel(self):
def total_devolucao_ao_tesouro(self):
return self.devolucoes_ao_tesouro_da_prestacao.all().aggregate(Sum('valor'))['valor__sum'] or 0.00

@property
def total_devolucao_ao_tesouro_str(self):
return f'{self.total_devolucao_ao_tesouro:.2f}'.replace('.', ',') if self.devolucoes_ao_tesouro_da_prestacao.count() > 0 else 'Não'

def __str__(self):
return f"{self.periodo} - {self.status}"

Expand Down
25 changes: 15 additions & 10 deletions sme_ptrf_apps/core/services/prestacao_contas_services.py
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,7 @@
)
from ..services import info_acoes_associacao_no_periodo
from ..services.relacao_bens import gerar_arquivo_relacao_de_bens, apagar_previas_relacao_de_bens
from ..services.processos_services import get_processo_sei_da_prestacao
from ..services.processos_services import get_processo_sei_da_prestacao, get_processo_sei_da_associacao_no_periodo
from ...despesas.models import RateioDespesa, Despesa
from ...receitas.models import Receita
from ..tasks import gerar_previa_demonstrativo_financeiro_async
Expand Down Expand Up @@ -320,19 +320,24 @@ def lista_prestacoes_de_conta_nao_recebidas(
if prestacao_conta and prestacao_conta.status not in filtro_status:
continue

if prestacao_conta:
processo_sei = get_processo_sei_da_prestacao(prestacao_contas=prestacao_conta)
else:
processo_sei = get_processo_sei_da_associacao_no_periodo(associacao=associacao, periodo=periodo)

info_prestacao = {
'periodo_uuid': f'{periodo.uuid}',
'data_recebimento': None,
'data_ultima_analise': None,
'processo_sei': get_processo_sei_da_prestacao(prestacao_contas=prestacao_conta) if prestacao_conta else '',
'data_ultima_analise': prestacao_conta.data_ultima_analise if prestacao_conta else None,
'processo_sei': processo_sei,
'status': prestacao_conta.status if prestacao_conta else PrestacaoConta.STATUS_NAO_APRESENTADA,
'tecnico_responsavel': '',
'tecnico_responsavel': prestacao_conta.tecnico_responsavel.nome if prestacao_conta and prestacao_conta.tecnico_responsavel else '',
'unidade_eol': associacao.unidade.codigo_eol,
'unidade_nome': associacao.unidade.nome,
'unidade_tipo_unidade': associacao.unidade.tipo_unidade,
'uuid': f'{prestacao_conta.uuid}' if prestacao_conta else '',
'associacao_uuid': f'{associacao.uuid}',
'devolucao_ao_tesouro': '0,00'
'devolucao_ao_tesouro': prestacao_conta.total_devolucao_ao_tesouro_str if prestacao_conta else 'Não'
}

prestacoes.append(info_prestacao)
Expand All @@ -359,7 +364,7 @@ def lista_prestacoes_de_conta_todos_os_status(

prestacoes = []
for associacao in associacoes_da_dre:
prestacao_conta = PrestacaoConta.objects.filter(associacao=associacao, periodo=periodo).first()
prestacao_conta: PrestacaoConta = PrestacaoConta.objects.filter(associacao=associacao, periodo=periodo).first()

if filtro_por_status and not prestacao_conta and PrestacaoConta.STATUS_NAO_APRESENTADA not in filtro_por_status:
# Pula PCs não apresentadas se existir um filtro por status e não contiver o status não apresentada
Expand All @@ -371,17 +376,17 @@ def lista_prestacoes_de_conta_todos_os_status(

info_prestacao = {
'periodo_uuid': f'{periodo.uuid}',
'data_recebimento': None,
'data_ultima_analise': None,
'data_recebimento': prestacao_conta.data_recebimento if prestacao_conta else None,
'data_ultima_analise': prestacao_conta.data_ultima_analise if prestacao_conta else None,
'processo_sei': get_processo_sei_da_prestacao(prestacao_contas=prestacao_conta) if prestacao_conta else '',
'status': prestacao_conta.status if prestacao_conta else PrestacaoConta.STATUS_NAO_APRESENTADA,
'tecnico_responsavel': '',
'tecnico_responsavel': prestacao_conta.tecnico_responsavel.nome if prestacao_conta and prestacao_conta.tecnico_responsavel else '',
'unidade_eol': associacao.unidade.codigo_eol,
'unidade_nome': associacao.unidade.nome,
'unidade_tipo_unidade': associacao.unidade.tipo_unidade,
'uuid': f'{prestacao_conta.uuid}' if prestacao_conta else '',
'associacao_uuid': f'{associacao.uuid}',
'devolucao_ao_tesouro': '0,00'
'devolucao_ao_tesouro': prestacao_conta.total_devolucao_ao_tesouro_str if prestacao_conta else 'Não'
}

prestacoes.append(info_prestacao)
Expand Down
6 changes: 6 additions & 0 deletions sme_ptrf_apps/core/services/processos_services.py
Original file line number Diff line number Diff line change
@@ -1,5 +1,11 @@
from ..models import ProcessoAssociacao


def get_processo_sei_da_prestacao(prestacao_contas):
return ProcessoAssociacao.by_associacao_periodo(associacao=prestacao_contas.associacao,
periodo=prestacao_contas.periodo)


def get_processo_sei_da_associacao_no_periodo(associacao, periodo):
return ProcessoAssociacao.by_associacao_periodo(associacao=associacao,
periodo=periodo)
Original file line number Diff line number Diff line change
Expand Up @@ -176,7 +176,7 @@ def test_api_list_prestacoes_conta_nao_recebidas_por_periodo_e_dre_sem_filtro_po
'unidade_tipo_unidade': 'CEU',
'uuid': '',
'associacao_uuid': f'{_associacao_c_dre_1.uuid}',
'devolucao_ao_tesouro': '0,00'
'devolucao_ao_tesouro': 'Não'

},
{
Expand All @@ -191,7 +191,7 @@ def test_api_list_prestacoes_conta_nao_recebidas_por_periodo_e_dre_sem_filtro_po
'unidade_tipo_unidade': 'EMEI',
'uuid': f'{_prestacao_conta_2020_1_unidade_a_dre1.uuid}',
'associacao_uuid': f'{_prestacao_conta_2020_1_unidade_a_dre1.associacao.uuid}',
'devolucao_ao_tesouro': '0,00'
'devolucao_ao_tesouro': 'Não'
}
]

Expand Down Expand Up @@ -243,7 +243,7 @@ def test_api_list_prestacoes_conta_nao_recebidas_por_periodo_e_dre_nao_inclui_ou
'unidade_tipo_unidade': 'CEU',
'uuid': '',
'associacao_uuid': f'{_associacao_c_dre_1.uuid}',
'devolucao_ao_tesouro': '0,00'
'devolucao_ao_tesouro': 'Não'

}
]
Expand Down Expand Up @@ -281,7 +281,7 @@ def test_api_list_prestacoes_conta_nao_recebidas_por_nome_unidade(jwt_authentica
'unidade_tipo_unidade': 'EMEI',
'uuid': f'{_prestacao_conta_2020_1_unidade_a_dre1.uuid}',
'associacao_uuid': f'{_prestacao_conta_2020_1_unidade_a_dre1.associacao.uuid}',
'devolucao_ao_tesouro': '0,00'
'devolucao_ao_tesouro': 'Não'
},
]

Expand All @@ -307,7 +307,7 @@ def test_api_list_prestacoes_conta_nao_recebidas_por_nome_unidade(jwt_authentica
'unidade_tipo_unidade': 'CEU',
'uuid': '',
'associacao_uuid': f'{_associacao_c_dre_1.uuid}',
'devolucao_ao_tesouro': '0,00'
'devolucao_ao_tesouro': 'Não'
}

]
Expand Down Expand Up @@ -347,7 +347,7 @@ def test_api_list_prestacoes_conta_nao_recebidas_por_nome_associacao(jwt_authent
'unidade_tipo_unidade': 'EMEI',
'uuid': f'{_prestacao_conta_2020_1_unidade_a_dre1.uuid}',
'associacao_uuid': f'{_prestacao_conta_2020_1_unidade_a_dre1.associacao.uuid}',
'devolucao_ao_tesouro': '0,00'
'devolucao_ao_tesouro': 'Não'
},
]

Expand All @@ -373,7 +373,7 @@ def test_api_list_prestacoes_conta_nao_recebidas_por_nome_associacao(jwt_authent
'unidade_tipo_unidade': 'CEU',
'uuid': '',
'associacao_uuid': f'{_associacao_c_dre_1.uuid}',
'devolucao_ao_tesouro': '0,00'
'devolucao_ao_tesouro': 'Não'
}

]
Expand Down Expand Up @@ -411,7 +411,7 @@ def test_api_list_prestacoes_conta_nao_recebidas_por_tipo_unidade(jwt_authentica
'unidade_tipo_unidade': 'EMEI',
'uuid': f'{_prestacao_conta_2020_1_unidade_a_dre1.uuid}',
'associacao_uuid': f'{_prestacao_conta_2020_1_unidade_a_dre1.associacao.uuid}',
'devolucao_ao_tesouro': '0,00'
'devolucao_ao_tesouro': 'Não'
},
]

Expand All @@ -437,7 +437,7 @@ def test_api_list_prestacoes_conta_nao_recebidas_por_tipo_unidade(jwt_authentica
'unidade_tipo_unidade': 'CEU',
'uuid': '',
'associacao_uuid': f'{_associacao_c_dre_1.uuid}',
'devolucao_ao_tesouro': '0,00'
'devolucao_ao_tesouro': 'Não'
}

]
Expand Down Expand Up @@ -479,7 +479,7 @@ def test_api_list_prestacoes_conta_nao_recebidas_por_status_nao_recebida(jwt_aut
'unidade_tipo_unidade': 'EMEI',
'uuid': f'{_prestacao_conta_2020_1_unidade_a_dre1.uuid}',
'associacao_uuid': f'{_prestacao_conta_2020_1_unidade_a_dre1.associacao.uuid}',
'devolucao_ao_tesouro': '0,00'
'devolucao_ao_tesouro': 'Não'
},
]

Expand Down Expand Up @@ -520,7 +520,7 @@ def test_api_list_prestacoes_conta_nao_recebidas_por_status_nao_apresentada(jwt_
'unidade_tipo_unidade': 'CEU',
'uuid': '',
'associacao_uuid': f'{_associacao_c_dre_1.uuid}',
'devolucao_ao_tesouro': '0,00'
'devolucao_ao_tesouro': 'Não'
}

]
Expand Down Expand Up @@ -588,7 +588,7 @@ def test_api_list_prestacoes_conta_nao_recebidas_por_mais_de_um_status(
'unidade_tipo_unidade': 'CEU',
'uuid': '',
'associacao_uuid': f'{_associacao_c_dre_1.uuid}',
'devolucao_ao_tesouro': '0,00'
'devolucao_ao_tesouro': 'Não'

},
{
Expand All @@ -603,7 +603,7 @@ def test_api_list_prestacoes_conta_nao_recebidas_por_mais_de_um_status(
'unidade_tipo_unidade': 'EMEI',
'uuid': f'{_prestacao_conta_2020_1_unidade_a_dre1.uuid}',
'associacao_uuid': f'{_prestacao_conta_2020_1_unidade_a_dre1.associacao.uuid}',
'devolucao_ao_tesouro': '0,00'
'devolucao_ao_tesouro': 'Não'
}
]

Expand Down
Loading

0 comments on commit f058bc2

Please sign in to comment.