8000 [Refactor] Update exception thrown by destroy method · laravel-json-api/laravel@3aa779e · GitHub
[go: up one dir, main page]

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Commit 3aa779e

Browse files
committed
[Refactor] Update exception thrown by destroy method
1 parent 89a5ad9 commit 3aa779e

File tree

1 file changed

+3
-6
lines changed

1 file changed

+3
-6
lines changed

src/Http/Controllers/Actions/Destroy.php

Lines changed: 3 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -26,6 +26,7 @@
2626
use Illuminate\Support\Facades\Auth;
2727
use LaravelJsonApi\Contracts\Routing\Route;
2828
use LaravelJsonApi\Contracts\Store\Store as StoreContract;
29+
use LaravelJsonApi\Laravel\Exceptions\HttpNotAcceptableException;
2930
use LaravelJsonApi\Laravel\Http\Requests\ResourceRequest;
3031

3132
trait Destroy
@@ -37,7 +38,7 @@ trait Destroy
3738
* @param Route $route
3839
* @param StoreContract $store
3940
* @return Response|Responsable
40-
* @throws AuthenticationException|AuthorizationException
41+
* @throws AuthenticationException|AuthorizationException|HttpNotAcceptableException
4142
*/
4243
public function destroy(Route $route, StoreContract $store)
4344
{
@@ -55,11 +56,7 @@ public function destroy(Route $route, StoreContract $store)
5556
}
5657
}
5758

58-
abort_unless(
59-
$acceptable,
60-
406,
61-
__("The requested resource is capable of generating only content not acceptable according to the Accept headers sent in the request."),
62-
);
59+
throw_unless($acceptable, new HttpNotAcceptableException());
6360

6461
$request = ResourceRequest::forResourceIfExists(
6562
$resourceType = $route->resourceType()

0 commit comments

Comments
 (0)
0