#53 Fully forbid network owner deletion

Fusionnée
jonathan a fusionné 7 révision(s) à partir de kris/forbid_network_owner_deletion vers master il y a 4 ans
kris a commenté il y a 4 ans

@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 s'est vu assigner cela par kris il y a 4 ans
jonathan a requis les changements il y a 4 ans
jonathan laisser un commentaire

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) {
jonathan a commenté il y a 4 ans

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) {
jonathan a commenté il y a 4 ans

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<>());
jonathan a commenté il y a 4 ans

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 a fermé cette pull request il y a 4 ans
jonathan a supprimé la branche kris/forbid_network_owner_deletion il y a 4 ans

Relecteurs

jonathan a requis les changements il y a 4 ans
La demande de tirage a été fusionnée en 189eda7d46.
Connectez-vous pour rejoindre cette conversation.
Pas de relecteur
Pas d'étiquette
Aucun jalon
Pas d'assignataires
2 participants
Notifications
Échéance

Aucune échéance n'a été définie.

Dépendances

La demande de fusion n'a actuellement pas de dépendance.

Chargement…
Il n'existe pas encore de contenu.