Error executing function for the second time

Question

Error executing function for the second time

Asked 6 years, 10 months ago

Viewed 163 times

4

Inside the controller I have this function:

public void salvaEncaminhamento() {
    Integer qtdEncaminhamento = 0;

    if (manifestacao.getTbEncaminhamentoCollection() != null) {
        qtdEncaminhamento = manifestacao.getTbEncaminhamentoCollection().size();
    }

    TbUsuario usuario = securityService.getUser();
    for(String idUnidadeStr : idOrgaoDestino) {
        try {
            int idUnidade = Integer.parseInt(idUnidadeStr);

            //atualiza data de modificacao da manifestacao
            manifestacao.setDtUltimaAtualizacao(new Date());
            //atualiza status da manifestacao caso ela ainda esteja como nova
            if (manifestacao.getStStatusManifestacao().equals(StatusManifestacaoEnum.EM_ANALISE.getId()) || manifestacao.getStStatusManifestacao().equals(StatusManifestacaoEnum.SOLICITACAO_RESPONDIDA.getId())) {
                manifestacao.setStStatusManifestacao(StatusManifestacaoEnum.EM_ANDAMENTO.getId());
                dao.edit(manifestacao);
            }

            /*-----------------------------------------------------*/
            /*------- inicio - gravando encaminhamento ------------*/
            /*-----------------------------------------------------*/
            TbEncaminhamento encaminhamento = new TbEncaminhamento();
            encaminhamento.setDsDescricao(dsEncaminhamento);
            encaminhamento.setDtCriacaoEncaminhamento(new Date());
            encaminhamento.setDtEnvioTramite(new Date());
            encaminhamento.setIdManifestacao(manifestacao);
            encaminhamento.setIdUsuarioEnviou(usuario);
            encaminhamento.setIdUnidadeEnviou(usuario.getIdUnidade());
            encaminhamento.setStEncaminhamento(StatusEncaminhamentoEnum.ENCAMINHADA.getId());

            TbUnidade unidade = unidadeDAO.find(idUnidade);
            encaminhamento.setIdUnidadeRecebeu(unidade);

            encaminhamentoDAO.create(encaminhamento);
            qtdEncaminhamento++;
            /*------- fim - gravando encaminhamento ------------*/


            /*-----------------------------------------------------*/
            /*------- inicio - gravando tramite -------------------*/
            /*-----------------------------------------------------*/
            TbTramite tramite = new TbTramite();
            tramite.setDsDescricao(dsEncaminhamento);
            tramite.setDtTramite(new Date());
            tramite.setIdEncaminhamento(encaminhamento);
            tramite.setIdUnidadeEnvio(unidade);
            tramite.setIdUsuarioEmissor(usuario);
            tramite.setStRetornada(BooleanEnum.NAO.getId());
            tramite.setStNotificacao(BooleanEnum.NAO.getId());
            tramite.setStTramitePublico(BooleanEnum.NAO.getId());

            TbUsuario usuarioReceptor = null;
            if (ValidacaoHelper.isNotEmpty(idUsuarioEncaminhamento)) {
                usuarioReceptor = usuarioDAO.find(idUsuarioEncaminhamento);
                tramite.setIdUsuarioReceptor(usuarioReceptor);
            }
            tramiteDAO.create(tramite);

            //----- gravando anexos ---------//
            ArrayList<TbTramitexAnexo> anexosTramite = new ArrayList<>();
            for (TbAnexo anexo : arquivosEncaminhamento) {
                anexoDAO.create(anexo);
                TbTramitexAnexo tbTramitexAnexo = new TbTramitexAnexo();
                tbTramitexAnexo.setIdAnexo(anexo);
                tbTramitexAnexo.setIdTramite(tramite);
                anexosTramite.add(tbTramitexAnexo);
            }
            tramite.setTbTramitexAnexoCollection(anexosTramite);
            tramiteDAO.edit(tramite);
            /*------- fim - gravando tramite ------------*/

            try {
                enviarEmailsEncaminhamento(unidade, usuarioReceptor);
                //----- enviando e-mail ao MANIFESTANTE ---------//
                if (qtdEncaminhamento == 1) {
                    EmailService.Email emailManifestante = emailService.newEmail();
                    if ((manifestacao.getEeEmailUsuario() != null) && "1".equals(manifestacao.getStResposta())) {

                        String nomeManifestante = (manifestacao.getNmPessoa() != null) ? manifestacao.getNmPessoa() : "Manifestante";
                        emailManifestante.addDestinatario(nomeManifestante, manifestacao.getEeEmailUsuario());

                        TbEmailAutomatizado emailAutomatizado = emailAutomatizadoDAO.findByTipo(EmailAutomatizadoEnum.PRIMEIRO_TRAMITE);

                        //adiciona texto e assunto de email padrao
                        StringBuilder emailTextoHtml = new StringBuilder();
                        StringBuilder emailTexto = new StringBuilder();

                        emailManifestante.setAssunto(PalavrasChavesHelper.converterPalavrasChaves(emailAutomatizado.getNmTituloEmail(), manifestacao, false));
                        emailTextoHtml.append(PalavrasChavesHelper.converterPalavrasChaves(emailAutomatizado.getDsEmail(), manifestacao, false));
                        emailTexto.append(PalavrasChavesHelper.converterPalavrasChaves(emailAutomatizado.getDsEmail(), manifestacao, false));

                        emailManifestante.setTextoHtml(emailTextoHtml.toString());
                        emailManifestante.setTextoSemFormatacao(emailTexto.toString());
                        emailService.envia(emailManifestante);
                    }
                }
                MensagemFaceUtil.info("Manifestação encaminhada com sucesso.", "Manifestação encaminhada com sucesso.");
            } catch (Exception e) {
                e.printStackTrace();
                MensagemFaceUtil.alerta("Manifestação encaminhada mas não possível notiticar por e-mail.", "O seu encaminhamento foi realizado com sucesso porém, não foi possível notificar os envolvidos por e-mail.");
            }
        } catch (Exception e) {
            e.printStackTrace();
            MensagemFaceUtil.erro("Erro ao encaminhar Manifestação.", e.getMessage());
        }
    }

    ajustaListaStatusManifestacao();
    preparaListaAreasSolucionadoras();
    mountTabs();
    //zerando variaveis
    idUnidadeEncaminhamento = null;
    idUsuarioEncaminhamento = null;
    mensagemEncaminhamento = null;
    dsEncaminhamento = null;
    idEncaminhamentoPadronizado = null;
    arquivosEncaminhamento = new ArrayList<>();

    showRedirectPageModal();
}

It is called when a manifestation is forwarded to a unit, but in the first forwarding everything occurs ok, but if I try to forward a second manifestation to the same organ this returns the catch message of "Error forwarding Manifestation.".

In the log prints the following error:

13:50:46,298 WARN  [org.hibernate.engine.jdbc.spi.SqlExceptionHelper] (http--127.0.0.1-8080-2) SQL Error: 0, SQLState: 23505
13:50:46,298 ERROR [org.hibernate.engine.jdbc.spi.SqlExceptionHelper] (http--127.0.0.1-8080-2) ERROR: duplicate key value violates unique constraint "uk_6w9ft31ycpj6j9pf1b17k8nw2"
  Detalhe: Key (tbclassificacao_idclassificacao)=(2177) already exists.
13:50:46,300 ERROR [stderr] (http--127.0.0.1-8080-2) javax.persistence.PersistenceException: org.hibernate.exception.ConstraintViolationException: could not execute statement
13:50:46,300 ERROR [stderr] (http--127.0.0.1-8080-2)    at org.hibernate.ejb.AbstractEntityManagerImpl.convert(AbstractEntityManagerImpl.java:1387)

And after these follows with a lot of generic messages that don’t even fit here in the text box, but can be seen here: https://pastebin.com/6Lfs8Z6F

The tbClassification that is in the log is this:

/*
 * To change this template, choose Tools | Templates
 * and open the template in the editor.
 */
package br.com.xti.ouvidoria.model;

import java.io.Serializable;
import java.util.ArrayList;
import java.util.Collection;

import javax.persistence.Basic;
import javax.persistence.Column;
import javax.persistence.Entity;
import javax.persistence.GeneratedValue;
import javax.persistence.GenerationType;
import javax.persistence.Id;
import javax.persistence.JoinColumn;
import javax.persistence.JoinTable;
import javax.persistence.NamedQueries;
import javax.persistence.NamedQuery;
import javax.persistence.OneToMany;
import javax.persistence.Table;
import javax.validation.constraints.NotNull;
import javax.validation.constraints.Size;

/**
 *
 * @author samuel.guimaraes
 */
@Entity
@Table(name = "tbClassificacao")
@NamedQueries({
    @NamedQuery(name = "TbClassificacao.findAll", query = "SELECT t FROM TbClassificacao t"),
    @NamedQuery(name = "TbClassificacao.findByIdClassificacao", query = "SELECT t FROM TbClassificacao t WHERE t.idClassificacao = :idClassificacao"),
    @NamedQuery(name = "TbClassificacao.findByUnidade", query = "SELECT t FROM TbClassificacao t WHERE :idUnidade MEMBER OF t.tbUnidadeCollection"),
    @NamedQuery(name = "TbClassificacao.findByDsClassificacao", query = "SELECT t FROM TbClassificacao t WHERE t.dsClassificacao = :dsClassificacao")})
public class TbClassificacao implements Serializable, Comparable<TbClassificacao> {

    private static final long serialVersionUID = 1L;
    @Id
    @GeneratedValue(strategy = GenerationType.IDENTITY)
    @Basic(optional = false)
    @Column(name = "idClassificacao")
    private Integer idClassificacao;
    @Basic(optional = false)
    @NotNull
    @Size(min = 1, max = 180)
    @Column(name = "dsClassificacao")
    private String dsClassificacao;
    @OneToMany
    private Collection<TbUnidade> tbUnidadeCollection = new ArrayList<>();
    @OneToMany
    @JoinTable(
            name="tbClassificacao_tbSubClassificacao",
            joinColumns={@JoinColumn(name="TbClassificacao_idClassificacao", referencedColumnName="idClassificacao")},
            inverseJoinColumns={@JoinColumn(name="tbSubClassificacaoCollection_idSubClassificacao", referencedColumnName="idSubClassificacao")})
    private Collection<TbSubClassificacao> tbSubClassificacaoCollection = new ArrayList<>();
    @OneToMany
    @JoinTable(
            name="tbManifestacao_tbClassificacao",
            joinColumns={@JoinColumn(name="tbClassificacao_idClassificacao", referencedColumnName="idClassificacao")},
            inverseJoinColumns={@JoinColumn(name="TbManifestacao_idManifestacao", referencedColumnName="idManifestacao")})
    private Collection<TbManifestacao> tbManifestacaoCollection = new ArrayList<>();


    public Collection<TbUnidade> getTbUnidadeCollection() {
        return tbUnidadeCollection;
    }

    public Collection<TbManifestacao> getTbManifestacaoCollection() {
        return tbManifestacaoCollection;
    }

    public void setTbManifestacaoCollection(Collection<TbManifestacao> tbManifestacaoCollection) {
        this.tbManifestacaoCollection = tbManifestacaoCollection;
    }

    public void setTbUnidadeCollection(Collection<TbUnidade> tbUnidadeCollection) {
        this.tbUnidadeCollection = tbUnidadeCollection;
    }

    public TbClassificacao() {
    }

    public TbClassificacao(Integer idClassificacao) {
        this.idClassificacao = idClassificacao;
    }

    public TbClassificacao(Integer idClassificacao, String dsClassificacao) {
        this.idClassificacao = idClassificacao;
        this.dsClassificacao = dsClassificacao;
    }

    public Integer getIdClassificacao() {
        return idClassificacao;
    }

    public void setIdClassificacao(Integer idClassificacao) {
        this.idClassificacao = idClassificacao;
    }

    public String getDsClassificacao() {
        return dsClassificacao;
    }

    public void setDsClassificacao(String dsClassificacao) {
        this.dsClassificacao = dsClassificacao;
    }

    public Collection<TbSubClassificacao> getTbSubClassificacaoCollection() {
        return tbSubClassificacaoCollection;
    }

    public void setTbSubClassificacaoCollection(Collection<TbSubClassificacao> tbSubClassificacaoCollection) {
        this.tbSubClassificacaoCollection = tbSubClassificacaoCollection;
    }

    @Override
    public int hashCode() {
        int hash = 0;
        hash += (idClassificacao != null ? idClassificacao.hashCode() : 0);
        return hash;
    }

    @Override
    public boolean equals(Object object) {
        if (!(object instanceof TbClassificacao)) {
            return false;
        }
        TbClassificacao other = (TbClassificacao) object;
        if ((this.idClassificacao == null && other.idClassificacao != null) || (this.idClassificacao != null && !this.idClassificacao.equals(other.idClassificacao))) {
            return false;
        }
        return true;
    }

    @Override
    public String toString() {
        return String.format("%s > %s", getEntidade(), getDescricao());
    }

    private String getEntidade() {
        return "Classificação";
    }

    private String getDescricao() {
        return dsClassificacao;
    }

    @Override
    public int compareTo(TbClassificacao o) {
        return getDsClassificacao().toUpperCase().compareTo(o.getDsClassificacao().toUpperCase());
    }
}

I am very strange, at first it was to be a project that they are using and should only be install and use. If anyone wants to check out the project is this: https://github.com/culturagovbr/GOG

Updating!

Through the comments and some tests I found a code in the initial load script of data found in their own project that excludes and then creates manually the tbclassificacao_tbunidade, table that performs a Join with the tbclassificacao and where the error occurs.

--Acertos nos mapeamentos gerados

DROP TABLE tbclassificacao_tbunidade;

CREATE TABLE tbclassificacao_tbunidade
(
  tbclassificacao_idclassificacao integer NOT NULL,
  tbunidadecollection_idunidade integer NOT NULL,
  CONSTRAINT fk107ce929d7b9b0e FOREIGN KEY (tbclassificacao_idclassificacao)
      REFERENCES tbclassificacao (idclassificacao) MATCH SIMPLE
      ON UPDATE NO ACTION ON DELETE NO ACTION,
  CONSTRAINT fk107ce92d27d48b9 FOREIGN KEY (tbunidadecollection_idunidade)
      REFERENCES tbunidade (idunidade) MATCH SIMPLE
      ON UPDATE NO ACTION ON DELETE NO ACTION,
  CONSTRAINT tbclassificacao_tbunidade_tbclassificacao_idclassificacao_key UNIQUE (tbclassificacao_idclassificacao, tbunidadecollection_idunidade)
)
WITH (
  OIDS=FALSE
);

Note: Without running this command it does not compile.

Have you tried changing the tbClassificacao the line @GeneratedValue(strategy = GenerationType.IDENTITY) for @GeneratedValue(strategy = GenerationType.AUTO) ?

– NoobSaibot

2017/09/06 at 23:12
I tried here @Wéllingthonm.Download, but same thing

– Mateus

2017/09/08 at 16:08
Actually with this modification when entering the data in the database indicates an error ERROR: relation "tbclassificacao_idclassificacao_seq" does not exist
********** Error **********

– Mateus

2017/09/08 at 18:17
And in case I delete the line ALTER SEQUENCE tbclassificacao_idclassificacao_seq RESTART WITH 5325; to avoid the error, so do not access the register of demonstrations

– Mateus

2017/09/08 at 18:18
inform the first erro complete.

– NoobSaibot

2017/09/08 at 22:39
see if this answer in soEN can solve this problem of first error.

– NoobSaibot

2017/09/08 at 22:48
It doesn’t help me because my problem is the lack of knowledge of JSP, and the creation is done by Hibernate and not directly by SQL

– Mateus

2017/09/11 at 16:49
This error has no relation to JSP. Need to find where you are saving this TbClassificacao, put the rest of the stack trace that there should point out the exact location. Another thing, that ALTER SEQUENCE... seems to indicate that in the table there are already other rows in this table. If you restart the counter and try to insert the same one again idclassificacao, will make a mistake anyway.

– Dudaskank

2017/09/11 at 17:31
I added a script to the above question because I found that the error is not in TbClassificacao and yes in the tbManifestacao_tbClassificacao which carries out a join with the TbClassificacao

– Mateus

2017/09/11 at 18:02
@Dudaskank the question is that several manifestations can be sent containing the same classification, so there could not be this error

– Mateus

2017/09/11 at 18:07
Boy, those names confuse... This update script, did you ever run it? According to him, the single key consists of 2 fields, but the error indicates that the single key is only in the field tbclassificacao_idclassificacao. And about the error of access on the manifestations screen, see if this helps: https://github.com/culturagovbr/GOG/issues/33

– Dudaskank

2017/09/11 at 18:25
@Dudaskank this is another mistake I’ve had to face before and I’ve solved it

– Mateus

2017/09/11 at 18:35
@Dudaskank yes this script is part of the installation process

– Mateus

2017/09/11 at 18:36
The complete error log can be seen here: https://pastebin.com/6Lfs8Z6F

– Mateus

2017/09/11 at 18:50
1

These names... if you look closely, it shows the name of the Constraint on that error snippet: "uk_6w9ft31ycpj6j9pf1b17k8nw2". In this script, the name of the Constraint is another, "tbclassificacao_tbunidade_tbclassificacao_idclassificacao_key",.

– Dudaskank

2017/09/11 at 18:51
1

Look for that name "uk_6w9ft31ycpj6j9pf1b17k8nw2" in the installation script and see if it’s there. If not, see postgresql for the table of this index.

– Dudaskank

2017/09/11 at 18:55
@Dudaskank although not there in the script the DBMS creates, if you see the creation code assigned by the DBMS to the table tbManifestacao_tbClassificacao is there CONSTRAINT uk_6w9ft31ycpj6j9pf1b17k8nw2 UNIQUE (tbclassificacao_idclassificacao),

– Mateus

2017/09/11 at 19:03
@Dudaskank can see here https://pastebin.com/XswRHjSN

– Mateus

2017/09/11 at 19:04
@Dudaskank mistake my you were talking about tbclassificacao_tbunidadeand I of tbmanifestacao_tbclassificacao

– Mateus

2017/09/11 at 19:35
1

@Dudaskank thank you very much, that last comment helped me to resolve the error

– Mateus

2017/09/11 at 19:36

Show 15 more comments

1 answer

Browser other questions tagged hibernate java-ee jsp

You are not signed in. Login or sign up in order to post.

by Mateus • **977** points · Answer 1 · 2017-09-11T19:39:59+00:00

Well, with a lot of analysis and help from everyone who commented it was really evident the error in the fact dq in the table tbmanifestacao_tbclassificacao the field tbclassificacao_idclassificacao could not have as a requirement UNIQUE because more than one manifestation with equal selected classification can be forwarded to the same unit and this table stores the chosen classification with respect to manifestation.

So I tested the creation of the table by removing all these dependencies, and the forwarding worked.

How it was:

-- Table: public.tbmanifestacao_tbclassificacao

-- DROP TABLE public.tbmanifestacao_tbclassificacao;

CREATE TABLE public.tbmanifestacao_tbclassificacao
(
  tbmanifestacao_idmanifestacao integer NOT NULL,
  tbclassificacao_idclassificacao integer NOT NULL,
  CONSTRAINT fk_6w9ft31ycpj6j9pf1b17k8nw2 FOREIGN KEY (tbclassificacao_idclassificacao)
      REFERENCES public.tbclassificacao (idclassificacao) MATCH SIMPLE
      ON UPDATE NO ACTION ON DELETE NO ACTION,
  CONSTRAINT fk_jbkoiljnamdf9icojnuxaskby FOREIGN KEY (tbmanifestacao_idmanifestacao)
      REFERENCES public.tbmanifestacao (idmanifestacao) MATCH SIMPLE
      ON UPDATE NO ACTION ON DELETE NO ACTION,
  CONSTRAINT uk_6w9ft31ycpj6j9pf1b17k8nw2 UNIQUE (tbclassificacao_idclassificacao),
  CONSTRAINT uk_jbkoiljnamdf9icojnuxaskby UNIQUE (tbmanifestacao_idmanifestacao)
)
WITH (
  OIDS=FALSE
);
ALTER TABLE public.tbmanifestacao_tbclassificacao
  OWNER TO postgres;

As I did:

-- Table: public.tbmanifestacao_tbclassificacao

-- DROP TABLE public.tbmanifestacao_tbclassificacao;

CREATE TABLE public.tbmanifestacao_tbclassificacao
(
  tbmanifestacao_idmanifestacao integer NOT NULL,
  tbclassificacao_idclassificacao integer NOT NULL,
  CONSTRAINT fk_6w9ft31ycpj6j9pf1b17k8nw2 FOREIGN KEY (tbclassificacao_idclassificacao)
      REFERENCES public.tbclassificacao (idclassificacao) MATCH SIMPLE
      ON UPDATE NO ACTION ON DELETE NO ACTION,
  CONSTRAINT fk_jbkoiljnamdf9icojnuxaskby FOREIGN KEY (tbmanifestacao_idmanifestacao)
      REFERENCES public.tbmanifestacao (idmanifestacao) MATCH SIMPLE
      ON UPDATE NO ACTION ON DELETE NO ACTION,
  CONSTRAINT uk_jbkoiljnamdf9icojnuxaskby UNIQUE (tbmanifestacao_idmanifestacao)
)
WITH (
  OIDS=FALSE
);
ALTER TABLE public.tbmanifestacao_tbclassificacao
  OWNER TO postgres;