From 64000344358f72fb5ec79edf6693b6880e99bf78 Mon Sep 17 00:00:00 2001 From: Marcus Moore Date: Wed, 2 Aug 2023 16:06:59 -0700 Subject: [PATCH] Improve variable name --- app/Http/Livewire/CategoryEditForm.php | 8 +++--- .../livewire/category-edit-form.blade.php | 2 +- .../Feature/Livewire/CategoryEditFormTest.php | 28 +++++++++---------- 3 files changed, 19 insertions(+), 19 deletions(-) diff --git a/app/Http/Livewire/CategoryEditForm.php b/app/Http/Livewire/CategoryEditForm.php index 2cba9a220d..d69c09971a 100644 --- a/app/Http/Livewire/CategoryEditForm.php +++ b/app/Http/Livewire/CategoryEditForm.php @@ -6,20 +6,20 @@ use Livewire\Component; class CategoryEditForm extends Component { - public $checkinEmail; - public $defaultEulaText; public $eulaText; public $requireAcceptance; + public $sendCheckInEmail; + public $useDefaultEula; public function mount() { if ($this->eulaText || $this->useDefaultEula) { - $this->checkinEmail = true; + $this->sendCheckInEmail = true; } } @@ -31,7 +31,7 @@ class CategoryEditForm extends Component public function updated($property, $value) { if (in_array($property, ['eulaText', 'useDefaultEula']) && ($this->eulaText || $this->useDefaultEula)) { - $this->checkinEmail = (bool)$value; + $this->sendCheckInEmail = (bool)$value; } } diff --git a/resources/views/livewire/category-edit-form.blade.php b/resources/views/livewire/category-edit-form.blade.php index fb54c125e2..e49d9cc0fa 100644 --- a/resources/views/livewire/category-edit-form.blade.php +++ b/resources/views/livewire/category-edit-form.blade.php @@ -41,7 +41,7 @@
@if ($this->shouldDisplayEmailMessage) diff --git a/tests/Feature/Livewire/CategoryEditFormTest.php b/tests/Feature/Livewire/CategoryEditFormTest.php index 753a74bca0..448e61cee6 100644 --- a/tests/Feature/Livewire/CategoryEditFormTest.php +++ b/tests/Feature/Livewire/CategoryEditFormTest.php @@ -16,56 +16,56 @@ class CategoryEditFormTest extends TestCase public function testSendEmailCheckboxIsCheckedOnLoadWhenSendEmailIsExistingSetting() { Livewire::test(CategoryEditForm::class, [ - 'checkinEmail' => true, + 'sendCheckInEmail' => true, 'eulaText' => '', 'useDefaultEula' => false, - ])->assertSet('checkinEmail', true); + ])->assertSet('sendCheckInEmail', true); } public function testSendEmailCheckboxIsCheckedOnLoadWhenCategoryEulaSet() { Livewire::test(CategoryEditForm::class, [ - 'checkinEmail' => false, + 'sendCheckInEmail' => false, 'eulaText' => 'Some Content', 'useDefaultEula' => false, - ])->assertSet('checkinEmail', true); + ])->assertSet('sendCheckInEmail', true); } public function testSendEmailCheckboxIsCheckedOnLoadWhenUsingDefaultEula() { Livewire::test(CategoryEditForm::class, [ - 'checkinEmail' => false, + 'sendCheckInEmail' => false, 'eulaText' => '', 'useDefaultEula' => true, - ])->assertSet('checkinEmail', true); + ])->assertSet('sendCheckInEmail', true); } public function testSendEmailCheckBoxIsUncheckedOnLoadWhenSendEmailIsFalseNoCategoryEulaSetAndNotUsingDefaultEula() { Livewire::test(CategoryEditForm::class, [ - 'checkinEmail' => false, + 'sendCheckInEmail' => false, 'eulaText' => '', 'useDefaultEula' => false, - ])->assertSet('checkinEmail', false); + ])->assertSet('sendCheckInEmail', false); } public function testSendEmailCheckboxIsCheckedWhenCategoryEulaEntered() { Livewire::test(CategoryEditForm::class, [ - 'checkinEmail' => false, + 'sendCheckInEmail' => false, 'useDefaultEula' => false, - ])->assertSet('checkinEmail', false) + ])->assertSet('sendCheckInEmail', false) ->set('eulaText', 'Some Content') - ->assertSet('checkinEmail', true); + ->assertSet('sendCheckInEmail', true); } public function testSendEmailCheckboxCheckedWhenUseDefaultEulaSelected() { Livewire::test(CategoryEditForm::class, [ - 'checkinEmail' => false, + 'sendCheckInEmail' => false, 'useDefaultEula' => false, - ])->assertSet('checkinEmail', false) + ])->assertSet('sendCheckInEmail', false) ->set('useDefaultEula', true) - ->assertSet('checkinEmail', true); + ->assertSet('sendCheckInEmail', true); } }