#12 Refactor bulk delete method

Слито
jonathan слито 8 коммит(ов) из kris/refactor_bulk_delete_methods в master 4 лет назад
kris прокомментировал 4 лет назад

@jonathan please review

@jonathan please review
jonathan был назначен kris 4 лет назад
jonathan запросил(а) изменения 4 лет назад
jonathan оставил комментарий

a couple changes requested

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

public int bulkDeleteWhere(String whereClause) {
jonathan прокомментировал 4 лет назад

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 прокомментировал 4 лет назад

No use of var in low-level code please

No use of `var` in low-level code please
jonathan рассмотрел(а) изменения 4 лет назад
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 прокомментировал 4 лет назад

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 closed this pull request 4 лет назад
jonathan удалена ветка kris/refactor_bulk_delete_methods 4 лет назад

Рецензенты

jonathan запросил(а) изменения 4 лет назад
Pull Request был объединен как %!с(string=e2783ac80f).
Войдите, чтобы присоединиться к обсуждению.
Нет рецензентов
Нет меток
Нет этапа
Нет назначенных лиц
2 участников
Уведомления
Срок выполнения

Срок выполнения не установлен.

Зависимости

Этот запрос на слияние в настоящее время не имеет никаких зависимостей.

Загрузка…
Пока нет содержимого.