-
Notifications
You must be signed in to change notification settings - Fork 22
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Adiciona compatibilidade com o PostgreSQL #366
Adiciona compatibilidade com o PostgreSQL #366
Conversation
Necessário executar testes completos para SEI3 e SEI4 e SEI4 - MGI. @mateussbh se possível colocar na próxima sprint de testes. |
Após as alterações realizadas nos commits 9231a99 e 334f084, foi possível instalar o módulo e enviar, com sucesso, um processo de um ambiente de homologação interno para o ambiente de teste disponibilizado pelo Tramita. Ainda não foi possível receber um processo do ambiente de teste no mesmo ambiente de homologação interno. A ação gerou, no Infra > Log do SEI, o seguinte erro: Descrição: Mensagem: Trilha de Processamento: Descrição Original: Trilha de Processamento Original: |
@mateussbh foi observado o mesmo comportamento no ambiente de testes? |
Isso me parece o comportamento padrão do postgres sobre uso de uma conexão quando dá problema em um SQL. É necessário maiores testes tanto do SEI core como do módulo para esse banco de dados. @PauloNehme por favor testar o SEI core em especial com falhas de negócio sendo logadas. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Necessário adicionar env_postgres nas pastas de testes.
Não consigo enviar alteração no seu PR
@@ -93,7 +93,8 @@ protected function atualizarVersaoConectado() | |||
if ( | |||
!(BancoSip::getInstance() instanceof InfraMySql) && | |||
!(BancoSip::getInstance() instanceof InfraSqlServer) && | |||
!(BancoSip::getInstance() instanceof InfraOracle) | |||
!(BancoSip::getInstance() instanceof InfraOracle) && | |||
!(BancoSEI::getInstance() instanceof InfraPostgreSql) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Não podemos referenciar o BancoSEI no arquivo de atualização do SIP.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Bem observado. Obrigado. Corrigido no commit 0c769f8.
Prezado @santosleonardo, poderia atualizar o PR para a versão do módulo do tramita 3.5.0, para que possamos dar prosseguimento com os testes? Por favor. Complementando o Change request realizado pelo @caduvieira, também foi realizada a mudança no arquivo /src/rn/ReceberProcedimentoRN.php, conforme abaixo:
Porém, conforme dito, peço que atualize o PR com a branch master e faça o rebase para prosseguirmos com os testes para garantir a estabilidade destas atualização nas demais versões do sistema. |
Fiz as alterações e um merge da master. |
Fechado em favor do #434 que possui os testes. |
Também corrige um erro exibido ao tentar carregar os Processos Tramitados Externamente.
Ambiente testado: Homologação
Versão do PHP: 7.3.33
Versão do sistema SEI: 4.0.11
Versão do módulo: 3.3.1
Versão do PostgreSQL: 12.16