Melhoria

[egrohs]

Estou fazendo uns testes com demoiselle 3. Notei que o método mergeHalf (baseado em PATCH) do AbstractDAO parece incompleto e errado.

1) ta invocando TUDO, os gets (o que é correto) e os sets tb (o que é errado) 2) linha 76, ta passando erradamente a classe entityClass ao invés da instância entity

Abaixo segue uma implementação sugerida que vai direto no field com mapped column...

try {
            StringBuilder sb = new StringBuilder();
            sb.append("UPDATE ");
            sb.append(entity.getClass().getCanonicalName());
            sb.append(" SET ");
            for (Field method : entity.getClass().getDeclaredFields()) {
                method.setAccessible(true);
                if (method.isAnnotationPresent(Column.class)) {
                    Object obj = method.get(entity);
                    if (obj != null) {
                        sb.append(method.getName()).append(" = ").append(obj).append(",");
                    }
                }
            }
            if (sb.toString().endsWith(",")) {
                int i = sb.lastIndexOf(",");
                sb.delete(i, i + 1);
                // TODO ainda falta testar chaves multiplas com @EmbeddedId
                sb.append(" WHERE ").append(CrudUtilHelper.getMethodAnnotatedWithID(entity.getClass())).append(" = ")
                        .append(id);
                getEntityManager().createQuery(sb.toString()).executeUpdate();
            }
            return entity;
        } catch (Exception e) {
            throw new DemoiselleCrudException("Não foi possível salvar", e);
        }

Att. Emanuel Motta Grohs

[UPDATE - Merlin]: apenas formatei o código pra ficar mais amigável ;)