demoiselle / framework

Repositório principal contendo o Core e Extensions: JPA, Security, WS
https://demoiselle.org
127 stars 77 forks source link

Comportamento Errado do MergeFull no AbstractDAO #110

Closed botelhojp closed 6 years ago

botelhojp commented 6 years ago

Verificar comportamento do método mergeFull da classe AbstractDAO. Conforme relato, ela retorna o entity não modificado. Se o método retornar o próprio retorno do merge o entity viria com o valor modificado.

https://github.com/demoiselle/framework/blob/422fb8f9069d35105b2ac0ead7e83fe52c3d44c7/demoiselle-crud/src/main/java/org/demoiselle/jee/crud/AbstractDAO.java#L131

Segue proposta:

    @Override
    public T mergeFull(T entity) {
        try {
            return getEntityManager().merge(entity);
        } catch (Exception e) {
            throw new DemoiselleCrudException("Não foi possível salvar", e);
        }
    }