#53 Fully forbid network owner deletion

Birleştirildi
jonathan 4 yıl önce kris/forbid_network_owner_deletion içindeki 7 işleme master ile birleştirdi
kris 4 yıl önce yorum yaptı

@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 4 yıl önce kris tarafından atandı
jonathan 4 yıl önce değişiklik istedi
jonathan bir yorum bırak

some minor changes requested

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

@Override
public int bulkDeleteWhere(@NonNull final String whereClause, @NonNull final Map<String, Object> parameters) {
jonathan 4 yıl önce yorum yaptı

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 4 yıl önce yorum yaptı

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 4 yıl önce yorum yaptı

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 4 yıl önce değişiklik isteğini kapattı
jonathan kris/forbid_network_owner_deletion dalı silindi 4 yıl önce

Gözden Geçirenler

jonathan 4 yıl önce değişiklik istedi
Değişiklik isteği 189eda7d46 olarak birleştirildi.
Bu konuşmaya katılmak için oturum aç.
Değerlendirici yok
Etiket Yok
Kilometre Taşı Yok
Atanan Kişi Yok
2 Katılımcı
Bildirimler
Bitiş Tarihi

Bitiş tarihi atanmadı.

Bağımlılıklar

Bu çekme isteği henüz bir bağımlılık içermiyor.

Yükleniyor…
Henüz bir içerik yok.