#53 Fully forbid network owner deletion

マージ済み
jonathan が 7 個のコミットを kris/forbid_network_owner_deletion から master へマージ 4年前
kris4年前 にコメント

@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.
jonathankris が担当者に指名 4年前
jonathan が変更を要請 4年前
jonathan がコメント

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) {
jonathan4年前 にコメント

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) {
jonathan4年前 にコメント

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/service/stream/AppDataCleaner.java
@@ -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<>());
jonathan4年前 にコメント

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年前
jonathan がブランチ kris/forbid_network_owner_deletion を削除 4年前

レビューア

jonathan が変更を要請 4年前
プルリクエストは 189eda7d46 でマージされています。
サインインしてこの会話に参加。
レビューアなし
ラベルなし
マイルストーンなし
担当者なし
2 人の参加者
通知
期日

期日は未設定です。

依存関係

このプルリクエストに依存関係はありません。

読み込み中…
まだ内容がありません