Gostaria de sugerir uma mudança na arquitetura de módulos do SEI, mais especificamente na interface ISEIIntegracao.
Atualmente, toda nova versão do SEI gera novos métodos na interface ISEIIntegracao, utilizada para expor os pontos de extensão do SEI para implementação pelos módulos.
Essa abordagem faz com que todos os plugins do SEI precisem ser modificados a cada nova versão do sistema apenas para a implementação de métodos vazios para não quebrar o contrato da interface.
Exemplo:
Minha sugestão é que os módulos passem a utilizar uma classe abstrata do core do SEI que implemente a Interface ISEIIntegracao. Dessa forma, os módulos teriam que sobrescrever apenas os métodos que eles forem utilizar e a adição de novos métodos não quebrariam os módulos já desenvolvidos. Exemplo de implementação:
Desta forma, os módulos utilizariam apenas a classe abstrata e herdaria apenas os métodos necessários. Novas versões do SEI com novos pontos de expensão não quebrariam os módulos atuais.
class PENIntegracao implements SeiIntegracao{ public function montarBotaoProcedimento(SeiIntegracaoDTO $objSeiIntegracaoDTO){ $objSessaoSEI = SessaoSEI::getInstance(); $objPaginaSEI = PaginaSEI::getInstance(); $strAcoesProcedimento = ""; $objProcedimentoDTO = $objSeiIntegracaoDTO->getObjProcedimentoDTO(); $dblIdProcedimento = $objProcedimentoDTO->getDblIdProcedimento(); $numIdUsuario = SessaoSEI::getInstance()->getNumIdUsuario(); $numIdUnidadeAtual = SessaoSEI::getInstance()->getNumIdUnidadeAtual(); //Verifica se o processo encontra-se aberto na unidade atual $objAtividadeRN = new AtividadeRN(); $objPesquisaPendenciaDTO = new PesquisaPendenciaDTO(); $objPesquisaPendenciaDTO->setDblIdProtocolo($dblIdProcedimento); $objPesquisaPendenciaDTO->setNumIdUsuario($numIdUsuario); $objPesquisaPendenciaDTO->setNumIdUnidade($numIdUnidadeAtual); $objPesquisaPendenciaDTO->setStrSinMontandoArvore('N'); $arrObjProcedimentoDTO = $objAtividadeRN->listarPendenciasRN0754( $objPesquisaPendenciaDTO); $bolFlagAberto = count($arrObjProcedimentoDTO) == 1; //Verificação da Restrição de Acesso à Funcionalidade $bolAcaoExpedirProcesso = $objSessaoSEI->verificarPermissao( 'pen_procedimento_expedir'); $objExpedirProcedimentoRN = new ExpedirProcedimentoRN(); $objProcedimentoDTO = $objExpedirProcedimentoRN ->consultarProcedimento($dblIdProcedimento); $bolProcessoEstadoNormal = $objProcedimentoDTO->getStrStaEstadoProtocolo() != ProtocoloRN:: $TE_PROCEDIMENTO_SOBRESTADO; //TODO: Não apresentar //$bolFlagAberto && $bolAcaoProcedimentoEnviar && $objProcedimentoDTO->getStrStaNivelAcessoGlobalProtocolo()!=ProtocoloRN::$NA_SIGILOSO if($bolFlagAberto && $bolAcaoExpedirProcesso && $bolProcessoEstadoNormal && $objProcedimentoDTO->getStrStaNivelAcessoGlobalProtocolo()!=ProtocoloRN:: $NA_SIGILOSO) { $numTabBotao = $objPaginaSEI ->getProxTabBarraComandosSuperior(); $strAcoesProcedimento .= ''; } if($objProcedimentoDTO ->getStrStaEstadoProtocolo() == ProtocoloRN::$TE_EM_PROCESSAMENTO){ $strAcoesProcedimento .= ''; $strAcoesProcedimento .= ''; $strAcoesProcedimento .= ''; } } } ?>
Dado o problema arquitetural que você mostrou, acredito que essa alteração é fundamental para o futuro saudável da aplicação e integração com seus Módulos.
Analista em Tecnologia da Informação Marcus Vinicius Pereira de Menezes ________________________________ De: sei-tecnico em nome de Guilherme Andrade Del Cantoni Enviado: segunda-feira, 21 de março de 2016 15:29 Para: sei-tecnico@listas.softwarepublico.gov.br; Mairon Assunto: [sei-tecnico] Alteração na Interface de Módulos do SEI (ISEIIntegracao.php)
Prezados,
Gostaria de sugerir uma mudança na arquitetura de módulos do SEI, mais especificamente na interface ISEIIntegracao.
Atualmente, toda nova versão do SEI gera novos métodos na interface ISEIIntegracao, utilizada para expor os pontos de extensão do SEI para implementação pelos módulos.
Essa abordagem faz com que todos os plugins do SEI precisem ser modificados a cada nova versão do sistema apenas para a implementação de métodos vazios para não quebrar o contrato da interface.
Exemplo: interface ISeiIntegracao { public function montarBotaoControleProcessos(); public function montarIconeControleProcessos($arrObjProcedimentoDTO);
public function montarIconeAcompanhamentoEspecial($arrObjProcedimentoDTO); public function montarBotaoProcedimento(SeiIntegracaoDTO $objSeiIntegracaoDTO); public function montarIconeProcedimento(SeiIntegracaoDTO $objSeiIntegracaoDTO); public function montarBotaoDocumento(SeiIntegracaoDTO $objSeiIntegracaoDTO); public function montarIconeDocumento(SeiIntegracaoDTO $objSeiIntegracaoDTO); public function atualizarConteudoDocumento(DocumentoDTO $objDocumentoDTO); public function excluirProcedimento(ProcedimentoDTO $objProcedimentoDTO);
public function excluirDocumento(DocumentoDTO $objDocumentoDTO); } ?>
Minha sugestão é que os módulos passem a utilizar uma classe abstrata do core do SEI que implemente a Interface ISEIIntegracao. Dessa forma, os módulos teriam que sobrescrever apenas os métodos que eles forem utilizar e a adição de novos métodos não quebrariam os módulos já desenvolvidos.
Exemplo de implementação:
abstract class SeiIntegracao implements ISeiIntegracao { public function montarBotaoControleProcessos() { } public function montarIconeControleProcessos($arrObjProcedimentoDTO) { }
public function montarIconeAcompanhamentoEspecial($arrObjProcedimentoDTO) { } public function montarBotaoProcedimento(SeiIntegracaoDTO $objSeiIntegracaoDTO) { } public function montarIconeProcedimento(SeiIntegracaoDTO $objSeiIntegracaoDTO) { } public function montarBotaoDocumento(SeiIntegracaoDTO $objSeiIntegracaoDTO) { } public function montarIconeDocumento(SeiIntegracaoDTO $objSeiIntegracaoDTO) { } public function atualizarConteudoDocumento(DocumentoDTO $objDocumentoDTO) { } public function excluirProcedimento(ProcedimentoDTO $objProcedimentoDTO) { }
public function excluirDocumento(DocumentoDTO $objDocumentoDTO) { } } ?>
Desta forma, os módulos utilizariam apenas a classe abstrata e herdaria apenas os métodos necessários. Novas versões do SEI com novos pontos de expensão não quebrariam os módulos atuais.
class PENIntegracao implements SeiIntegracao{ public function montarBotaoProcedimento(SeiIntegracaoDTO $objSeiIntegracaoDTO){ $objSessaoSEI = SessaoSEI::getInstance(); $objPaginaSEI = PaginaSEI::getInstance(); $strAcoesProcedimento = ""; $objProcedimentoDTO = $objSeiIntegracaoDTO->getObjProcedimentoDTO(); $dblIdProcedimento = $objProcedimentoDTO->getDblIdProcedimento(); $numIdUsuario = SessaoSEI::getInstance()->getNumIdUsuario(); $numIdUnidadeAtual = SessaoSEI::getInstance()->getNumIdUnidadeAtual(); //Verifica se o processo encontra-se aberto na unidade atual $objAtividadeRN = new AtividadeRN(); $objPesquisaPendenciaDTO = new PesquisaPendenciaDTO(); $objPesquisaPendenciaDTO->setDblIdProtocolo($dblIdProcedimento); $objPesquisaPendenciaDTO->setNumIdUsuario($numIdUsuario); $objPesquisaPendenciaDTO->setNumIdUnidade($numIdUnidadeAtual); $objPesquisaPendenciaDTO->setStrSinMontandoArvore('N'); $arrObjProcedimentoDTO = $objAtividadeRN->listarPendenciasRN0754($objPesquisaPendenciaDTO); $bolFlagAberto = count($arrObjProcedimentoDTO) == 1; //Verificação da Restrição de Acesso à Funcionalidade $bolAcaoExpedirProcesso = $objSessaoSEI->verificarPermissao('pen_procedimento_expedir'); $objExpedirProcedimentoRN = new ExpedirProcedimentoRN(); $objProcedimentoDTO = $objExpedirProcedimentoRN->consultarProcedimento($dblIdProcedimento); $bolProcessoEstadoNormal = $objProcedimentoDTO->getStrStaEstadoProtocolo() != ProtocoloRN::$TE_PROCEDIMENTO_SOBRESTADO; //TODO: Não apresentar //$bolFlagAberto && $bolAcaoProcedimentoEnviar && $objProcedimentoDTO->getStrStaNivelAcessoGlobalProtocolo()!=ProtocoloRN::$NA_SIGILOSO if($bolFlagAberto && $bolAcaoExpedirProcesso && $bolProcessoEstadoNormal && $objProcedimentoDTO->getStrStaNivelAcessoGlobalProtocolo()!=ProtocoloRN::$NA_SIGILOSO) { $numTabBotao = $objPaginaSEI->getProxTabBarraComandosSuperior(); $strAcoesProcedimento .= ''; } if($objProcedimentoDTO->getStrStaEstadoProtocolo() == ProtocoloRN::$TE_EM_PROCESSAMENTO){ $strAcoesProcedimento .= ''; $strAcoesProcedimento .= ''; $strAcoesProcedimento .= ''; } } } ?>
O ponto não é um problema em si, mas melhora bastante a forma de implementar módulos e evoluir o SEI se afetar módulos já funcionando.
De qualquer forma já existe a solução sugerida pelo Cantoni e precisamos apenas espera tal melhoria.
Atenciosamente,
[cid:image001.png@01D185D9.0F1002C0]
Nei Jobson da Costa Carneiro Assessor Superintendência de Gestão Interna da Informação (SGI) _______________________________________________ Agência Nacional de Telecomunicações - Anatel SAUS Quadra 6, Bloco E, 3º Andar, Ala Norte CEP: 70070-940 - Brasília/DF Tel: (61) 2312-1751
De: sei-tecnico [mailto:sei-tecnico-bounces@listas.softwarepublico.gov.br] Em nome de Marcus Vinicius Pereira de Menezes Enviada em: quinta-feira, 24 de março de 2016 11:41 Para: Guilherme Andrade Del Cantoni; sei-tecnico@listas.softwarepublico.gov.br; Mairon Assunto: Re: [sei-tecnico] Alteração na Interface de Módulos do SEI (ISEIIntegracao.php)
Guilherme,
Dado o problema arquitetural que você mostrou, acredito que essa alteração é fundamental para o futuro saudável da aplicação e integração com seus Módulos.
Analista em Tecnologia da Informação Marcus Vinicius Pereira de Menezes ________________________________
De: sei-tecnico > em nome de Guilherme Andrade Del Cantoni >
Assunto: [sei-tecnico] Alteração na Interface de Módulos do SEI (ISEIIntegracao.php)
Prezados,
Gostaria de sugerir uma mudança na arquitetura de módulos do SEI, mais especificamente na interface ISEIIntegracao.
Atualmente, toda nova versão do SEI gera novos métodos na interface ISEIIntegracao, utilizada para expor os pontos de extensão do SEI para implementação pelos módulos.
Essa abordagem faz com que todos os plugins do SEI precisem ser modificados a cada nova versão do sistema apenas para a implementação de métodos vazios para não quebrar o contrato da interface.
Exemplo:
Minha sugestão é que os módulos passem a utilizar uma classe abstrata do core do SEI que implemente a Interface ISEIIntegracao. Dessa forma, os módulos teriam que sobrescrever apenas os métodos que eles forem utilizar e a adição de novos métodos não quebrariam os módulos já desenvolvidos. Exemplo de implementação:
Desta forma, os módulos utilizariam apenas a classe abstrata e herdaria apenas os métodos necessários. Novas versões do SEI com novos pontos de expensão não quebrariam os módulos atuais.
class PENIntegracao implements SeiIntegracao{ public function montarBotaoProcedimento(SeiIntegracaoDTO $objSeiIntegracaoDTO){ $objSessaoSEI = SessaoSEI::getInstance(); $objPaginaSEI = PaginaSEI::getInstance(); $strAcoesProcedimento = ""; $objProcedimentoDTO = $objSeiIntegracaoDTO->getObjProcedimentoDTO(); $dblIdProcedimento = $objProcedimentoDTO->getDblIdProcedimento(); $numIdUsuario = SessaoSEI::getInstance()->getNumIdUsuario(); $numIdUnidadeAtual = SessaoSEI::getInstance()->getNumIdUnidadeAtual(); //Verifica se o processo encontra-se aberto na unidade atual $objAtividadeRN = new AtividadeRN(); $objPesquisaPendenciaDTO = new PesquisaPendenciaDTO(); $objPesquisaPendenciaDTO->setDblIdProtocolo($dblIdProcedimento); $objPesquisaPendenciaDTO->setNumIdUsuario($numIdUsuario); $objPesquisaPendenciaDTO->setNumIdUnidade($numIdUnidadeAtual); $objPesquisaPendenciaDTO->setStrSinMontandoArvore('N'); $arrObjProcedimentoDTO = $objAtividadeRN->listarPendenciasRN0754($objPesquisaPendenciaDTO); $bolFlagAberto = count($arrObjProcedimentoDTO) == 1; //Verificação da Restrição de Acesso à Funcionalidade $bolAcaoExpedirProcesso = $objSessaoSEI->verificarPermissao('pen_procedimento_expedir'); $objExpedirProcedimentoRN = new ExpedirProcedimentoRN(); $objProcedimentoDTO = $objExpedirProcedimentoRN->consultarProcedimento($dblIdProcedimento); $bolProcessoEstadoNormal = $objProcedimentoDTO->getStrStaEstadoProtocolo() != ProtocoloRN::$TE_PROCEDIMENTO_SOBRESTADO; //TODO: Não apresentar //$bolFlagAberto && $bolAcaoProcedimentoEnviar && $objProcedimentoDTO->getStrStaNivelAcessoGlobalProtocolo()!=ProtocoloRN::$NA_SIGILOSO if($bolFlagAberto && $bolAcaoExpedirProcesso && $bolProcessoEstadoNormal && $objProcedimentoDTO->getStrStaNivelAcessoGlobalProtocolo()!=ProtocoloRN::$NA_SIGILOSO) { $numTabBotao = $objPaginaSEI->getProxTabBarraComandosSuperior(); $strAcoesProcedimento .= ''; } if($objProcedimentoDTO->getStrStaEstadoProtocolo() == ProtocoloRN::$TE_EM_PROCESSAMENTO){ $strAcoesProcedimento .= ''; $strAcoesProcedimento .= ''; $strAcoesProcedimento .= ''; } } } ?>
Guilherme Del Cantoni
Marcus Menezes
Nei Jobson da Costa Carneiro