FIX #39 - unassign bug
This commit is contained in:
parent
d343d105fc
commit
da16e247c4
@ -38,16 +38,18 @@ public function remove(AssignPhysicianRequest $request)
|
|||||||
{
|
{
|
||||||
abort_unless(me()->may(Permission::AssignRadiologist), 403);
|
abort_unless(me()->may(Permission::AssignRadiologist), 403);
|
||||||
$study = $this->getStudy('assignedPhysicians');
|
$study = $this->getStudy('assignedPhysicians');
|
||||||
$user = User::active()->findOrFail($request->input('rad_id'));
|
$rad = User::active()->findOrFail($request->input('rad_id'));
|
||||||
|
|
||||||
if ($study->isAssigned($user)) {
|
if ($study->isAssigned($rad)) {
|
||||||
$study->assignedPhysicians()->detach($user->id);
|
$study->assignedPhysicians()->detach($rad->id);
|
||||||
if ($study->assignedPhysicians->count() === 0) {
|
|
||||||
|
if ($study->assignedPhysicians()->count() === 0) {
|
||||||
$study->update(['assigned_at' => null]);
|
$study->update(['assigned_at' => null]);
|
||||||
}
|
}
|
||||||
|
|
||||||
audit()
|
audit()
|
||||||
->did(Activity::Unassign_Physician)
|
->did(Activity::Unassign_Physician)
|
||||||
->notes($user->display_name)
|
->notes($rad->display_name)
|
||||||
->on($study)
|
->on($study)
|
||||||
->log();
|
->log();
|
||||||
}
|
}
|
||||||
|
Loading…
Reference in New Issue
Block a user