AE-2274: Refactor liite deletion to use the same pattern as others

solita · Oct 16, 2024 · 59f8710 · 59f8710
1 parent 001ade1
commit 59f8710
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 5 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
@@ -53,14 +53,13 @@
     (when (file/file-exists? aws-s3-client file-key)
       (delete-from-s3 aws-s3-client file-key))))
 
-(defn- delete-energiatodistus-liite [db aws-s3-client liite]
-  (let [liite-id (:id liite)]
+(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-audit! db {:liite_id liite-id}))
 
 (defn- destroy-energiatodistus-liitteet [db aws-s3-client energiatodistus-id]
-  (let [liitteet (energiatodistus-destruction-db/select-to-be-destroyed-liitteet-by-energiatodistus-id db {:energiatodistus_id energiatodistus-id})]
+  (let [liitteet (map :liite-id (energiatodistus-destruction-db/select-to-be-destroyed-liitteet-by-energiatodistus-id db {:energiatodistus_id energiatodistus-id}))]
     (mapv #(delete-energiatodistus-liite db aws-s3-client %) liitteet)))
 
 (defn- destroy-viesti! [db viesti-id]

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
@@ -15,7 +15,7 @@ from vo_toimenpide
 where energiatodistus_id = :energiatodistus_id;
 
 -- name: select-to-be-destroyed-liitteet-by-energiatodistus-id
-select id
+select id as liite_id
 from liite
 where energiatodistus_id = :energiatodistus_id;