Skip to content
Snippets Groups Projects
Commit c1ec98c9 authored by Yassine Doghri's avatar Yassine Doghri
Browse files

fix(persons): delete person avatar when deleting a person

fixes #419
parent 30a34738
No related branches found
No related tags found
No related merge requests found
......@@ -14,6 +14,7 @@ use App\Entities\Person;
use App\Models\PersonModel;
use CodeIgniter\Exceptions\PageNotFoundException;
use CodeIgniter\HTTP\RedirectResponse;
use Modules\Media\Models\MediaModel;
class PersonController extends BaseController
{
......@@ -151,6 +152,11 @@ class PersonController extends BaseController
public function delete(): RedirectResponse
{
if ($this->person->avatar_id !== null) {
// delete avatar to prevent collision if recreating person
(new MediaModel())->deleteMedia($this->person->avatar);
}
(new PersonModel())->delete($this->person->id);
return redirect()->route('person-list')
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment