#53 Fully forbid network owner deletion

Yhdistetty
jonathan yhdistetty 7 committia lähteestä kris/forbid_network_owner_deletion kohteeseen master 4 vuotta sitten
kris kommentoi 4 vuotta sitten

@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 was assigned by kris 4 vuotta sitten
jonathan requested changes 4 vuotta sitten
jonathan left a comment

some minor changes requested

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

@Override
public int bulkDeleteWhere(@NonNull final String whereClause, @NonNull final Map<String, Object> parameters) {
jonathan kommentoi 4 vuotta sitten

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 kommentoi 4 vuotta sitten

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 kommentoi 4 vuotta sitten

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 vuotta sitten
jonathan poisti haaran kris/forbid_network_owner_deletion 4 vuotta sitten

Reviewers

jonathan requested changes 4 vuotta sitten
The pull request has been merged as 189eda7d46.
Sign in to join this conversation.
No reviewers
Ei tunnistetta
Ei merkkipaalua
Ei käsittelijää
2 osallistujaa
Ilmoitukset
Määräpäivä

Määräpäivää ei asetettu.

Riippuvuudet

Tällä vetopyynnöllä ei tällä hetkellä ole riippuvuuksia.

Ladataan…
Sisältöä ei vielä ole.