[BugFix] Ocultar módulos não utilizados #346
Conversation
| if (srvRecuperado == null) { | ||
| // Constroi uma linha completa, tipo full path | ||
| for (String s : servicoPath.split(";")) { | ||
| String[] asParts = s.split(":"); // Separa a sigla da | ||
| // descrição | ||
| String sSigla = asParts[0]; | ||
| srv = new CpServico(); | ||
| srv.setSiglaServico(srvPai != null ? srvPai.getSigla() + "-" + sSigla : sSigla); | ||
| srv.setCpServicoPai(srvPai); | ||
| srvRecuperado = dao().consultarPorSigla(srv); | ||
| if (srvRecuperado == null) { | ||
| CpTipoServico tpsrv = dao().consultar(CpTipoServico.TIPO_CONFIG_SISTEMA, CpTipoServico.class, false); | ||
| String sDesc = (asParts.length > 1 ? asParts[1] : ""); | ||
| srv.setDscServico(sDesc); | ||
| srv.setCpServicoPai(srvPai); | ||
| srv.setCpTipoServico(tpsrv); | ||
| ContextoPersistencia.begin(); | ||
| dao().acrescentarServico(srv); | ||
| ContextoPersistencia.commit(); | ||
| } | ||
| srvPai = srvRecuperado; | ||
| } | ||
| } |
There was a problem hiding this comment.
@caiosantos-codata não faço idéia se dá pra a gente remover isso. Eu sei que tem inclusive uma tabela no banco que usa essa forma de construir serviços: XXX;YYY;ZZZ etc... Qual a motivação pra isso? Como a gente garante que isso não vai causar problemas? Pode dar mais detalhes?
There was a problem hiding this comment.
@michelrisucci-codata, sua preocupação faz sentido.
esse trecho é reponsável por recuperar serviços que não estão no cache, que é inicializado no método consultarCpServicoPorChave.O cache, por sua vez, é obtido a partir da tabela CpServico.
Entendi que isso foi uma forma de 'contornar' os serviços listados no banco. Naveguei no histórico de commits e não achei uma razao para isso.
Não há garantia total que não causará problemas. Porém, o risco é limitado já que a ideia é ocultar serviço que não são utilizados.
ficou mais claro?
Related to Issues #20 #22 #23 #24.