refactor(desfile): remove unused image deletion endpoint
This commit is contained in:
@@ -10,7 +10,6 @@ use App\Domains\Desfile\Services\EntryService;
|
|||||||
use App\Http\Controllers\Controller;
|
use App\Http\Controllers\Controller;
|
||||||
use Illuminate\Http\JsonResponse;
|
use Illuminate\Http\JsonResponse;
|
||||||
use Illuminate\Http\Request;
|
use Illuminate\Http\Request;
|
||||||
use Illuminate\Http\Response;
|
|
||||||
|
|
||||||
class EntryController extends Controller
|
class EntryController extends Controller
|
||||||
{
|
{
|
||||||
@@ -48,10 +47,4 @@ class EntryController extends Controller
|
|||||||
));
|
));
|
||||||
}
|
}
|
||||||
|
|
||||||
public function destroyImage(Request $request): Response
|
|
||||||
{
|
|
||||||
$this->entryService->deleteImage($request->user()->tenant()->firstOrFail());
|
|
||||||
|
|
||||||
return response()->noContent();
|
|
||||||
}
|
|
||||||
}
|
}
|
||||||
|
|||||||
@@ -166,19 +166,6 @@ class EntryService
|
|||||||
return $this->current($tenant);
|
return $this->current($tenant);
|
||||||
}
|
}
|
||||||
|
|
||||||
public function deleteImage(Tenant $tenant): void
|
|
||||||
{
|
|
||||||
$attachment = DB::transaction(function () use ($tenant): Attachment {
|
|
||||||
$entry = $this->entryQuery($tenant)->lockForUpdate()->firstOrFail();
|
|
||||||
$attachment = $entry->allAttachments()->lockForUpdate()->firstOrFail();
|
|
||||||
$entry->allAttachments()->detach($attachment->id);
|
|
||||||
|
|
||||||
return $attachment;
|
|
||||||
});
|
|
||||||
|
|
||||||
$this->deleteIfUnused($attachment);
|
|
||||||
}
|
|
||||||
|
|
||||||
/** @return Collection<string, ItemAttribute> */
|
/** @return Collection<string, ItemAttribute> */
|
||||||
private function itemAttributes(CatalogItem $entry): Collection
|
private function itemAttributes(CatalogItem $entry): Collection
|
||||||
{
|
{
|
||||||
|
|||||||
@@ -14,6 +14,4 @@ Route::prefix('v1/adminapp/tenant/desfile')
|
|||||||
->name('adminapp.desfile.entries.image.replace');
|
->name('adminapp.desfile.entries.image.replace');
|
||||||
Route::patch('entries/image', [EntryController::class, 'updateImage'])
|
Route::patch('entries/image', [EntryController::class, 'updateImage'])
|
||||||
->name('adminapp.desfile.entries.image.update');
|
->name('adminapp.desfile.entries.image.update');
|
||||||
Route::delete('entries/image', [EntryController::class, 'destroyImage'])
|
|
||||||
->name('adminapp.desfile.entries.image.destroy');
|
|
||||||
});
|
});
|
||||||
|
|||||||
Reference in New Issue
Block a user