From 7c4dbabd43d2140c8f93c802520254aa91645e7c Mon Sep 17 00:00:00 2001 From: magno.oliveira Date: Mon, 15 Sep 2014 01:02:48 +0000 Subject: [PATCH] Identificação de campos obrigatgorios e validação. --- Desenvolvimento/Codificacao/e-Selo/WebContent/WEB-INF/jsp/faixa/form.jsp | 23 ++++++++++++----------- Desenvolvimento/Codificacao/e-Selo/src/main/java/br/com/eselo/component/messages.properties | 19 ++++++++++--------- Desenvolvimento/Codificacao/e-Selo/src/main/java/br/com/eselo/controller/CriteriosController.java | 128 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++--------------------------------------------------------------- Desenvolvimento/Codificacao/e-Selo/src/main/java/br/com/eselo/controller/FaixaController.java | 68 ++++++++++++++++++++++++++++++++++++-------------------------------- Desenvolvimento/Codificacao/e-Selo/src/main/java/br/com/eselo/controller/ParametroCalculoController.java | 31 +++++++++++++++++-------------- Desenvolvimento/Codificacao/e-Selo/src/main/java/br/com/eselo/controller/RecomendacaoController.java | 116 +++++++++++++++++++++++++++++++++++++++++++++++++++++++------------------------------------------------------------- Desenvolvimento/Codificacao/e-Selo/src/main/java/br/com/eselo/model/Faixa.java | 2 +- 7 files changed, 196 insertions(+), 191 deletions(-) diff --git a/Desenvolvimento/Codificacao/e-Selo/WebContent/WEB-INF/jsp/faixa/form.jsp b/Desenvolvimento/Codificacao/e-Selo/WebContent/WEB-INF/jsp/faixa/form.jsp index 42d09b9..c6d4750 100644 --- a/Desenvolvimento/Codificacao/e-Selo/WebContent/WEB-INF/jsp/faixa/form.jsp +++ b/Desenvolvimento/Codificacao/e-Selo/WebContent/WEB-INF/jsp/faixa/form.jsp @@ -25,43 +25,44 @@
-
" method="post"> + " method="post">
- +
- +
- +
- +
- +
- +
- +
- +
- +
- + + diff --git a/Desenvolvimento/Codificacao/e-Selo/src/main/java/br/com/eselo/component/messages.properties b/Desenvolvimento/Codificacao/e-Selo/src/main/java/br/com/eselo/component/messages.properties index 61bdc7b..9fe9eef 100644 --- a/Desenvolvimento/Codificacao/e-Selo/src/main/java/br/com/eselo/component/messages.properties +++ b/Desenvolvimento/Codificacao/e-Selo/src/main/java/br/com/eselo/component/messages.properties @@ -1,12 +1,13 @@ -conformidades.mensagem.salva=Conformidade salva com sucesso. -conformidades.mensagem.alterada=Conformidade alterada com sucesso. -conformidades.validacao.diretriz=A Diretriz é obrigatória. -conformidades.validacao.descricao=A descrição é obrigatória. -conformidades.validacao.faixa=O número da Faixa é obrigatório. -conformidades.validacao.faixa_final=A Faixa Final é obrigatória. -conformidades.validacao.faixa_inicial=A Faixa Inicial é obrigatória. -conformidades.validacao.tipo_conformidade=O Tipo de Conformidade é obrigatório. -conformidades.validacao.range_faixas=A Faixa Inicial deve ser menor que a Faixa Inicial. +faixas.mensagem.salva=Conformidade salva com sucesso. +faixas.mensagem.alterada=Conformidade alterada com sucesso. +faixas.validacao.diretriz=A Diretriz é obrigatória. +faixas.validacao.descricao=A descrição é obrigatória. +faixas.validacao.faixa=O número da Faixa é obrigatório. +faixas.validacao.percentual_conformidade=O percentual de conformidade da Faixa é obrigatório. +faixas.validacao.faixa_final=A Faixa Final é obrigatória. +faixas.validacao.faixa_inicial=A Faixa Inicial é obrigatória. +faixas.validacao.tipo_conformidade=O Tipo de Conformidade é obrigatório. +faixas.validacao.range_faixas=A Faixa Inicial deve ser menor que a Faixa Inicial. parametro_calculo.validacao.descricao=A descrição é obrigatória. tipo_conformidade.mensagem.salva=Tipo de Diretriz salvo com sucesso. tipo_conformidade.mensagem.alterada=Tipo de Diretriz alterado com sucesso. diff --git a/Desenvolvimento/Codificacao/e-Selo/src/main/java/br/com/eselo/controller/CriteriosController.java b/Desenvolvimento/Codificacao/e-Selo/src/main/java/br/com/eselo/controller/CriteriosController.java index 5c1a815..8cbb009 100644 --- a/Desenvolvimento/Codificacao/e-Selo/src/main/java/br/com/eselo/controller/CriteriosController.java +++ b/Desenvolvimento/Codificacao/e-Selo/src/main/java/br/com/eselo/controller/CriteriosController.java @@ -20,6 +20,7 @@ import br.com.eselo.dao.TipoTesteDAO; import br.com.eselo.model.Criterio; import br.com.eselo.model.ParametroCalculo; import br.com.eselo.model.Recomendacao; +import br.com.eselo.model.TipoTeste; @Resource public class CriteriosController { @@ -71,34 +72,9 @@ public class CriteriosController { @Restrito(descricao = "CRITERIO_ADICIONAR") public void adicionar(Criterio criterio) { - if (criterio.getNumeroCriterio() == null) { - validator.add(new ValidationMessage( - "ɉ obrigatório informar o número do critério.", - "criterio.numeroCriterio")); - } else if (hasNumeroCriterioRecomendacaoAtual(criterio)) { - validator - .add(new ValidationMessage( - "Já existe um críterio ativo com esse numero. Critério não cadastrado.", - "criterio.numeroCriterio")); - } else if (criterio.getPeso() == null) { - validator.add(new ValidationMessage( - "É obrigatório informar o peso do critério.", - "criterio.peso")); - } else if (criterio.getDescricao() == null) { - validator.add(new ValidationMessage( - "ɉ obrigatório informar a descrição do critério.", - "criterio.descricao")); - } else if (criterio.getPeso() <= 0) { - validator.add(new ValidationMessage( - "O peso do critério deve ser maior que zero.", - "criterio.peso")); - } else if (criterio.getTipoTeste().getId() == null) { - validator.add(new ValidationMessage( - "É obrigatório informar o Tipo de teste.", - "criterio.tipoTeste")); - }else{ - criterio.setTipoTeste(new TipoTesteDAO(dao.getSession()).load(criterio.getTipoTeste().getId())); - } + criterio.setRecomendacao(new RecomendacaoDAO(dao.getSession()).load(criterio.getRecomendacao().getId())); + + validarCriterio(criterio); result.include("recomendacaoId", criterio.getRecomendacao().getId()); result.include("parametroCalculo", criterio.getRecomendacao().getParametroCalculo()); @@ -107,7 +83,6 @@ public class CriteriosController { validator.onErrorUsePageOf(CriteriosController.class).novo(criterio.getRecomendacao().getId()); - this.criarNovoParametroAvaliacao(criterio, ParametroCalculoController.MODO_ADD); result.include("mensagem", "Critério salvo com sucesso."); @@ -117,40 +92,26 @@ public class CriteriosController { @Put("/criterio") @Restrito(descricao = "CRITERIO_ATUALIZAR") public void atualizar(Criterio criterio) { + + criterio.setRecomendacao(new RecomendacaoDAO(dao.getSession()).load(criterio.getRecomendacao().getId())); + + @SuppressWarnings("unchecked") + List tipoTestes = (List) dao.getAllTipoTeste(); Criterio criterioCadastrado = dao.load(criterio.getId()); criterioCadastrado.setNumeroCriterio(criterio.getNumeroCriterio()); criterioCadastrado.setDescricao(criterio.getDescricao()); criterioCadastrado.setPeso(criterio.getPeso()); criterioCadastrado.setTipoTeste(criterio.getTipoTeste()); - if (criterio.getNumeroCriterio() == null) { - validator.add(new ValidationMessage( - "ɉ obrigatório informar o número da recomendação.", - "criteiro.numeroCriterio")); - } else if (hasNumeroCriterioRecomendacaoAtual(criterioCadastrado)) { - validator - .add(new ValidationMessage( - "Já existe um criteiro ativo com esse numero. Criteiro não atualizado.", - "criteiro.numeroCriteiro")); - } else if (criterio.getPeso() == null) { - validator.add(new ValidationMessage( - "ɉ obrigatório informar o peso do criteiro.", - "criteiro.peso")); - } else if (criterio.getDescricao() == null) { - validator.add(new ValidationMessage( - "ɉ obrigatório informar a descrição do criteiro.", - "criteiro.descricao")); - } else if (criterio.getTipoTeste() == null) { - validator.add(new ValidationMessage( - "É obrigatório informar o Tipo de teste.", - "criterio.tipoTeste")); - } + result.include("recomendacaoId", criterio.getRecomendacao().getId()); result.include("parametroCalculo", criterio.getRecomendacao().getParametroCalculo()); result.include("tipoTeste", criterio.getTipoTeste()); - populateSelectLists(); - + result.include("tipoTesteList", tipoTestes); + + validarCriterio(criterio); + validator.onErrorUsePageOf(CriteriosController.class).form( criterioCadastrado.getId()); @@ -226,7 +187,37 @@ public class CriteriosController { } } - + private void validarCriterio(Criterio criterio){ + + if (criterio.getNumeroCriterio() == null) { + validator.add(new ValidationMessage( + "ɉ obrigatório informar o número da recomendação.", + "criteiro.numeroCriterio")); + } else if ( hasNumeroCriterioRecomendacaoAtual(criterio)) { + validator + .add(new ValidationMessage( + "Já existe um criteiro ativo com esse numero.", + "criteiro.numeroCriteiro")); + }else if (criterio.getPeso() == null) { + validator.add(new ValidationMessage( + "ɉ obrigatório informar o peso do criteiro.", + "criteiro.peso")); + } else if (criterio.getPeso() <= 0 ) { + validator.add(new ValidationMessage( + "O peso do criteiro deve ser maior que 0.", + "criteiro.peso")); + } else if (criterio.getDescricao() == null) { + validator.add(new ValidationMessage( + "ɉ obrigatório informar a descrição do criteiro.", + "criteiro.descricao")); + } else if (criterio.getTipoTeste().getId() == null) { + validator.add(new ValidationMessage( + "É obrigatório informar o Tipo de teste.", + "criterio.tipoTeste")); + }else{ + criterio.setTipoTeste(new TipoTesteDAO(dao.getSession()).load(criterio.getTipoTeste().getId())); + } + } /** * Recebe um critério antes de ser cadastrad0 e verifica se ja existe um @@ -237,21 +228,32 @@ public class CriteriosController { */ private boolean hasNumeroCriterioRecomendacaoAtual(Criterio criterio) { boolean retorno = false; + + if( criterio.getDescricao() != null && + criterio.getNumeroCriterio() != null && + criterio.getPeso() != null && + criterio.getRecomendacao() != null && + criterio.getTipoTeste().getId() != null){ + Recomendacao recomendacao = criterio.getRecomendacao(); - List listaCadastrada = new RecomendacaoDAO(dao.getSession()).load(criterio.getRecomendacao().getId()).getCriterios(); - for (Criterio criterioCadastrado : listaCadastrada) { - if (criterioCadastrado.getNumeroCriterio().longValue() == criterio - .getNumeroCriterio().longValue()) { - if (criterio.getId() != null) { - if (criterioCadastrado.getId().longValue() != criterio - .getId().longValue()) { + List listaCadastrada = dao.listAllAtivas(recomendacao); + + for (Criterio criterioCadastrado : listaCadastrada) { + if (criterioCadastrado.getNumeroCriterio().longValue() == criterio + .getNumeroCriterio().longValue()) { + if (criterio.getId() != null) { + if (criterioCadastrado.getId().longValue() != criterio + .getId().longValue()) { + return true; + } + } else { return true; } - } else { - return true; } } + } + return retorno; } diff --git a/Desenvolvimento/Codificacao/e-Selo/src/main/java/br/com/eselo/controller/FaixaController.java b/Desenvolvimento/Codificacao/e-Selo/src/main/java/br/com/eselo/controller/FaixaController.java index 73cb81a..9d04aa5 100644 --- a/Desenvolvimento/Codificacao/e-Selo/src/main/java/br/com/eselo/controller/FaixaController.java +++ b/Desenvolvimento/Codificacao/e-Selo/src/main/java/br/com/eselo/controller/FaixaController.java @@ -49,7 +49,7 @@ public class FaixaController { return this.index(idCriterio, 1); } - @Get("/conformidades") + @Get("/faixas") @Restrito(descricao = "CONFORMIDADES_LISTA") public List index(Long idCriterio, int pagina) { CriterioDAO cDAO = new CriterioDAO(dao.getSession()); @@ -133,8 +133,8 @@ public class FaixaController { for (Faixa faixa : faixas) { faixa.setCriterio(criterio); - //criterioId = conformidade.getCriterio().getId(); - validarConformidade(faixa); + //criterioId = faixa.getCriterio().getId(); + validarFaixa(faixa); validator.onErrorUsePageOf(FaixaController.class).add( faixa.getCriterio().getId()); @@ -143,20 +143,21 @@ public class FaixaController { criarNovoParametroAvaliacao(faixas, ParametroCalculoController.MODO_ADD); - result.include("mensagem", Messages.getString("conformidades.mensagem.salva")); + result.include("mensagem", Messages.getString("faixas.mensagem.salva")); result.redirectTo(ParametroCalculoController.class).index(1); } - @Put("/conformidades") + @Put("/faixas") @Restrito(descricao = "CONFORMIDADES_ATUALIZAR") public void atualizar(Faixa faixa) { Faixa faixaCadastrada = dao.load(faixa.getId()); - validarConformidade(faixa); - - populateSelectLists(); + Criterio criterio = daoCriterio.load(faixa.getCriterio().getId()); + faixa.setCriterio(criterio); + + validarFaixa(faixa); - validator.onErrorUsePageOf(FaixaController.class).form(faixaCadastrada.getId()); + validator.onErrorRedirectTo(FaixaController.class).form(faixaCadastrada.getId()); faixaCadastrada.setCriterio(faixa.getCriterio()); faixaCadastrada.setFaixaInicio(faixa.getFaixaInicio()); @@ -166,9 +167,8 @@ public class FaixaController { faixas.add(faixaCadastrada); criarNovoParametroAvaliacao(faixas, ParametroCalculoController.MODO_EDIT); - - result.include("mensagem", - Messages.getString("conformidades.mensagem.alterada")); + + result.include("mensagem", Messages.getString("faixas.mensagem.alterada")); result.redirectTo(ParametroCalculoController.class).index(1); } @@ -189,34 +189,38 @@ public class FaixaController { result.include("faixaList", dao.getAllRecomendacao()); } - private void validarConformidade(Faixa conformidade) { - if (conformidade.getCriterio().getId() == null) { + private void validarFaixa(Faixa faixa) { + if (faixa.getCriterio().getId() == null) { + validator.add(new ValidationMessage(Messages + .getString("faixas.validacao.faixa"), + "faixa.diretriz.id")); + } else if (faixa.getFaixa() == null) { validator.add(new ValidationMessage(Messages - .getString("conformidades.validacao.diretriz"), - "conformidade.diretriz.id")); - } else if (conformidade.getFaixa() == null) { + .getString("faixas.validacao.faixa"), + "faixa.faixa")); + } else if (faixa.getDescricao() == null) { validator.add(new ValidationMessage(Messages - .getString("conformidades.validacao.faixa"), - "conformidade.faixa")); - } else if (conformidade.getDescricao() == null) { + .getString("faixas.validacao.descricao"), + "faixa.descricao")); + } else if (faixa.getFaixaInicio() == null) { validator.add(new ValidationMessage(Messages - .getString("conformidades.validacao.descricao"), - "conformidade.descricao")); - } else if (conformidade.getFaixaInicio() == null) { + .getString("faixas.validacao.faixa_inicial"), + "faixa.faixaInicio")); + } else if (faixa.getFaixaFim() == null) { validator.add(new ValidationMessage(Messages - .getString("conformidades.validacao.faixa_inicial"), - "conformidade.diretriz.id")); - } else if (conformidade.getFaixaFim() == null) { + .getString("faixas.validacao.faixa_final"), + "faixa.faixaFim")); + } else if (faixa.getFaixaInicio() > faixa.getFaixaFim()) { validator.add(new ValidationMessage(Messages - .getString("conformidades.validacao.faixa_final"), - "conformidade.diretriz.id")); - } else if (conformidade.getFaixaInicio() > conformidade.getFaixaFim()) { + .getString("faixas.validacao.range_faixas"), + "faixa.faixaInicio")); + } else if (faixa.getPercentualConformidade() == null) { validator.add(new ValidationMessage(Messages - .getString("conformidades.validacao.range_faixas"), - "conformidade.diretriz.id")); + .getString("faixas.validacao.percentual_conformidade"), + "faixa.percentualConformidade")); } } - + private void criarNovoParametroAvaliacao(List faixas, int tipo) { ParametroCalculo ultimoParametroAvaliacao = paDAO.getLastParametrosCalculo(); boolean ok = false; diff --git a/Desenvolvimento/Codificacao/e-Selo/src/main/java/br/com/eselo/controller/ParametroCalculoController.java b/Desenvolvimento/Codificacao/e-Selo/src/main/java/br/com/eselo/controller/ParametroCalculoController.java index 49d834e..9ac8007 100644 --- a/Desenvolvimento/Codificacao/e-Selo/src/main/java/br/com/eselo/controller/ParametroCalculoController.java +++ b/Desenvolvimento/Codificacao/e-Selo/src/main/java/br/com/eselo/controller/ParametroCalculoController.java @@ -34,9 +34,11 @@ import br.com.eselo.component.Messages; import br.com.eselo.component.UsuarioWeb; import br.com.eselo.dao.ParametroCalculoDAO; import br.com.eselo.dao.RecomendacaoDAO; +import br.com.eselo.dao.UsuarioDAO; import br.com.eselo.model.Criterio; import br.com.eselo.model.ParametroCalculo; import br.com.eselo.model.Recomendacao; +import br.com.eselo.model.Usuario; @Resource public class ParametroCalculoController { @@ -138,12 +140,13 @@ public class ParametroCalculoController { @Restrito(descricao = "PARAMETRO_CALCULO_ATUALIZAR") public void atualizar(ParametroCalculo parametroCalculo) { - ParametroCalculo parametroCalculoCadastrado = dao.load(parametroCalculo - .getId()); - parametroCalculoCadastrado.setTsFim(parametroCalculo.getTsFim()); - parametroCalculoCadastrado.setUsuario(parametroCalculo.getUsuario()); - parametroCalculoCadastrado - .setDescricao(parametroCalculo.getDescricao()); + List usuarios = dao.getAllUsuarios(); + ParametroCalculo parametroCalculoCadastrado = dao.load(parametroCalculo.getId()); + + parametroCalculo.setTsInicio(parametroCalculoCadastrado.getTsInicio()); + parametroCalculoCadastrado.setUsuario(new UsuarioDAO(dao.getSession()).load(usuarioWeb.getId())); + parametroCalculo.setUsuario(parametroCalculoCadastrado.getUsuario()); + parametroCalculoCadastrado.setDescricao(parametroCalculo.getDescricao()); if (parametroCalculo.getTsFim() != null) { if (parametroCalculo.getTsFim().before( @@ -159,14 +162,14 @@ public class ParametroCalculoController { "parametroCalculo.descricao")); } - // result.include("usuarioList", dao.getAllUsuarios()); - // result.include("recomendacaoList", new - // RecomendacaoDAO(dao.getSession()).listAllAtivas(this.dao.load(parametroCalculo.getId()))); - System.err.println(parametroCalculo.getId() - + "Id do parametro de calculo"); - - validator.onErrorUsePageOf(ParametroCalculoController.class).form( - parametroCalculo.getId()); + + + result.include("usuarioList", usuarios); + result.include("recomendacaoList", parametroCalculoCadastrado.getRecomendacoes()); + result.include("parametroCalculo", parametroCalculoCadastrado); + + + validator.onErrorUsePageOf(ParametroCalculoController.class).form(parametroCalculo.getId()); dao.saveOrUpdate(parametroCalculoCadastrado); diff --git a/Desenvolvimento/Codificacao/e-Selo/src/main/java/br/com/eselo/controller/RecomendacaoController.java b/Desenvolvimento/Codificacao/e-Selo/src/main/java/br/com/eselo/controller/RecomendacaoController.java index 7719b35..fb71db3 100644 --- a/Desenvolvimento/Codificacao/e-Selo/src/main/java/br/com/eselo/controller/RecomendacaoController.java +++ b/Desenvolvimento/Codificacao/e-Selo/src/main/java/br/com/eselo/controller/RecomendacaoController.java @@ -66,31 +66,8 @@ public class RecomendacaoController { @Restrito(descricao = "RECOMENDACAO_ADICIONAR") public void adicionar(Recomendacao recomendacao) { - if (recomendacao.getNumeroRecomendacao() == null) { - validator.add(new ValidationMessage( - "É obrigatório informar o número da recomendação.", - "recomendacao.numeroRecomendacao")); - } else if (hasNumeroRecomendacaoNoParametroAtual(recomendacao)) { - validator - .add(new ValidationMessage( - "Já existe uma recomendação ativa com esse numero. Recomendação não cadastrada.", - "recomendacao.numeroRecomendacao")); - } else if (recomendacao.getDescricao() == null) { - validator.add(new ValidationMessage( - "É obrigatório informar a descrição da recomendação.", - "recomendacao.descricao")); - } else if (recomendacao.getPeso() == null) { - validator.add(new ValidationMessage( - "É obrigatório informar o peso da recomendação.", - "recomendacao.peso")); - } else if (recomendacao.getPeso() <= 0) { - validator.add(new ValidationMessage( - "O peso da recomendação deve ser maior que zero.", - "recomendacao.peso")); - } + validarRecomendacao(recomendacao); - result.include("parametroCalculoId", recomendacao.getParametroCalculo().getId()); - result.include("parametroCalculo", recomendacao.getParametroCalculo()); validator.onErrorUsePageOf(RecomendacaoController.class).form( recomendacao.getParametroCalculo().getId()); @@ -107,34 +84,12 @@ public class RecomendacaoController { recomendacaoCadastrada.setNumeroRecomendacao(recomendacao.getNumeroRecomendacao()); recomendacaoCadastrada.setDescricao(recomendacao.getDescricao()); recomendacaoCadastrada.setPeso(recomendacao.getPeso()); + recomendacao.setCriterios(recomendacaoCadastrada.getCriterios()); - if (recomendacao.getNumeroRecomendacao() == null) { - validator.add(new ValidationMessage( - "É obrigatório informar o número da recomendação.", - "recomendacao.numeroRecomendacao")); - } else if (hasNumeroRecomendacaoNoParametroAtual(recomendacao)) { - validator - .add(new ValidationMessage( - "Já existe uma recomendação ativa com esse numero. Recomendação não atualizada.", - "recomendacao.numeroRecomendacao")); - } else if (recomendacao.getDescricao() == null) { - validator.add(new ValidationMessage( - "É obrigatório informar a descrição da recomendação.", - "recomendacao.descricao")); - } else if (recomendacao.getPeso() == null) { - validator.add(new ValidationMessage( - "É obrigatório informar o peso da recomendação.", - "recomendacao.peso")); - } else if (recomendacao.getPeso() <= 0) { - validator.add(new ValidationMessage( - "O peso da recomendação deve ser maior que zero.", - "recomendacao.peso")); - } - - result.include("parametroCalculoId", recomendacao.getParametroCalculo().getId()); - result.include("parametroCalculo", recomendacao.getParametroCalculo()); - validator.onErrorUsePageOf(ParametroCalculoController.class).form( - recomendacaoCadastrada.getParametroCalculo().getId()); + validarRecomendacao(recomendacao); + + validator.onErrorUsePageOf(RecomendacaoController.class).form( + recomendacaoCadastrada.getId()); criarNovoParametroAvaliacao(recomendacao, ParametroCalculoController.MODO_EDIT); @@ -194,6 +149,41 @@ public class RecomendacaoController { new CopiadorParametroCalculo().criarNovoParametroCalculo(ultimoParametroAvaliacao, usuarioWeb, dao.getSession()); } } + + + private void validarRecomendacao(Recomendacao recomendacao){ + + + if (recomendacao.getNumeroRecomendacao() == null) { + validator.add(new ValidationMessage( + "É obrigatório informar o número da recomendação.", + "recomendacao.numeroRecomendacao")); + } else if (hasNumeroRecomendacaoNoParametroAtual(recomendacao)) { + validator + .add(new ValidationMessage( + "Já existe uma recomendação ativa com esse numero. Recomendação não atualizada.", + "recomendacao.numeroRecomendacao")); + } else if (recomendacao.getDescricao() == null) { + validator.add(new ValidationMessage( + "É obrigatório informar a descrição da recomendação.", + "recomendacao.descricao")); + } else if (recomendacao.getPeso() == null) { + validator.add(new ValidationMessage( + "É obrigatório informar o peso da recomendação.", + "recomendacao.peso")); + } else if (recomendacao.getPeso() <= 0) { + validator.add(new ValidationMessage( + "O peso da recomendação deve ser maior que zero.", + "recomendacao.peso")); + } + + + result.include("parametroCalculoId", recomendacao.getParametroCalculo().getId()); + result.include("parametroCalculo", recomendacao.getParametroCalculo()); + + } + + /** * Recebe uma recomendação antes de ser cadastrada e verifica se ja existe uma @@ -205,18 +195,22 @@ public class RecomendacaoController { private boolean hasNumeroRecomendacaoNoParametroAtual(Recomendacao recomendacao) { boolean retorno = false; - List listaCadastrada = dao.recomendacoesDoUltimoParametro(dao - .getLastParametrosCalculo()); - for (Recomendacao recomendacaoCadastrada : listaCadastrada) { - if (recomendacaoCadastrada.getNumeroRecomendacao().longValue() == recomendacao - .getNumeroRecomendacao().longValue()) { - if (recomendacao.getId() != null) { - if (recomendacaoCadastrada.getId().longValue() != recomendacao - .getId().longValue()) { + if(recomendacao.getDescricao()!=null && + recomendacao.getNumeroRecomendacao()!=null && + recomendacao.getPeso()!=null ){ + List listaCadastrada = dao.recomendacoesDoUltimoParametro(dao + .getLastParametrosCalculo()); + for (Recomendacao recomendacaoCadastrada : listaCadastrada) { + if (recomendacaoCadastrada.getNumeroRecomendacao().longValue() == recomendacao + .getNumeroRecomendacao().longValue()) { + if (recomendacao.getId() != null) { + if (recomendacaoCadastrada.getId().longValue() != recomendacao + .getId().longValue()) { + return true; + } + } else { return true; } - } else { - return true; } } } diff --git a/Desenvolvimento/Codificacao/e-Selo/src/main/java/br/com/eselo/model/Faixa.java b/Desenvolvimento/Codificacao/e-Selo/src/main/java/br/com/eselo/model/Faixa.java index f0ed062..1877e98 100644 --- a/Desenvolvimento/Codificacao/e-Selo/src/main/java/br/com/eselo/model/Faixa.java +++ b/Desenvolvimento/Codificacao/e-Selo/src/main/java/br/com/eselo/model/Faixa.java @@ -91,7 +91,7 @@ public class Faixa { this.criterio = criterio; } - public double getPercentualConformidade() { + public Double getPercentualConformidade() { return percentualConformidade; } -- libgit2 0.21.2