feat: Clear finalized states for codes

lib/medicode/transcriptions.ex

@@ -5,6 +5,8 @@ defmodule Medicode.Transcriptions do
 
   import Ecto.Query, warn: false
 
+  alias Ecto.Multi
+
   alias Medicode.Repo
 
   alias Medicode.Transcriptions.{
@@ -184,23 +186,32 @@ defmodule Medicode.Transcriptions do
         code_vector_id,
         user_id
       ) do
-    query =
-      TranscriptionChunkCodeVector
-      |> where(
-        [v],
-        v.transcription_chunk_id == ^chunk_id and v.code_vector_id == ^code_vector_id
-      )
-
-    with %TranscriptionChunkCodeVector{} = chunk_vector <- Medicode.Repo.one(query),
-         changeset <-
-           TranscriptionChunkCodeVector.changeset(chunk_vector, %{
-             finalized_at: DateTime.utc_now(),
-             finalized_by_user_id: user_id
-           }),
-         {:ok, chunk_vector} <- Medicode.Repo.update(changeset) do
-      {:ok, chunk_vector}
-    else
-      res -> res
+    multi =
+      Multi.new()
+      |> Multi.one(:chunk_code_vector, fn _changes ->
+        TranscriptionChunkCodeVector
+        |> where(
+          [v],
+          v.transcription_chunk_id == ^chunk_id and v.code_vector_id == ^code_vector_id
+        )
+      end)
+      |> Multi.update(:clear_finalized_state, fn %{chunk_code_vector: chunk_code_vector} ->
+        if is_nil(chunk_code_vector.finalized_at) do
+          Ecto.Changeset.change(chunk_code_vector,
+            finalized_at: DateTime.utc_now() |> DateTime.truncate(:second),
+            finalized_by_user_id: user_id
+          )
+        else
+          Ecto.Changeset.change(chunk_code_vector, finalized_at: nil, finalized_by_user_id: nil)
+        end
+      end)
+
+    case Repo.transaction(multi) do
+      {:ok, %{chunk_code_vector: chunk_code_vector}} ->
+        {:ok, chunk_code_vector}
+
+      {:error, failed_operation, failed_value, changes_so_far} ->
+        {:error, failed_operation, failed_value, changes_so_far}
     end
   end
 

lib/medicode_web/components/transcription_chunk_codings_component.ex

@@ -80,11 +80,18 @@ defmodule MedicodeWeb.Components.TranscriptionTextCodingsComponent do
           >
             Finalize
           </button>
-          <.icon
+          <button
             :if={!is_nil(@transcription_chunk_code_vector.finalized_at)}
-            name="hero-check-badge"
-            class="opacity-40 group-hover:opacity-70"
-          />
+            type="button"
+            class="transition-all duration-300 opacity-0 group-hover:opacity-100 hover:bg-slate-200 px-2 border border-1 border-iron-mountain rounded-md"
+            phx-click="finalize_code"
+            phx-value-chunk_id={@chunk_id}
+            phx-value-code_vector_id={@code_vector.id}
+            phx-target={@myself}
+            title="Clear"
+          >
+            <.icon name="hero-check-badge" class="opacity-40 group-hover:opacity-70" />
+          </button>
         </div>
         <p class="text-base leading-[20.42px]"><%= @code_vector.description %></p>
         <div class="flex flex-row items-center w-full gap-4">
@@ -102,6 +109,7 @@ defmodule MedicodeWeb.Components.TranscriptionTextCodingsComponent do
             type="button"
             aria-label={gettext("close")}
           >
+            Clear
             <.icon name="hero-x-mark-solid" class="opacity-40 group-hover:opacity-70" />
           </button>
         </div>

test/medicode_web/live/transcriptions_live_show_test.exs

@@ -32,7 +32,9 @@ defmodule MedicodeWeb.TranscriptionsLive.ShowTest do
 
     transcription_chunk_code_vector_fixture(%{
       transcription_chunk_id: transcription_chunk1.id,
-      code_vector_id: code_vector2.id
+      code_vector_id: code_vector2.id,
+      finalized_at: DateTime.utc_now(),
+      finalized_by_user_id: user.id
     })
 
     transcription_chunk_code_vector_fixture(%{
@@ -45,7 +47,8 @@ defmodule MedicodeWeb.TranscriptionsLive.ShowTest do
       current_user: user,
       transcription: transcription,
       transcription_chunk1: transcription_chunk1,
-      code_vector1: code_vector1
+      code_vector1: code_vector1,
+      code_vector2: code_vector2
     }
   end
 
@@ -97,5 +100,30 @@ defmodule MedicodeWeb.TranscriptionsLive.ShowTest do
       assert rendered =~ "Finalized Codes"
       assert rendered =~ "001"
     end
+
+    test "renders removes finalized state from selected codes", %{
+      conn: conn,
+      transcription: transcription,
+      transcription_chunk1: transcription_chunk1,
+      code_vector2: code_vector2
+    } do
+      conn = get(conn, "/transcriptions/#{transcription.id}")
+      assert html_response(conn, 200) =~ "MediCode"
+
+      {:ok, view, _html} = live(conn)
+
+      view
+      |> element(
+        "#chunk-#{transcription_chunk1.id}-coding-#{code_vector2.id} button[title=Clear]"
+      )
+      |> render_click()
+
+      _ = :sys.get_state(view.pid)
+
+      rendered = render(view)
+
+      refute rendered =~ "<span class=\"hero-check-badge"
+      refute rendered =~ "Finalized Codes"
+    end
   end
 end