demoiselle / framework

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

Null Pointer Exception #78

Closed ghost closed 7 years ago

ghost commented 7 years ago

Na classe AbstractDAO, o método getMaxResult() não verifica se um atributo OU outro está nulo, ele apenas verifica se os dois estão nulos.

É necessário validar individualmente o Limit e o Offset:

método Original:

private Integer getMaxResult() {
        if (drc.getLimit() == null && drc.getOffset() == null) {
            return paginationConfig.getDefaultPagination();
        }

        return (drc.getLimit() - drc.getOffset()) + 1;
}

Método sugerido:

private Integer getMaxResult() {
        if (drc.getLimit() == null && drc.getOffset() == null) {
                return paginationConfig.getDefaultPagination();
        }
        if (drc.getLimit() == null) {
            drc.setLimit(0);
         }
        if (drc.getOffset() == null) {
        drc.setOffset(0);
    }
    return (drc.getLimit() - drc.getOffset()) + 1;
}
PauloGladson commented 7 years ago

Corrigido, vou publicar no 3.0.1-SNAPSHOT, para a próxima versão.

Obrigado por avisar!

ghost commented 7 years ago

Perfeito!