8000 [12.x] Add resource helper functions to Model/Collections by TimKunze96 · Pull Request #55107 · laravel/framework · GitHub
[go: up one dir, main page]
Skip to content

[12.x] Add resource helper functions to Model/Collections #55107

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 18 commits into from
Apr 2, 2025
+361 −3
Merged

[12.x] Add resource helper functions to Model/Collections #55107

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
18 commits
Select commit Hold shift + click to select a range
0909ce1
Add resource helper functions
TimKunze96 Mar 20, 2025
b7d0768
Fix doc-block style
TimKunze96 Mar 20, 2025
469709a
Remove empty line
TimKunze96 Mar 20, 2025
a4f0faf
Make resource param optional and guess resource name
TimKunze96 Mar 23, 2025
491522e
Use throw_unless instead of assert
TimKunze96 Mar 24, 2025
c060601
Extract name guessing function
TimKunze96 Mar 24, 2025
7ef6563
Add tests for resource helpers
TimKunze96 Mar 24, 2025
ada10bd
Fix formatting
TimKunze96 Mar 24, 2025
48c0865
Fix namespace conflicts and add current page
TimKunze96 Mar 24, 2025
93f3df6
Use more descriptive LogicException instead of Exception
TimKunze96 Mar 24, 2025
5f7092d
Refactor to use traits and extend base collection instead of eloquent…
TimKunze96 Mar 26, 2025
fbf4f78
Ensure trait method exists
TimKunze96 Mar 26, 2025
6e878f4
formatting
taylorotwell Apr 1, 2025
28c1d3f
formatting
taylorotwell Apr 1, 2025
c738f7f
formatting
taylorotwell Apr 1, 2025
9e9f8a6
adjust tests and logic
taylorotwell Apr 2, 2025
bf7676b
add testS
taylorotwell Apr 2, 2025
bbe8ffb
remove comment
taylorotwell Apr 2, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
formatting
  • Loading branch information
@taylorotwell
taylorotwell committed Apr 1, 2025
commit c738f7f26f38a68c0423f72ff208de22f7c43d31
11 changes: 11 additions & 0 deletions tests/Pagination/Fixtures/Models/PaginatorResourceTestModel.php
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,11 @@
<?php

namespace Illuminate\Tests\Pagination\Fixtures\Models;

use Illuminate\Database\Eloquent\Concerns\HasUlids;
use Illuminate\Database\Eloquent\Model;

class PaginatorResourceTestModel extends Model
{
//
}
10 changes: 3 additions & 7 deletions tests/Pagination/PaginatorResourceTest.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,7 @@
use Illuminate\Database\Eloquent\Model;
use Illuminate\Http\Resources\Json\JsonResource;
use Illuminate\Pagination\LengthAwarePaginator;
use Illuminate\Tests\Pagination\Fixtures\Models\PaginatorResourceTestModel;
use PHPUnit\Framework\TestCase;

class PaginatorResourceTest extends TestCase
Expand All @@ -23,7 +24,7 @@ public function testItCanTransformToExplicitResource()
public function testItThrowsExceptionWhenResourceCannotBeFound()
{
$this->expectException(\LogicException::class);
$this->expectExceptionMessage('Failed to find resource class for model [Illuminate\Tests\Pagination\PaginatorResourceTestModel].');
$this->expectExceptionMessage('Failed to find resource class for model [Illuminate\Tests\Pagination\Fixtures\Models\PaginatorResourceTestModel].');

$paginator = new PaginatorResourceTestPaginator([
new PaginatorResourceTestModel(),
Expand All @@ -38,19 +39,14 @@ public function testItCanGuessResourceWhenNotProvided()
new PaginatorResourceTestModel(),
], 1, 1, 1);

class_alias(PaginatorResourceTestResource::class, 'App\Http\Resources\PaginatorResourceTestModelResource');
class_alias(PaginatorResourceTestResource::class, 'Illuminate\Tests\Pagination\Fixtures\Http\Resources\PaginatorResourceTestModelResource');

$resource = $paginator->toResourceCollection();

$this->assertInstanceOf(JsonResource::class, $resource);
}
}

class PaginatorResourceTestModel extends Model
{
//
}

class PaginatorResourceTestResource extends JsonResource
{
//
Expand Down
Loading
0