#12 Refactor bulk delete method

Sapludināts
jonathan sapludināja 8 revīzijas no kris/refactor_bulk_delete_methods uz master pirms 4 gadiem
kris komentēja pirms 4 gadiem

@jonathan please review

@jonathan please review
jonathan tika piešķirta problēma no kris pirms 4 gadiem
jonathan pieprasīja izmaiņas pirms 4 gadiem
jonathan atstāja komentāru

a couple changes requested

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

public int bulkDeleteWhere(String whereClause) {
jonathan komentēja pirms 4 gadiem

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
@@ -353,3 +343,1 @@
} else {
query = session.createQuery(deleteSql + field + " = :" + field)
.setParameter(field, value);
final var params = new HashMap<String, Object>();
jonathan komentēja pirms 4 gadiem

No use of var in low-level code please

No use of `var` in low-level code please
jonathan recenzēja pirms 4 gadiem
@@ -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 komentēja pirms 4 gadiem

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 aizvēra šo izmaiņu pieprasījumu pirms 4 gadiem
jonathan izdzēsa atzaru kris/refactor_bulk_delete_methods pirms 4 gadiem

Recenzenti

jonathan pieprasīja izmaiņas pirms 4 gadiem
Izmaiņu pieprasījums tika sapludināts ar revīziju e2783ac80f.
Pierakstieties, lai pievienotos šai sarunai.
Nav recenzentu
Nav etiķešu
Nav atskaites punktu
Nav atbildīgo
2 dalībnieki
Paziņojumi
Izpildes termiņš

Izpildes termiņš nav uzstādīts.

Atkarības

Šim izmaiņu pieprasījumam pagaidām nav nevienas atkarības.

Notiek ielāde…
Vēl nav satura.