Skip to content
This repository was archived by the owner on Sep 22, 2024. It is now read-only.

Commit af6ffbb

Browse files
authored
Merge pull request #1 from telkins/fix-respond-trait
Two unrelated fixes...
2 parents c2f0aaf + 969d0d1 commit af6ffbb

File tree

2 files changed

+6
-6
lines changed

2 files changed

+6
-6
lines changed

src/Concerns/Respondable.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -7,8 +7,8 @@
77

88
trait Respondable
99
{
10-
protected function respond($data): Responsable
10+
protected function respond($data, ?string $resourceClass = null): Responsable
1111
{
12-
return app(Responder::class)->respond($data);
12+
return app(Responder::class)->respond($data, $resourceClass);
1313
}
1414
}

src/Responder.php

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -62,9 +62,9 @@ public function respond($data, ?string $resourceClass = null): Responsable
6262
*/
6363
protected function respondForCollection(Collection $data, ?string $resourceClass)
6464
{
65-
$modelClass = $this->modelResolver->resolve($data, 'collection');
66-
6765
if (is_null($resourceClass)) {
66+
$modelClass = $this->modelResolver->resolve($data, 'collection');
67+
6868
$resourceClass = empty($modelClass)
6969
? config('responder.default_resource', JsonResource::class)
7070
: $this->resourceResolver->resolve($modelClass);
@@ -96,9 +96,9 @@ protected function respondForModel(Model $model, ?string $resourceClass)
9696
*/
9797
protected function respondForPaginator(LengthAwarePaginator $data, ?string $resourceClass)
9898
{
99-
$modelClass = $this->modelResolver->resolve($data, 'paginator');
100-
10199
if (is_null($resourceClass)) {
100+
$modelClass = $this->modelResolver->resolve($data, 'paginator');
101+
102102
$resourceClass = empty($modelClass)
103103
? config('responder.default_resource', JsonResource::class)
104104
: $this->resourceResolver->resolve($modelClass);

0 commit comments

Comments
 (0)