Refactor bulk delete method

Nieuwe Pull aanvraag

Discussie 0 Commits 8 Bestanden gewijzigd 1

@jonathan please review

jonathan was toegekend door kris 4 jaren geleden

jonathan aangevraagde wijzigingen 4 jaren geleden

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 commented 4 jaren geleden

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 beoordeeld 4 jaren geleden

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

jonathan verwees naar dit probleem vanuit commit 4 jaren geleden'

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 heeft deze pull request gesloten 4 jaren geleden

jonathan heeft 4 jaren geleden de branch kris/refactor_bulk_delete_methods verwijderd.

Reviewers

jonathan aangevraagde wijzigingen 4 jaren geleden

The pull request has been merged as e2783ac80f.

Log in om deel te nemen aan deze discussie.

jonathan

Geen label

Geen mijlpaal

jonathan

2 deelnemers

Notificaties

Vervaldatum

Geen vervaldatum ingesteld.

Afhankelijkheden

Deze pull-aanvraag heeft momenteel geen afhankelijkheden.

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

Reviewers