From 69f5dd932380473b42b12bd4de722bbe105b0d4d Mon Sep 17 00:00:00 2001 From: Romulus21 Date: Wed, 9 Sep 2020 12:42:20 +0200 Subject: [PATCH] refact App\User to App\Models\User --- .../Controllers/Auth/RegisterController.php | 4 +-- app/Http/Controllers/ImageController.php | 2 +- app/Http/Controllers/UserController.php | 2 +- app/Jobs/SendMailNewUserJob.php | 2 +- app/Mail/NewUserInvitation.php | 2 +- app/Models/Memo.php | 2 +- app/Models/ToDoList.php | 2 +- app/Models/User.php | 2 +- app/Policies/BookmarkPolicy.php | 16 ++++++------ app/Policies/MemoPolicy.php | 16 ++++++------ app/Policies/ToDoListPolicy.php | 16 ++++++------ app/Policies/UserPolicy.php | 26 +++++++++---------- app/Providers/AuthServiceProvider.php | 2 +- config/auth.php | 2 +- database/factories/BookmarkFactory.php | 2 +- database/factories/MemoFactory.php | 2 +- database/factories/ToDoListFactory.php | 2 +- database/factories/UserFactory.php | 2 +- tests/Feature/BookmarkTest.php | 6 ++--- tests/Feature/EventsTest.php | 8 +++--- tests/Feature/ImagesTest.php | 2 +- tests/Feature/MemosTest.php | 8 +++--- tests/Feature/ToDoItemsTest.php | 2 +- tests/Feature/ToDoListsTest.php | 4 +-- tests/Feature/UserAuthTest.php | 2 +- 25 files changed, 68 insertions(+), 68 deletions(-) diff --git a/app/Http/Controllers/Auth/RegisterController.php b/app/Http/Controllers/Auth/RegisterController.php index c6a6de6..ed1a5e0 100644 --- a/app/Http/Controllers/Auth/RegisterController.php +++ b/app/Http/Controllers/Auth/RegisterController.php @@ -4,7 +4,7 @@ namespace App\Http\Controllers\Auth; use App\Http\Controllers\Controller; use App\Providers\RouteServiceProvider; -use App\User; +use App\Models\User; use Illuminate\Foundation\Auth\RegistersUsers; use Illuminate\Support\Facades\Hash; use Illuminate\Support\Facades\Validator; @@ -60,7 +60,7 @@ class RegisterController extends Controller * Create a new user instance after a valid registration. * * @param array $data - * @return \App\User + * @return \App\Models\User */ protected function create(array $data) { diff --git a/app/Http/Controllers/ImageController.php b/app/Http/Controllers/ImageController.php index 8db32b5..afb8a1c 100644 --- a/app/Http/Controllers/ImageController.php +++ b/app/Http/Controllers/ImageController.php @@ -4,7 +4,7 @@ namespace App\Http\Controllers; use App\Http\Resources\Image as ImageResource; use App\Models\Memo; -use App\User; +use App\Models\User; use Illuminate\Http\Request; use Illuminate\Support\Facades\File; use Intervention\Image\Facades\Image; diff --git a/app/Http/Controllers/UserController.php b/app/Http/Controllers/UserController.php index efb04e1..995fad1 100644 --- a/app/Http/Controllers/UserController.php +++ b/app/Http/Controllers/UserController.php @@ -3,7 +3,7 @@ namespace App\Http\Controllers; use App\Jobs\SendMailNewUserJob; -use App\User; +use App\Models\User; use App\Http\Resources\User as UserResource; use Illuminate\Support\Facades\Hash; use Illuminate\Support\Str; diff --git a/app/Jobs/SendMailNewUserJob.php b/app/Jobs/SendMailNewUserJob.php index db18734..a69a66f 100644 --- a/app/Jobs/SendMailNewUserJob.php +++ b/app/Jobs/SendMailNewUserJob.php @@ -3,7 +3,7 @@ namespace App\Jobs; use App\Mail\NewUserInvitation; -use App\User; +use App\Models\User; use Illuminate\Bus\Queueable; use Illuminate\Contracts\Queue\ShouldQueue; use Illuminate\Foundation\Bus\Dispatchable; diff --git a/app/Mail/NewUserInvitation.php b/app/Mail/NewUserInvitation.php index 4aa3acc..1b44d19 100644 --- a/app/Mail/NewUserInvitation.php +++ b/app/Mail/NewUserInvitation.php @@ -2,7 +2,7 @@ namespace App\Mail; -use App\User; +use App\Models\User; use Illuminate\Bus\Queueable; use Illuminate\Contracts\Queue\ShouldQueue; use Illuminate\Mail\Mailable; diff --git a/app/Models/Memo.php b/app/Models/Memo.php index 58df691..4ca65a4 100644 --- a/app/Models/Memo.php +++ b/app/Models/Memo.php @@ -2,7 +2,7 @@ namespace App\Models; -use App\User; +use App\Models\User; use Illuminate\Database\Eloquent\Model; use Illuminate\Database\Eloquent\Relations\MorphMany; use Illuminate\Database\Eloquent\Relations\MorphOne; diff --git a/app/Models/ToDoList.php b/app/Models/ToDoList.php index e2cbb44..347d4b1 100644 --- a/app/Models/ToDoList.php +++ b/app/Models/ToDoList.php @@ -2,7 +2,7 @@ namespace App\Models; -use App\User; +use App\Models\User; use Illuminate\Database\Eloquent\Model; use Illuminate\Database\Eloquent\Relations\BelongsTo; use Illuminate\Database\Eloquent\Relations\HasMany; diff --git a/app/Models/User.php b/app/Models/User.php index 9b49667..124b46d 100644 --- a/app/Models/User.php +++ b/app/Models/User.php @@ -1,6 +1,6 @@ 'App\Policies\ModelPolicy', - 'App\User' => 'App\Policies\UserPolicy', + 'App\Models\User' => 'App\Policies\UserPolicy', 'App\Models\Memo' => 'App\Policies\MemoPolicy', 'App\Models\ToDoList' => 'App\Policies\ToDoListPolicy', 'App\Models\Bookmark' => 'App\Policies\BookmarkPolicy', diff --git a/config/auth.php b/config/auth.php index 04c6eec..1192b86 100644 --- a/config/auth.php +++ b/config/auth.php @@ -68,7 +68,7 @@ return [ 'providers' => [ 'users' => [ 'driver' => 'eloquent', - 'model' => App\User::class, + 'model' => App\Models\User::class, ], // 'users' => [ diff --git a/database/factories/BookmarkFactory.php b/database/factories/BookmarkFactory.php index bedc0cd..74ce6e6 100644 --- a/database/factories/BookmarkFactory.php +++ b/database/factories/BookmarkFactory.php @@ -7,7 +7,7 @@ use Faker\Generator as Faker; $factory->define(Bookmark::class, function (Faker $faker) { return [ - 'user_id' => factory(\App\User::class), + 'user_id' => factory(\App\Models\User::class), 'name' => $faker->words(3, [false]), 'url' => $faker->url, 'favicon' => $faker->imageUrl(), diff --git a/database/factories/MemoFactory.php b/database/factories/MemoFactory.php index aefbb57..1cc80d3 100644 --- a/database/factories/MemoFactory.php +++ b/database/factories/MemoFactory.php @@ -7,7 +7,7 @@ use Faker\Generator as Faker; $factory->define(\App\Models\Memo::class, function (Faker $faker) { return [ - 'user_id' => factory(\App\User::class), + 'user_id' => factory(\App\Models\User::class), 'name' => $faker->words(3, [false]), 'memo' => $faker->text($maxNbChars = 200), ]; diff --git a/database/factories/ToDoListFactory.php b/database/factories/ToDoListFactory.php index 6287b75..43c606a 100644 --- a/database/factories/ToDoListFactory.php +++ b/database/factories/ToDoListFactory.php @@ -7,7 +7,7 @@ use Faker\Generator as Faker; $factory->define(ToDoList::class, function (Faker $faker) { return [ - 'user_id' => factory(\App\User::class), + 'user_id' => factory(\App\Models\User::class), 'name' => $faker->words(3, [false]), ]; }); diff --git a/database/factories/UserFactory.php b/database/factories/UserFactory.php index 741edea..fa9360e 100644 --- a/database/factories/UserFactory.php +++ b/database/factories/UserFactory.php @@ -2,7 +2,7 @@ /** @var \Illuminate\Database\Eloquent\Factory $factory */ -use App\User; +use App\Models\User; use Faker\Generator as Faker; use Illuminate\Support\Str; diff --git a/tests/Feature/BookmarkTest.php b/tests/Feature/BookmarkTest.php index 79b0c8f..7918651 100644 --- a/tests/Feature/BookmarkTest.php +++ b/tests/Feature/BookmarkTest.php @@ -3,7 +3,7 @@ namespace Tests\Feature; use App\Models\Bookmark; -use App\User; +use App\Models\User; use Illuminate\Foundation\Testing\RefreshDatabase; use Illuminate\Foundation\Testing\WithFaker; use Tests\TestCase; @@ -17,7 +17,7 @@ class BookmarkTest extends TestCase { $this->withoutExceptionHandling(); - $this->actingAs($user = factory(\App\User::class)->create(), 'api'); + $this->actingAs($user = factory(\App\Models\User::class)->create(), 'api'); $response = $this->post('/api/bookmarks', $this->data()); @@ -49,7 +49,7 @@ class BookmarkTest extends TestCase /** @test */ public function bookmark_url_are_required() { - $this->actingAs($user = factory(\App\User::class)->create(), 'api'); + $this->actingAs($user = factory(\App\Models\User::class)->create(), 'api'); $response = $this->post('/api/bookmarks', array_merge($this->data(), ['url' => ''])); $response->assertSessionHasErrors('url'); diff --git a/tests/Feature/EventsTest.php b/tests/Feature/EventsTest.php index 44663f9..15ee005 100644 --- a/tests/Feature/EventsTest.php +++ b/tests/Feature/EventsTest.php @@ -5,7 +5,7 @@ namespace Tests\Feature; use App\Models\Event; use App\Models\EventCategory; use App\Models\Memo; -use App\User; +use App\Models\User; use Illuminate\Foundation\Testing\RefreshDatabase; use Illuminate\Foundation\Testing\WithFaker; use Symfony\Component\HttpFoundation\Response; @@ -50,7 +50,7 @@ class EventsTest extends TestCase /** @test */ public function event_category_name_are_required() { - $this->actingAs($user = factory(\App\User::class)->create(['role' => 2]), 'api'); + $this->actingAs($user = factory(\App\Models\User::class)->create(['role' => 2]), 'api'); $response = $this->post('/api/events/categories', ['name' => '', 'description' => 'test name required']); $response->assertSessionHasErrors('name'); @@ -257,7 +257,7 @@ class EventsTest extends TestCase /** @test */ public function event_name_are_required() { - $this->actingAs($user = factory(\App\User::class)->create(), 'api'); + $this->actingAs($user = factory(\App\Models\User::class)->create(), 'api'); $response = $this->post('/api/events', array_merge($this->data(), ['name' => ''])); $response->assertSessionHasErrors('name'); @@ -267,7 +267,7 @@ class EventsTest extends TestCase /** @test */ public function event_start_date_are_required() { - $this->actingAs($user = factory(\App\User::class)->create(), 'api'); + $this->actingAs($user = factory(\App\Models\User::class)->create(), 'api'); $response = $this->post('/api/events', array_merge($this->data(), ['start_date' => ''])); $response->assertSessionHasErrors('start_date'); diff --git a/tests/Feature/ImagesTest.php b/tests/Feature/ImagesTest.php index d87df54..f9ed282 100644 --- a/tests/Feature/ImagesTest.php +++ b/tests/Feature/ImagesTest.php @@ -4,7 +4,7 @@ namespace Tests\Feature; use App\Models\Image; use App\Models\Memo; -use App\User; +use App\Models\User; use Illuminate\Foundation\Testing\RefreshDatabase; use Illuminate\Foundation\Testing\WithFaker; use Illuminate\Http\UploadedFile; diff --git a/tests/Feature/MemosTest.php b/tests/Feature/MemosTest.php index a2334e0..7117c7a 100644 --- a/tests/Feature/MemosTest.php +++ b/tests/Feature/MemosTest.php @@ -3,7 +3,7 @@ namespace Tests\Feature; use App\Models\Memo; -use App\User; +use App\Models\User; use Illuminate\Foundation\Testing\RefreshDatabase; use Illuminate\Foundation\Testing\WithFaker; use Symfony\Component\HttpFoundation\Response; @@ -28,7 +28,7 @@ class MemosTest extends TestCase { $this->withoutExceptionHandling(); - $this->actingAs($user = factory(\App\User::class)->create(), 'api'); + $this->actingAs($user = factory(\App\Models\User::class)->create(), 'api'); $response = $this->post('/api/memos', $this->data()); @@ -52,7 +52,7 @@ class MemosTest extends TestCase /** @test */ public function memo_name_are_required() { - $this->actingAs($user = factory(\App\User::class)->create(), 'api'); + $this->actingAs($user = factory(\App\Models\User::class)->create(), 'api'); $response = $this->post('/api/memos', array_merge($this->data(), ['name' => ''])); $response->assertSessionHasErrors('name'); @@ -62,7 +62,7 @@ class MemosTest extends TestCase /** @test */ public function memo_are_required() { - $this->actingAs($user = factory(\App\User::class)->create(), 'api'); + $this->actingAs($user = factory(\App\Models\User::class)->create(), 'api'); $response = $this->post('/api/memos', array_merge($this->data(), ['memo' => ''])); $response->assertSessionHasErrors('memo'); diff --git a/tests/Feature/ToDoItemsTest.php b/tests/Feature/ToDoItemsTest.php index f114a3e..ae402c0 100644 --- a/tests/Feature/ToDoItemsTest.php +++ b/tests/Feature/ToDoItemsTest.php @@ -4,7 +4,7 @@ namespace Tests\Feature; use App\Models\ToDo; use App\Models\ToDoList; -use App\User; +use App\Models\User; use Illuminate\Foundation\Testing\RefreshDatabase; use Illuminate\Foundation\Testing\WithFaker; use Tests\TestCase; diff --git a/tests/Feature/ToDoListsTest.php b/tests/Feature/ToDoListsTest.php index 7c21539..0ba1010 100644 --- a/tests/Feature/ToDoListsTest.php +++ b/tests/Feature/ToDoListsTest.php @@ -3,7 +3,7 @@ namespace Tests\Feature; use App\Models\ToDoList; -use App\User; +use App\Models\User; use Illuminate\Foundation\Testing\RefreshDatabase; use Illuminate\Foundation\Testing\WithFaker; use Tests\TestCase; @@ -44,7 +44,7 @@ class ToDoListsTest extends TestCase /** @test */ public function to_do_list_name_are_required() { - $this->actingAs($user = factory(\App\User::class)->create(), 'api'); + $this->actingAs($user = factory(\App\Models\User::class)->create(), 'api'); $response = $this->post('/api/to-do-lists/', ['name' => '']); $response->assertSessionHasErrors('name'); diff --git a/tests/Feature/UserAuthTest.php b/tests/Feature/UserAuthTest.php index b346ea8..3fa7cdd 100644 --- a/tests/Feature/UserAuthTest.php +++ b/tests/Feature/UserAuthTest.php @@ -2,7 +2,7 @@ namespace Tests\Feature; -use App\User; +use App\Models\User; use Illuminate\Foundation\Testing\RefreshDatabase; use Illuminate\Foundation\Testing\WithFaker; use Symfony\Component\HttpFoundation\Response;