bubblev
/
cobbzilla-wizard
6
0
Derivar 0
Código Questões 0 Pedidos de integração 0 Lançamentos 1 Wiki Trabalho
Etiquetas Etapas
Novo pedido de integração

#12 Refactor bulk delete method

Integração executada
jonathan integrou 8 cometimentos de %!<(string=kris/refactor_bulk_delete_methods)/code> em master há 4 anos
Diálogo 0 Cometimentos 8 Ficheiros modificados 1
kris comentado há 4 anos

@jonathan please review

@jonathan please review
jonathan foi atribuída por kris há 4 anos
jonathan alterações solicitadas há 4 anos
jonathan deixou um comentário

a couple changes requested

wizard-server/src/main/java/org/cobbzilla/wizard/dao/AbstractCRUDDAO.java
@@ -361,3 +355,3 @@
return bulkDeleteWhere(condition, params);
}

public int bulkDeleteWhere(String whereClause) {
jonathan comentado há 4 anos

We still need the version of bulkDeleteWhere that just takes a String and no parameters Map

We still need the version of `bulkDeleteWhere` that just takes a String and no `parameters` Map
wizard-server/src/main/java/org/cobbzilla/wizard/dao/AbstractCRUDDAO.java
@@ -353,3 +343,1 @@
} else {
query = session.createQuery(deleteSql + field + " = :" + field)
.setParameter(field, value);
final var params = new HashMap<String, Object>();
jonathan comentado há 4 anos

No use of var in low-level code please

No use of `var` in low-level code please
jonathan reviu há 4 anos
wizard-server/src/main/java/org/cobbzilla/wizard/dao/AbstractCRUDDAO.java
@@ -364,3 +359,4 @@
final Session session = getHibernateTemplate().getSessionFactory().getCurrentSession();
final String deleteSql = "DELETE FROM " + getEntityClass().getSimpleName() + " WHERE " + whereClause;
final Query query = session.createQuery(deleteSql);
parameters.forEach(query::setParameter);
jonathan comentado há 4 anos

Just prefix this line with if (!empty(parameters)) so that bulkDeleteWhere(String) can pass a null parameters arg.

Just prefix this line with `if (!empty(parameters))` so that `bulkDeleteWhere(String)` can pass a null `parameters` arg.
jonathan fechou este pedido de integração há 4 anos
jonathan eliminou o ramo kris/refactor_bulk_delete_methods há 4 anos

Revisores

jonathan alterações solicitadas há 4 anos
A integração constante no pedido foi executada como e2783ac80f.
Inicie a sessão para participar neste diálogo.
Revisores
Solicitar avaliação
Sem revisores
 jonathan
Etiquetas
Aplicar etiquetas
Limpar etiquetas
Sem itens
Sem etiquetas
Etapa
Definir etapa
Limpar etapa
Sem itens
Sem etapa
Responsáveis
Definir responsáveis
Limpar responsáveis
Sem responsáveis
 jonathan
2 Participantes
Notificações
Date limite

Sem data limite definida.

Dependências

Este pedido de integração não tem quaisquer dependências, neste momento.

Escrever Pré-visualizar
Carregando…
Cancelar
Guardar
Ainda não há conteúdo.
Eliminar o ramo 'kris/refactor_bulk_delete_methods'

Eliminar um ramo é algo permanente. NÃO PODERÁ ser revertido. Quer continuar?

Não
Sim