#53 Fully forbid network owner deletion

Sapludināts
jonathan sapludināja 7 revīzijas no kris/forbid_network_owner_deletion uz master pirms 4 gadiem
kris komentēja pirms 4 gadiem

@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 tika piešķirta problēma no kris pirms 4 gadiem
jonathan pieprasīja izmaiņas pirms 4 gadiem
jonathan atstāja komentāru

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 komentēja pirms 4 gadiem

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 komentēja pirms 4 gadiem

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 komentēja pirms 4 gadiem

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 aizvēra šo izmaiņu pieprasījumu pirms 4 gadiem
jonathan izdzēsa atzaru kris/forbid_network_owner_deletion pirms 4 gadiem

Recenzenti

jonathan pieprasīja izmaiņas pirms 4 gadiem
Izmaiņu pieprasījums tika sapludināts ar revīziju 189eda7d46.
Pierakstieties, lai pievienotos šai sarunai.
Nav recenzentu
Nav etiķešu
Nav atskaites punktu
Nav atbildīgo
2 dalībnieki
Paziņojumi
Izpildes termiņš

Izpildes termiņš nav uzstādīts.

Atkarības

Šim izmaiņu pieprasījumam pagaidām nav nevienas atkarības.

Notiek ielāde…
Vēl nav satura.