From 18981e8cf17d4dccd02339423dc53f7b06a88481 Mon Sep 17 00:00:00 2001
From: Dr Masroor Ehsan <masroore@gmail.com>
Date: Tue, 7 Jan 2025 02:01:13 +0600
Subject: [PATCH] UI

---
 .../Staff/StudyAssignmentController.php       |  8 ++++++
 .../Staff/StudyHistoryController.php          | 10 +++++++-
 .../Controllers/Staff/WorklistController.php  |  3 +++
 app/Services/SessionHelper.php                | 25 +++++++++++++++++++
 .../staff/studies/show-details.blade.php      | 16 ++++++------
 5 files changed, 54 insertions(+), 8 deletions(-)
 create mode 100644 app/Services/SessionHelper.php

diff --git a/app/Http/Controllers/Staff/StudyAssignmentController.php b/app/Http/Controllers/Staff/StudyAssignmentController.php
index 813e12f..8942d46 100644
--- a/app/Http/Controllers/Staff/StudyAssignmentController.php
+++ b/app/Http/Controllers/Staff/StudyAssignmentController.php
@@ -52,6 +52,14 @@ public function save(AssignPhysicianRequest $request)
         $this->decodeKeys();
         $study = Study::findOrFail($this->key);
         $rad = User::active()->findOrFail($request->input('rad_id'));
+        if ($study->assigned_physician_id !== null) {
+            audit()
+                ->did(Activity::Unassign_Physician)
+                ->notes("Unassigned: {$study->assignedPhysician?->display_name}")
+                ->on($study)
+                ->log();
+        }
+
         $study->update(['assigned_physician_id' => $rad->id, 'assigned_at' => now()]);
         audit()
             ->did(Activity::Assign_Physician)
diff --git a/app/Http/Controllers/Staff/StudyHistoryController.php b/app/Http/Controllers/Staff/StudyHistoryController.php
index a072a84..3167033 100644
--- a/app/Http/Controllers/Staff/StudyHistoryController.php
+++ b/app/Http/Controllers/Staff/StudyHistoryController.php
@@ -7,6 +7,8 @@
 use App\Models\Enums\Permission;
 use App\Models\Study;
 use App\Models\StudyDetails;
+use App\Services\AuditTrail\Activity;
+use App\Services\SessionHelper;
 
 class StudyHistoryController extends HashidControllerBase
 {
@@ -22,6 +24,7 @@ public function view()
 
     public function edit()
     {
+        // SessionHelper::setIntendedUrl();
         abort_unless(auth()->user()->may(Permission::StudyHistoryEdit), 403);
         $this->decodeKeys();
         $details = StudyDetails::historyOnly($this->key);
@@ -39,7 +42,12 @@ public function save(StudyHistoryRequest $request)
         unset($payload['study_id']);
         $details->update($payload);
 
+        audit()
+            ->did(Activity::Study_History_Update)
+            ->on($this->key)
+            ->log();
+
         // return redirect()->route('staff.history.view', _h($this->key));
-        return redirect()->route('staff.history.view', $details->hash);
+        return SessionHelper::redirectIntended();
     }
 }
diff --git a/app/Http/Controllers/Staff/WorklistController.php b/app/Http/Controllers/Staff/WorklistController.php
index a1a0624..62ef8bd 100644
--- a/app/Http/Controllers/Staff/WorklistController.php
+++ b/app/Http/Controllers/Staff/WorklistController.php
@@ -4,11 +4,14 @@
 
 use App\DataTables\WorklistDataTable;
 use App\Http\Controllers\HashidControllerBase;
+use App\Services\SessionHelper;
 
 class WorklistController extends HashidControllerBase
 {
     public function index(WorklistDataTable $dataTable)
     {
+        SessionHelper::setIntendedUrl();
+
         return $dataTable->render('staff.worklist.index');
     }
 }
diff --git a/app/Services/SessionHelper.php b/app/Services/SessionHelper.php
new file mode 100644
index 0000000..1d9df3e
--- /dev/null
+++ b/app/Services/SessionHelper.php
@@ -0,0 +1,25 @@
+<?php
+
+namespace App\Services;
+
+final readonly class SessionHelper
+{
+    public static function setIntendedUrl()
+    {
+        // if (! session()->has('url.intended'))
+        session(['url.intended' => url()->current()]);
+    }
+
+    public static function redirectIntended()
+    {
+        $url = session('url.intended');
+        if (! blank($url)) {
+            session()->forget('url.intended');
+
+            return redirect($url);
+        }
+
+        // return redirect()->route('staff.worklist.index');
+        return redirect()->back();
+    }
+}
diff --git a/resources/views/staff/studies/show-details.blade.php b/resources/views/staff/studies/show-details.blade.php
index b09bb30..6e44479 100644
--- a/resources/views/staff/studies/show-details.blade.php
+++ b/resources/views/staff/studies/show-details.blade.php
@@ -18,13 +18,15 @@
 
     @include('staff.history.partials._history', ['details' => $study->details])
 
-    <div class="card shadow-none bg-transparent border">
-        <div class="card-header">
-            Attachments
+    @if( $study->hasMedia(\App\Models\Study::MEDIA_COLLECTION) )
+        <div class="card shadow-none bg-transparent border">
+            <div class="card-header">
+                Attachments
+            </div>
+            <div class="card-body">
+                @include('staff.history.partials._uploaded-studies-list', ['study' => $study, 'allow_delete' => false, 'table_header' => false])
+            </div>
         </div>
-        <div class="card-body">
-            @include('staff.history.partials._uploaded-studies-list', ['study' => $study, 'allow_delete' => false, 'table_header' => false])
-        </div>
-    </div>
+    @endif
 
 </div>