Skip to content

Commit ad69668

Browse files
committed
Bugfix
1 parent 0e21d30 commit ad69668

File tree

1 file changed

+13
-16
lines changed

1 file changed

+13
-16
lines changed

app/Http/Controllers/AdminController.php

Lines changed: 13 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -365,15 +365,13 @@ public function editSite(request $request)
365365
//Delete avatar
366366
public function delAvatar()
367367
{
368+
$path = findFile('avatar');
369+
$path = base_path('/assets/linkstack/images/'.$path);
370+
368371
// Delete existing image
369-
$directory = base_path('/assets/linkstack/images/');
370-
$files = scandir($directory);
371-
$pathinfo = "error.error";
372-
foreach($files as $file) {
373-
if (strpos($file, "avatar".'.') !== false) {
374-
$pathinfo = "avatar". "." . pathinfo($file, PATHINFO_EXTENSION);
375-
}}
376-
if(file_exists(base_path('/assets/linkstack/images/').$pathinfo)){File::delete(base_path('/assets/linkstack/images/').$pathinfo);}
372+
if (File::exists($path)) {
373+
File::delete($path);
374+
}
377375

378376
return back();
379377
}
@@ -382,14 +380,13 @@ public function delAvatar()
382380
public function delFavicon()
383381
{
384382
// Delete existing image
385-
$directory = base_path('/assets/linkstack/images/');
386-
$files = scandir($directory);
387-
$pathinfo = "error.error";
388-
foreach($files as $file) {
389-
if (strpos($file, "favicon".'.') !== false) {
390-
$pathinfo = "favicon". "." . pathinfo($file, PATHINFO_EXTENSION);
391-
}}
392-
if(file_exists(base_path('/assets/linkstack/images/').$pathinfo)){File::delete(base_path('/assets/linkstack/images/').$pathinfo);}
383+
$path = findFile('favicon');
384+
$path = base_path('/assets/linkstack/images/'.$path);
385+
386+
// Delete existing image
387+
if (File::exists($path)) {
388+
File::delete($path);
389+
}
393390

394391
return back();
395392
}

0 commit comments

Comments
 (0)