#53 Fully forbid network owner deletion

Sammanfogat
jonathan sammanfogade 7 incheckningar från kris/forbid_network_owner_deletion in i master 4 år sedan
kris kommenterad 4 år sedan

@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 blev tilldelad denna av kris 4 år sedan
jonathan begärda ändringar 4 år sedan
jonathan lämnade en kommentar

some minor changes requested

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

@Override
public int bulkDeleteWhere(@NonNull final String whereClause, @NonNull final Map<String, Object> parameters) {
jonathan kommenterad 4 år sedan

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 kommenterad 4 år sedan

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 kommenterad 4 år sedan

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 år sedan
jonathan tog bort grenen kris/forbid_network_owner_deletion 4 år sedan

Granskare

jonathan begärda ändringar 4 år sedan
Pull-förfrågan har sammanfogats som 189eda7d46.
Logga in för att delta i denna konversation.
Inga granskare
Ingen Etikett
Ingen Milsten
Ingen tilldelad
2 Deltagare
Notiser
Förfallodatum

Inget förfallodatum satt.

Beroenden

Denna pull-förfrågan har för närvarande inga beroenden.

Laddar…
Det finns inget innehåll än.