Commit a1bd62fc39f04e8d1f9a0d223ea65804a0c469f4
1 parent
d8ad23e3
Exists in
master
[Redmine Atendimento #4576]Validação notificação por email
Showing
1 changed file
with
5 additions
and
5 deletions
Show diff stats
cit-almoxarifado-api/src/main/java/br/com/centralit/api/job/NotificacaoRessuprimentoJob.java
@@ -6,6 +6,7 @@ import java.util.Collection; | @@ -6,6 +6,7 @@ import java.util.Collection; | ||
6 | import java.util.Date; | 6 | import java.util.Date; |
7 | import java.util.List; | 7 | import java.util.List; |
8 | 8 | ||
9 | +import org.apache.commons.lang3.StringUtils; | ||
9 | import org.apache.commons.logging.Log; | 10 | import org.apache.commons.logging.Log; |
10 | import org.apache.commons.logging.LogFactory; | 11 | import org.apache.commons.logging.LogFactory; |
11 | import org.quartz.Job; | 12 | import org.quartz.Job; |
@@ -13,8 +14,6 @@ import org.quartz.JobExecutionContext; | @@ -13,8 +14,6 @@ import org.quartz.JobExecutionContext; | ||
13 | import org.quartz.JobExecutionException; | 14 | import org.quartz.JobExecutionException; |
14 | import org.quartz.SchedulerException; | 15 | import org.quartz.SchedulerException; |
15 | 16 | ||
16 | -import com.googlecode.genericdao.search.Field; | ||
17 | - | ||
18 | import br.com.centralit.api.model.ConfiguracaoNotificacaoRessuprimento; | 17 | import br.com.centralit.api.model.ConfiguracaoNotificacaoRessuprimento; |
19 | import br.com.centralit.api.model.ConfiguracaoNotificacaoRessuprimentoGrupo; | 18 | import br.com.centralit.api.model.ConfiguracaoNotificacaoRessuprimentoGrupo; |
20 | import br.com.centralit.api.model.ConfiguracaoNotificacaoRessuprimentoUsuario; | 19 | import br.com.centralit.api.model.ConfiguracaoNotificacaoRessuprimentoUsuario; |
@@ -28,7 +27,6 @@ import br.com.centralit.api.service.impl.NotificacaoGrupoService; | @@ -28,7 +27,6 @@ import br.com.centralit.api.service.impl.NotificacaoGrupoService; | ||
28 | import br.com.centralit.framework.dao.arquitetura.SearchSeven; | 27 | import br.com.centralit.framework.dao.arquitetura.SearchSeven; |
29 | import br.com.centralit.framework.mail.HTMLMail; | 28 | import br.com.centralit.framework.mail.HTMLMail; |
30 | import br.com.centralit.framework.model.Filter; | 29 | import br.com.centralit.framework.model.Filter; |
31 | -import br.com.centralit.framework.model.Notificacao; | ||
32 | import br.com.centralit.framework.model.NotificacaoGrupo; | 30 | import br.com.centralit.framework.model.NotificacaoGrupo; |
33 | import br.com.centralit.framework.model.NotificacaoUsuario; | 31 | import br.com.centralit.framework.model.NotificacaoUsuario; |
34 | import br.com.centralit.framework.model.SearchParams; | 32 | import br.com.centralit.framework.model.SearchParams; |
@@ -208,7 +206,9 @@ public class NotificacaoRessuprimentoJob implements Job { | @@ -208,7 +206,9 @@ public class NotificacaoRessuprimentoJob implements Job { | ||
208 | } | 206 | } |
209 | 207 | ||
210 | if(HTMLMail.permiteEnvioEmail){ | 208 | if(HTMLMail.permiteEnvioEmail){ |
211 | - this.notificacaoUsuarioService.sendNotificacaoEmailGrupo(emailDestinatario, nomeDestinatario, mensagem); | 209 | + if(StringUtils.isNotBlank(emailDestinatario) && StringUtils.isNotBlank(nomeDestinatario) && StringUtils.isNotBlank(mensagem)){ |
210 | + this.notificacaoUsuarioService.sendNotificacaoEmailGrupo(emailDestinatario, nomeDestinatario, mensagem); | ||
211 | + } | ||
212 | } | 212 | } |
213 | 213 | ||
214 | NotificacaoRessuprimentoJob.logger.info("Inclusão da Notificação para o grupo: " + configuracaoGrupo.getGrupo().getNome() + new Date()); | 214 | NotificacaoRessuprimentoJob.logger.info("Inclusão da Notificação para o grupo: " + configuracaoGrupo.getGrupo().getNome() + new Date()); |
@@ -252,7 +252,7 @@ public class NotificacaoRessuprimentoJob implements Job { | @@ -252,7 +252,7 @@ public class NotificacaoRessuprimentoJob implements Job { | ||
252 | mensagem = (String) values[2]; | 252 | mensagem = (String) values[2]; |
253 | } | 253 | } |
254 | 254 | ||
255 | - if(HTMLMail.permiteEnvioEmail){ | 255 | + if(StringUtils.isNotBlank(emailDestinatario) && StringUtils.isNotBlank(nomeDestinatario) && StringUtils.isNotBlank(mensagem)){ |
256 | this.notificacaoUsuarioService.sendNotificacaoEmailGrupo(emailDestinatario, nomeDestinatario, mensagem); | 256 | this.notificacaoUsuarioService.sendNotificacaoEmailGrupo(emailDestinatario, nomeDestinatario, mensagem); |
257 | } | 257 | } |
258 | 258 |