From b429bf54454884ee678920ab3c3f58867fcb04d3 Mon Sep 17 00:00:00 2001 From: Juho Haapakoski Date: Wed, 16 Oct 2024 18:08:49 +0300 Subject: [PATCH] AE-2274: Refactor jeesql parameters to use kebab-case --- .../service/energiatodistus_destruction.clj | 49 +++++++++---------- .../etp/db/energiatodistus-destruction.sql | 46 ++++++++--------- 2 files changed, 47 insertions(+), 48 deletions(-) diff --git a/etp-core/etp-backend/src/main/clj/solita/etp/service/energiatodistus_destruction.clj b/etp-core/etp-backend/src/main/clj/solita/etp/service/energiatodistus_destruction.clj index 618a3229a..69e800fdc 100644 --- a/etp-core/etp-backend/src/main/clj/solita/etp/service/energiatodistus_destruction.clj +++ b/etp-core/etp-backend/src/main/clj/solita/etp/service/energiatodistus_destruction.clj @@ -13,19 +13,19 @@ (db/require-queries 'energiatodistus-destruction) (defn- anonymize-energiatodistus! [db energiatodistus-id] - (energiatodistus-destruction-db/anonymize-energiatodistus! db {:energiatodistus_id energiatodistus-id})) + (energiatodistus-destruction-db/anonymize-energiatodistus! db {:energiatodistus-id energiatodistus-id})) (defn- destroy-energiatodistus-audit-data! [db energiatodistus-id] - (energiatodistus-destruction-db/destroy-energiatodistus-audit! db {:energiatodistus_id energiatodistus-id})) + (energiatodistus-destruction-db/destroy-energiatodistus-audit! db {:energiatodistus-id energiatodistus-id})) (defn- destroy-energiatodistus-oikeellisuuden-valvonta! [db energiatodistus-id] - (energiatodistus-destruction-db/destroy-energiatodistus-oikeellisuuden-valvonta-note! db {:energiatodistus_id energiatodistus-id}) - (energiatodistus-destruction-db/destroy-energiatodistus-oikeellisuuden-valvonta-virhe! db {:energiatodistus_id energiatodistus-id}) - (energiatodistus-destruction-db/destroy-energiatodistus-oikeellisuuden-valvonta-tiedoksi! db {:energiatodistus_id energiatodistus-id}) - (energiatodistus-destruction-db/destroy-energiatodistus-oikeellisuuden-valvonta-toimenpide! db {:energiatodistus_id energiatodistus-id}) + (energiatodistus-destruction-db/destroy-energiatodistus-oikeellisuuden-valvonta-note! db {:energiatodistus-id energiatodistus-id}) + (energiatodistus-destruction-db/destroy-energiatodistus-oikeellisuuden-valvonta-virhe! db {:energiatodistus-id energiatodistus-id}) + (energiatodistus-destruction-db/destroy-energiatodistus-oikeellisuuden-valvonta-tiedoksi! db {:energiatodistus-id energiatodistus-id}) + (energiatodistus-destruction-db/destroy-energiatodistus-oikeellisuuden-valvonta-toimenpide! db {:energiatodistus-id energiatodistus-id}) - (energiatodistus-destruction-db/destroy-energiatodistus-oikeellisuuden-valvonta-toimenpide-audit! db {:energiatodistus_id energiatodistus-id}) - (energiatodistus-destruction-db/destroy-energiatodistus-oikeellisuuden-valvonta-note-audit! db {:energiatodistus_id energiatodistus-id})) + (energiatodistus-destruction-db/destroy-energiatodistus-oikeellisuuden-valvonta-toimenpide-audit! db {:energiatodistus-id energiatodistus-id}) + (energiatodistus-destruction-db/destroy-energiatodistus-oikeellisuuden-valvonta-note-audit! db {:energiatodistus-id energiatodistus-id})) (defn- delete-from-s3 [aws-s3-client file-key] (if (file/file-exists? aws-s3-client file-key) @@ -54,17 +54,17 @@ (delete-from-s3 aws-s3-client file-key)))) (defn- delete-energiatodistus-liite [db aws-s3-client liite-id] - (energiatodistus-destruction-db/destroy-liite! db {:liite_id liite-id}) - (delete-energiatodistus-liite-s3 aws-s3-client liite-id) - (energiatodistus-destruction-db/destroy-liite-audit! db {:liite_id liite-id})) + (energiatodistus-destruction-db/destroy-liite! db {:liite-id liite-id}) + (delete-energiatodistus-liite-s3 aws-s3-client liite-id) + (energiatodistus-destruction-db/destroy-liite-audit! db {:liite-id liite-id})) (defn- destroy-energiatodistus-liitteet [db aws-s3-client energiatodistus-id] (let [liitteet (map :liite-id (energiatodistus-destruction-db/select-to-be-destroyed-liitteet-by-energiatodistus-id db {:energiatodistus_id energiatodistus-id}))] (run! #(delete-energiatodistus-liite db aws-s3-client %) liitteet))) (defn- destroy-viesti! [db viesti-id] - (energiatodistus-destruction-db/destroy-viesti-reader! db {:viesti_id viesti-id}) - (energiatodistus-destruction-db/destroy-viesti! db {:viesti_id viesti-id})) + (energiatodistus-destruction-db/destroy-viesti-reader! db {:viesti-id viesti-id}) + (energiatodistus-destruction-db/destroy-viesti! db {:viesti-id viesti-id})) (defn- delete-viestiketju-liite-s3 [aws-s3-client viestiketju-id liite-id] (let [file-key (viesti-service/file-path viestiketju-id liite-id)] @@ -73,21 +73,21 @@ (delete-from-s3 aws-s3-client file-key)))) (defn- destroy-viestiketju-liitteet [db aws-s3-client viestiketju-id] - (let [viestiketju-liite-ids (energiatodistus-destruction-db/select-liitteet-by-viestiketju-id db {:viestiketju_id viestiketju-id})] + (let [viestiketju-liite-ids (energiatodistus-destruction-db/select-liitteet-by-viestiketju-id db {:viestiketju-id viestiketju-id})] (run! #(delete-viestiketju-liite-s3 aws-s3-client viestiketju-id (:viesti-liite-id %)) viestiketju-liite-ids) - (energiatodistus-destruction-db/destroy-viestiketju-liite! db {:viestiketju_id viestiketju-id}) - (energiatodistus-destruction-db/destroy-viestiketju-liite-audit! db {:viestiketju_id viestiketju-id}))) + (energiatodistus-destruction-db/destroy-viestiketju-liite! db {:viestiketju-id viestiketju-id}) + (energiatodistus-destruction-db/destroy-viestiketju-liite-audit! db {:viestiketju-id viestiketju-id}))) (defn- destroy-viestiketju [db aws-s3-client viestiketju-id] - (let [viestit (energiatodistus-destruction-db/select-viestit-by-viestiketju-id db {:viestiketju_id viestiketju-id})] - (energiatodistus-destruction-db/destroy-vastaanottaja! db {:viestiketju_id viestiketju-id}) + (let [viestit (energiatodistus-destruction-db/select-viestit-by-viestiketju-id db {:viestiketju-id viestiketju-id})] + (energiatodistus-destruction-db/destroy-vastaanottaja! db {:viestiketju-id viestiketju-id}) (destroy-viestiketju-liitteet db aws-s3-client viestiketju-id) (run! #(destroy-viesti! db (:viesti-id %)) viestit) - (energiatodistus-destruction-db/destroy-viestiketju! db {:viestiketju_id viestiketju-id}) - (energiatodistus-destruction-db/destroy-viestiketju-audit! db {:viestiketju_id viestiketju-id}))) + (energiatodistus-destruction-db/destroy-viestiketju! db {:viestiketju-id viestiketju-id}) + (energiatodistus-destruction-db/destroy-viestiketju-audit! db {:viestiketju-id viestiketju-id}))) (defn- check-oikeellisuuden-valvonta-viestiketjut [db vo-toimenpide-id] - (let [viestiketjut (energiatodistus-destruction-db/select-viestiketjut-by-vo-toimenpide-id db {:vo_toimenpide_id vo-toimenpide-id})] + (let [viestiketjut (energiatodistus-destruction-db/select-viestiketjut-by-vo-toimenpide-id db {:vo-toimenpide-id vo-toimenpide-id})] ;; These should be empty as they are destroyed via energiatodistus? Do something if they are not? ;; Maybe unlink the valvonta from viestiketju or just delete them? ;; For now just log an error if this happens as this should not happen. @@ -95,12 +95,12 @@ (log/error "There exists one or many viestiketju for oikeellisuuden valvonta (id: " vo-toimenpide-id ")")))) (defn- check-oikeellisuuden-valvontojen-viestiketjut [db energiatodistus-id] - (let [vo-toimenpide-ids (map :id (energiatodistus-destruction-db/select-vo-toimenpiteet-by-energiatodistus-id db {:energiatodistus_id energiatodistus-id}))] + (let [vo-toimenpide-ids (map :id (energiatodistus-destruction-db/select-vo-toimenpiteet-by-energiatodistus-id db {:energiatodistus-id energiatodistus-id}))] (run! (partial check-oikeellisuuden-valvonta-viestiketjut db) vo-toimenpide-ids))) (defn- destroy-energiatodistus-viestiketjut [db aws-s3-client energiatodistus-id] - (let [viestiketjut-ids (map :viestiketju-id (energiatodistus-destruction-db/select-viestiketjut-by-energiatodistus-id db {:energiatodistus_id energiatodistus-id}))] - (run! #(destroy-viestiketju db aws-s3-client %) viestiketjut-ids) + (let [viestiketjut-ids (map :viestiketju-id (energiatodistus-destruction-db/select-viestiketjut-by-energiatodistus-id db {:energiatodistus-id energiatodistus-id}))] + (run! #(destroy-viestiketju db aws-s3-client %) viestiketjut-ids) (check-oikeellisuuden-valvontojen-viestiketjut db energiatodistus-id))) (defn- destroy-expired-energiatodistus! [db aws-s3-client energiatodistus-id] @@ -121,4 +121,3 @@ (log/info (str "Destruction of expired energiatodistukset initiated.")) (let [expired-todistukset-ids (get-currently-expired-todistus-ids db)] (run! #(destroy-expired-energiatodistus! db aws-s3-client %) expired-todistukset-ids))) - diff --git a/etp-core/etp-backend/src/main/sql/solita/etp/db/energiatodistus-destruction.sql b/etp-core/etp-backend/src/main/sql/solita/etp/db/energiatodistus-destruction.sql index eca9ef0be..cd952cc7a 100644 --- a/etp-core/etp-backend/src/main/sql/solita/etp/db/energiatodistus-destruction.sql +++ b/etp-core/etp-backend/src/main/sql/solita/etp/db/energiatodistus-destruction.sql @@ -12,112 +12,112 @@ where voimassaolo_paattymisaika < current_date -- name: select-vo-toimenpiteet-by-energiatodistus-id select id as vo_toimenpide_id from vo_toimenpide -where energiatodistus_id = :energiatodistus_id; +where energiatodistus_id = :energiatodistus-id; -- name: select-to-be-destroyed-liitteet-by-energiatodistus-id select id as liite_id from liite -where energiatodistus_id = :energiatodistus_id; +where energiatodistus_id = :energiatodistus-id; -- name: select-liitteet-by-viestiketju-id select id as viesti_liite_id from viesti_liite -where viestiketju_id = :viestiketju_id; +where viestiketju_id = :viestiketju-id; -- name: select-viestiketjut-by-energiatodistus-id select id as viestiketju_id from viestiketju -where energiatodistus_id = :energiatodistus_id; +where energiatodistus_id = :energiatodistus-id; -- name: select-viestiketjut-by-vo-toimenpide-id select id as viestiketju_id from viestiketju -where vo_toimenpide_id = :vo_toimenpide_id; +where vo_toimenpide_id = :vo-toimenpide-id; -- name: select-viestit-by-viestiketju-id select id as viesti_id from viesti -where viestiketju_id = :viestiketju_id; +where viestiketju_id = :viestiketju-id; -- name: destroy-energiatodistus-oikeellisuuden-valvonta-toimenpide! delete from vo_toimenpide -where energiatodistus_id = :energiatodistus_id; +where energiatodistus_id = :energiatodistus-id; -- name: destroy-energiatodistus-oikeellisuuden-valvonta-toimenpide-audit! delete from audit.vo_toimenpide -where energiatodistus_id = :energiatodistus_id; +where energiatodistus_id = :energiatodistus-id; -- name: destroy-energiatodistus-oikeellisuuden-valvonta-note! delete from vo_note -where energiatodistus_id = :energiatodistus_id; +where energiatodistus_id = :energiatodistus-id; -- name: destroy-energiatodistus-oikeellisuuden-valvonta-note-audit! delete from audit.vo_note -where energiatodistus_id = :energiatodistus_id; +where energiatodistus_id = :energiatodistus-id; -- name: destroy-energiatodistus-oikeellisuuden-valvonta-virhe! delete from vo_virhe -where toimenpide_id in (select id from vo_toimenpide where energiatodistus_id = :energiatodistus_id); +where toimenpide_id in (select id from vo_toimenpide where energiatodistus_id = :energiatodistus-id); -- name: destroy-energiatodistus-oikeellisuuden-valvonta-tiedoksi! delete from vo_tiedoksi -where toimenpide_id in (select id from vo_toimenpide where energiatodistus_id = :energiatodistus_id); +where toimenpide_id in (select id from vo_toimenpide where energiatodistus_id = :energiatodistus-id); -- name: destroy-liite! delete from liite -where id = :liite_id; +where id = :liite-id; -- name: destroy-liite-audit! delete from audit.liite -where id = :liite_id; +where id = :liite-id; -- name: destroy-viesti-reader! delete from viesti_reader -where viesti_id = :viesti_id; +where viesti_id = :viesti-id; -- name: destroy-viestiketju-liite! delete from viesti_liite -where viestiketju_id = :viestiketju_id; +where viestiketju_id = :viestiketju-id; -- name: destroy-viestiketju-liite-audit! delete from audit.viesti_liite -where viestiketju_id = :viestiketju_id; +where viestiketju_id = :viestiketju-id; -- name: destroy-viesti! delete from viesti -where id = :viesti_id; +where id = :viesti-id; -- name: destroy-viestiketju! delete from viestiketju -where id = :viestiketju_id; +where id = :viestiketju-id; -- name: destroy-viestiketju-audit! delete from audit.viestiketju -where id = :viestiketju_id; +where id = :viestiketju-id; -- name: destroy-vastaanottaja! delete from vastaanottaja -where viestiketju_id = :viestiketju_id; +where viestiketju_id = :viestiketju-id; -- name: destroy-energiatodistus-audit! delete from audit.energiatodistus -where id = :energiatodistus_id; +where id = :energiatodistus-id; -- name: anonymize-energiatodistus! update energiatodistus @@ -343,5 +343,5 @@ set laskutettava_yritys_defined = false, valvonta$pending = false, valvonta$valvoja_id = null -where id = :energiatodistus_id; +where id = :energiatodistus-id;