#12 Refactor bulk delete method

Fusionnée
jonathan a fusionné 8 révision(s) à partir de kris/refactor_bulk_delete_methods vers master il y a 4 ans
kris a commenté il y a 4 ans

@jonathan please review

@jonathan please review
jonathan s'est vu assigner cela par kris il y a 4 ans
jonathan a requis les changements il y a 4 ans
jonathan laisser un commentaire

a couple changes requested

@@ -361,3 +355,3 @@
return bulkDeleteWhere(condition, params);
}

public int bulkDeleteWhere(String whereClause) {
jonathan a commenté il y a 4 ans

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 a commenté il y a 4 ans

No use of var in low-level code please

No use of `var` in low-level code please
jonathan révisé il y a 4 ans
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 a commenté il y a 4 ans

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 a fermé cette pull request il y a 4 ans
jonathan a supprimé la branche kris/refactor_bulk_delete_methods il y a 4 ans

Relecteurs

jonathan a requis les changements il y a 4 ans
La demande de tirage a été fusionnée en e2783ac80f.
Connectez-vous pour rejoindre cette conversation.
Pas de relecteur
Pas d'étiquette
Aucun jalon
Pas d'assignataires
2 participants
Notifications
Échéance

Aucune échéance n'a été définie.

Dépendances

La demande de fusion n'a actuellement pas de dépendance.

Chargement…
Il n'existe pas encore de contenu.