Refactor bulk delete method

Neuer Pull-Request

Diskussion 0 Commits 8 Geänderte Dateien 1

@jonathan please review

jonathan wurde von kris vor 4 Jahren zugewiesen

jonathan hat vor 4 Jahren Änderungen angefragt

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 hat vor 4 Jahren kommentiert

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

jonathan hat vor 4 Jahren überprüft

wizard-server/src/main/java/org/cobbzilla/wizard/dao/AbstractCRUDDAO.java

jonathan hat dieses Issue vor 4 Jahren aus einem Commit referenziert

Refactor bulk delete method (#12) Merge branch 'master' into kris/refactor_bulk_delete_methods Allow null as parameters in bulk delete method Use StringBuilder instead of appending Strings Use strict types instead of var Add new method for bulk delete with old signature Do not use SingletonMap so it may appended Set bulkDelete's parameters to be non null Refactor bulkDelete methods Co-authored-by: jonathan <jonathan@noreply.git.bubblev.org> Co-authored-by: Kristijan Mitrovic <kmitrovic@itekako.com> Reviewed-on: https://git.bubblev.org/bubblev/cobbzilla-wizard/pulls/12

jonathan hat diesen Pull-Request vor 4 Jahren geschlossen

jonathan löschte die Branch kris/refactor_bulk_delete_methods vor 4 Jahren

Reviewer

jonathan hat vor 4 Jahren Änderungen angefragt

Der Pull Request wurde als e2783ac80f zusammengeführt.

Anmelden, um an der Diskussion teilzunehmen.

jonathan

Kein Label

Kein Meilenstein

jonathan

2 Beteiligte

Nachrichten

Fällig am

Kein Fälligkeitsdatum gesetzt.

Abhängigkeiten

Dieser Pull-Request hat momentan keine Abhängigkeiten.

@@ -353,3 +343,1 @@
         } else {
             query = session.createQuery(deleteSql + field + " = :" + field)
                     .setParameter(field, value);
         final var params = new HashMap<String, Object>();

@@ -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);

#12 Refactor bulk delete method

Reviewer