#53 Fully forbid network owner deletion

Scalone
jonathan scala 7 commity/ów z kris/forbid_network_owner_deletion do master 4 lat temu
kris skomentował(-a) 4 lat temu

@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 zostaje przypisany(-a) przez kris 4 lat temu
jonathan zażądał(-a) zmian 4 lat temu
jonathan zostawił komentarz

some minor changes requested

@@ -347,2 +349,4 @@
}

@Override
public int bulkDeleteWhere(@NonNull final String whereClause, @NonNull final Map<String, Object> parameters) {
jonathan skomentował(-a) 4 lat temu

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

Please put `@Override` on the same line as the method declaration.
@@ -349,0 +355,4 @@
}

@Override
public void delete(@Nullable final Collection<Account> accounts) {
jonathan skomentował(-a) 4 lat temu

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 skomentował(-a) 4 lat temu

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 zamknął(-ęła) ten pull request 4 lat temu
jonathan usuwa gałąź kris/forbid_network_owner_deletion 4 lat temu

Recenzenci

jonathan zażądał(-a) zmian 4 lat temu
Pull Request został scalony jako 189eda7d46.
Zaloguj się, aby dołączyć do tej rozmowy.
Brak recenzentów
Brak etykiety
Brak kamienia milowego
Brak przypisanych
Uczestnicy 2
Powiadomienia
Termin realizacji

Brak ustawionego terminu realizacji.

Zależności

Ten Pull Request nie zawiera w tej chwili żadnych zależności.

Ładowanie…
Nie ma jeszcze treści.