SPDT/sapl: Sistema de Apoio ao Processo Legislativo - sapl

Sistema de Apoio ao Processo Legislativo

joaohortsenado 2e6acfa24d continuous-integration/drone/push Build is passing Details Impede presenças duplicadas de parlamentar na mesma sessão (#3850 ) * Impede presenças duplicadas de parlamentar na mesma sessão O relatório de presença exibia percentuais acima de 100% — em Capanema/PR, um vereador aparecia com 46 sessões em um período de 45. A contagem não estava errada: existiam mesmo duas linhas de presença dele na sessão nº 22 de 30/06/2025, e o relatório contava linhas com Count('id'). A origem é PresencaView/PresencaOrdemDiaView. `presentes_banco` vem do banco como inteiros e `marcados` vem do POST como strings, então `set(presentes_banco) - set(marcados)` nunca casava e resultava em apagar todas as presenças da sessão a cada salvamento, recriando-as em seguida. Sequencialmente o resultado é correto, mas duas submissões concorrentes do formulário — um duplo clique em Salvar — passam ambas pela janela entre o apagar e o recriar e gravam duas linhas por parlamentar. Os ids gravados em Capanema confirmam: pares adjacentes intercalados na ordem alfabética da tela, assinatura de dois laços concorrentes. Nada no banco impedia isso. Não é um caso isolado: Capanema tinha 625 linhas excedentes em 313 sessões, e a base de Agudo/RS, 9 linhas em 1 sessão. Corrige em quatro frentes: - compara os ids com o mesmo tipo e passa a criar apenas quem ainda não tem presença, sem apagar e recriar quem permanece marcado; - usa bulk_create(ignore_conflicts=True), para que a submissão concorrente descarte a inserção repetida em vez de estourar IntegrityError; - adiciona unique_together (sessao_plenaria, parlamentar) nos dois modelos, que é a proteção efetiva contra concorrência, com migração que remove as duplicatas existentes antes de criar a restrição; - conta sessões distintas no relatório, em vez de linhas, para que bases ainda não migradas não exibam percentuais impossíveis. OSTicket #212222 * Remove presenças duplicadas com RunSQL na migração 0070 Substitui o RunPython por um DELETE ... USING por tabela, conforme sugestão da revisão: mantém a presença de menor id de cada par (sessão, parlamentar), agora também em sessao_presencaordemdia. A preferência por data_sessao preenchida foi abandonada, já que o campo não é lido nem escrito em nenhum ponto do código.	1 week ago
..
test_sessao.py	Fix #3168 - lib migration (#3169)	6 years ago
test_sessao_view.py	Impede presenças duplicadas de parlamentar na mesma sessão (#3850)	1 week ago

Impede presenças duplicadas de parlamentar na mesma sessão (#3850 )

* Impede presenças duplicadas de parlamentar na mesma sessão

O relatório de presença exibia percentuais acima de 100% — em Capanema/PR,
um vereador aparecia com 46 sessões em um período de 45. A contagem não
estava errada: existiam mesmo duas linhas de presença dele na sessão nº 22
de 30/06/2025, e o relatório contava linhas com Count('id').

A origem é PresencaView/PresencaOrdemDiaView. `presentes_banco` vem do banco
como inteiros e `marcados` vem do POST como strings, então
`set(presentes_banco) - set(marcados)` nunca casava e resultava em apagar
todas as presenças da sessão a cada salvamento, recriando-as em seguida.
Sequencialmente o resultado é correto, mas duas submissões concorrentes do
formulário — um duplo clique em Salvar — passam ambas pela janela entre o
apagar e o recriar e gravam duas linhas por parlamentar. Os ids gravados em
Capanema confirmam: pares adjacentes intercalados na ordem alfabética da
tela, assinatura de dois laços concorrentes.

Nada no banco impedia isso. Não é um caso isolado: Capanema tinha 625 linhas
excedentes em 313 sessões, e a base de Agudo/RS, 9 linhas em 1 sessão.

Corrige em quatro frentes:

- compara os ids com o mesmo tipo e passa a criar apenas quem ainda não tem
  presença, sem apagar e recriar quem permanece marcado;
- usa bulk_create(ignore_conflicts=True), para que a submissão concorrente
  descarte a inserção repetida em vez de estourar IntegrityError;
- adiciona unique_together (sessao_plenaria, parlamentar) nos dois modelos,
  que é a proteção efetiva contra concorrência, com migração que remove as
  duplicatas existentes antes de criar a restrição;
- conta sessões distintas no relatório, em vez de linhas, para que bases
  ainda não migradas não exibam percentuais impossíveis.

OSTicket #212222

* Remove presenças duplicadas com RunSQL na migração 0070

Substitui o RunPython por um DELETE ... USING por tabela, conforme
sugestão da revisão: mantém a presença de menor id de cada par
(sessão, parlamentar), agora também em sessao_presencaordemdia.
A preferência por data_sessao preenchida foi abandonada, já que o
campo não é lido nem escrito em nenhum ponto do código.

1 week ago

test_sessao.py

Fix #3168 - lib migration (#3169)

6 years ago

test_sessao_view.py

Impede presenças duplicadas de parlamentar na mesma sessão (#3850)

1 week ago