#53 Fully forbid network owner deletion

Злито
jonathan злито 7 комітів з kris/forbid_network_owner_deletion до master 4 роки тому
kris прокоментував(ла) 4 роки тому

@jonathan please review

Also included fix for Jenkins builds when utils submodules’ links are changed in PRs.

@jonathan please review Also included fix for Jenkins builds when utils submodules' links are changed in PRs.
jonathan був призначений kris 4 роки тому
jonathan зробив запит змін 4 роки тому
jonathan left a comment

some minor changes requested

bubble-server/src/main/java/bubble/dao/account/AccountDAO.java
@@ -347,2 +349,4 @@
}

@Override
public int bulkDeleteWhere(@NonNull final String whereClause, @NonNull final Map<String, Object> parameters) {
jonathan прокоментував(ла) 4 роки тому

Please put @Override on the same line as the method declaration.

Please put `@Override` on the same line as the method declaration.
bubble-server/src/main/java/bubble/dao/account/AccountDAO.java
@@ -349,0 +355,4 @@
}

@Override
public void delete(@Nullable final Collection<Account> accounts) {
jonathan прокоментував(ла) 4 роки тому

Please put @Override on the same line as the method declaration.

Please put `@Override` on the same line as the method declaration.
bubble-server/src/main/java/bubble/service/stream/AppDataCleaner.java
@@ -26,3 +28,3 @@
@Override protected void process() {
try {
final int ct = dataDAO.bulkDeleteWhere("expiration < " + now());
final int ct = dataDAO.bulkDeleteWhere("expiration < " + now(), new HashMap<>());
jonathan прокоментував(ла) 4 роки тому

If we keep the original bulkDeleteWhere we don’t need this change.

If we keep the original `bulkDeleteWhere` we don't need this change.
jonathan closed this pull request 4 роки тому
jonathan видалена гілка kris/forbid_network_owner_deletion 4 роки тому

Рецензенти

jonathan зробив запит змін 4 роки тому
Запит на злиття був влитиий як 189eda7d46.
Підпишіться щоб приєднатися до обговорення.
No reviewers
Без мітки
Етап відсутній
Немає виконавеця
2 учасників
Сповіщення
Дата завершення

Термін виконання не встановлений.

Залежності

Цей запит на злиття в даний час не має залежностей.

Завантаження…
Тут ще немає жодного змісту.