#53 Fully forbid network owner deletion

Zusammengeführt
jonathan hat 7 Commits von kris/forbid_network_owner_deletion nach master vor 4 Jahren zusammengeführt
kris hat vor 4 Jahren kommentiert

@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 wurde von kris vor 4 Jahren zugewiesen
jonathan hat vor 4 Jahren Änderungen angefragt
jonathan hat einen Kommentar hinterlassen

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

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

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

Please put `@Override` on the same line as the method declaration.
@@ -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 hat vor 4 Jahren kommentiert

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 hat diesen Pull-Request vor 4 Jahren geschlossen
jonathan löschte die Branch kris/forbid_network_owner_deletion vor 4 Jahren

Reviewer

jonathan hat vor 4 Jahren Änderungen angefragt
Der Pull Request wurde als 189eda7d46 zusammengeführt.
Anmelden, um an der Diskussion teilzunehmen.
Keine Reviewer
Kein Label
Kein Meilenstein
Niemand zuständig
2 Beteiligte
Nachrichten
Fällig am

Kein Fälligkeitsdatum gesetzt.

Abhängigkeiten

Dieser Pull-Request hat momentan keine Abhängigkeiten.

Laden…
Hier gibt es bis jetzt noch keinen Inhalt.