refactor(traits): remove return type declarations from methods for consistency
This commit is contained in:
parent
b1d5e74fe7
commit
cd7c89b48b
@ -8,12 +8,12 @@ trait WithFacilityHandler
|
|||||||
{
|
{
|
||||||
public OutletForm $form;
|
public OutletForm $form;
|
||||||
|
|
||||||
public function addFacility(): void
|
public function addFacility()
|
||||||
{
|
{
|
||||||
$this->form->facilities[] = '';
|
$this->form->facilities[] = '';
|
||||||
}
|
}
|
||||||
|
|
||||||
public function removeFacility(int $index): void
|
public function removeFacility(int $index)
|
||||||
{
|
{
|
||||||
unset($this->form->facilities[$index]);
|
unset($this->form->facilities[$index]);
|
||||||
$this->form->facilities = array_values($this->form->facilities);
|
$this->form->facilities = array_values($this->form->facilities);
|
||||||
|
|||||||
@ -4,7 +4,7 @@
|
|||||||
|
|
||||||
trait WithAuthorization
|
trait WithAuthorization
|
||||||
{
|
{
|
||||||
public function canOrAbort(string $ability, $model = null): void
|
public function canOrAbort(string $ability, $model = null)
|
||||||
{
|
{
|
||||||
if (auth()->user()->cannot($ability, $model)) {
|
if (auth()->user()->cannot($ability, $model)) {
|
||||||
abort(403);
|
abort(403);
|
||||||
|
|||||||
@ -4,7 +4,7 @@
|
|||||||
|
|
||||||
trait WithCommentEnum
|
trait WithCommentEnum
|
||||||
{
|
{
|
||||||
public static function comment(): string
|
public static function comment()
|
||||||
{
|
{
|
||||||
return implode(', ', array_map(fn ($case) => "{$case->value}: {$case->label()}", self::cases()));
|
return implode(', ', array_map(fn ($case) => "{$case->value}: {$case->label()}", self::cases()));
|
||||||
}
|
}
|
||||||
|
|||||||
@ -20,7 +20,7 @@ public function mapMediaCollection(MediaCollection $mediaCollection)
|
|||||||
])->toArray();
|
])->toArray();
|
||||||
}
|
}
|
||||||
|
|
||||||
protected function syncMedia(array $newMedia, $model, string $collectionName): void
|
protected function syncMedia(array $newMedia, $model, string $collectionName)
|
||||||
{
|
{
|
||||||
$existingMediaIds = $model->getMedia($collectionName)->pluck('id')->toArray();
|
$existingMediaIds = $model->getMedia($collectionName)->pluck('id')->toArray();
|
||||||
|
|
||||||
@ -36,7 +36,7 @@ protected function syncMedia(array $newMedia, $model, string $collectionName): v
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
protected function uploadMedia(array $mediaArray, $model, string $collectionName): void
|
protected function uploadMedia(array $mediaArray, $model, string $collectionName)
|
||||||
{
|
{
|
||||||
foreach ($mediaArray as $file) {
|
foreach ($mediaArray as $file) {
|
||||||
if (isset($file['path'])) {
|
if (isset($file['path'])) {
|
||||||
|
|||||||
@ -4,7 +4,7 @@
|
|||||||
|
|
||||||
trait WithValueEnum
|
trait WithValueEnum
|
||||||
{
|
{
|
||||||
public static function values(): array
|
public static function values()
|
||||||
{
|
{
|
||||||
return array_map(fn ($case) => $case->value, self::cases());
|
return array_map(fn ($case) => $case->value, self::cases());
|
||||||
}
|
}
|
||||||
|
|||||||
Loading…
Reference in New Issue
Block a user