add bootstrap validation and Logging for Entrance Examination

2024-03-13 10:26:59 +03:00 · 2024-03-13 10:26:59 +03:00 · 4e2d8bec4b
parent 2104362d4e
commit 4e2d8bec4b
7 changed files with 187 additions and 57 deletions
--- a/app/Helpers/PositionHelper.php
+++ b/app/Helpers/PositionHelper.php
@ -4,6 +4,7 @@ namespace App\Helpers;

 use App\Models\Direction;
 use App\Models\DirectionProfile;
+use App\Models\EntranceExamination;
 use App\Models\ExaminationType;
 use App\Models\Faculty;
 use App\Models\EducationalInstitution;
@ -53,4 +54,10 @@ class PositionHelper
        $maxPosition = ExaminationType::max('position');
        return $maxPosition >= 254 ? 255 : $maxPosition + 1;
    }
+
+    public static function entranceExamination()
+    {
+        $maxPosition = EntranceExamination::max('position');
+        return $maxPosition >= 254 ? 255 : $maxPosition + 1;
+    }
 }
--- a/app/Http/Controllers/admin/Catalog/Direction/EntranceExaminationController.php
+++ b/app/Http/Controllers/admin/Catalog/Direction/EntranceExaminationController.php
@ -12,6 +12,8 @@ use App\Models\Subject;
 use App\Models\SubjectType;
 use Illuminate\Contracts\View\View;
 use Illuminate\Http\RedirectResponse;
+use Illuminate\Support\Facades\Auth;
+use Illuminate\Support\Facades\Log;

 class EntranceExaminationController extends Controller
 {
@ -51,6 +53,12 @@ class EntranceExaminationController extends Controller
        $entranceExamination->subject_type_id = $validated['subject_type_id'];
        $entranceExamination->save();

+        Log::channel('app')
+            ->info(
+                'CREATE вступ. испытания {entranceExamination} - user {user}',
+                ['user' => Auth::user()->name, 'entranceExamination' => $entranceExamination->name, 'data' => $validated]
+            );
+
        return redirect()->route('entrance_examinations.index');
    }

@ -85,6 +93,7 @@ class EntranceExaminationController extends Controller
        EntranceExamination $entranceExamination
    ): RedirectResponse {
        $validated = $request->validated();
+        $oldData = $entranceExamination->toArray();

        $entranceExamination->examination_type_id = $validated['examination_type_id'];
        $entranceExamination->direction_id = $validated['direction_id'];
@ -92,6 +101,17 @@ class EntranceExaminationController extends Controller
        $entranceExamination->scores = $validated['scores'];
        $entranceExamination->position = $validated['position'];
        $entranceExamination->subject_type_id = $validated['subject_type_id'];
+
+        Log::channel('app')
+            ->warning(
+                'UPDATE вступ. испытания {entranceExamination} - user {user}',
+                [
+                    'user' => Auth::user()->name,
+                    'entranceExamination' => $entranceExamination->name,
+                    'oldData' => $oldData,
+                    'newData' => $validated
+                ]
+            );
        $entranceExamination->save();

        return redirect()->route('entrance_examinations.index');
@ -99,6 +119,15 @@ class EntranceExaminationController extends Controller

    public function destroy(EntranceExamination $entranceExamination): RedirectResponse
    {
+        Log::channel('app')
+            ->critical(
+                'DELETE вступ. испытания {entranceExamination} - user {user}',
+                [
+                    'user' => Auth::user()->name,
+                    'entranceExamination' => $entranceExamination->name,
+                    'data' => $entranceExamination->toArray(),
+                ]
+            );
        $entranceExamination->delete();
        return redirect()->route('entrance_examinations.index');
    }
--- a/app/Http/Requests/admin/Catalog/Direction/StoreEntranceExaminationRequest.php
+++ b/app/Http/Requests/admin/Catalog/Direction/StoreEntranceExaminationRequest.php
@ -18,8 +18,38 @@ class StoreEntranceExaminationRequest extends FormRequest
            'examination_type_id' => 'required|numeric|int|max:1000',
            'subject_id' => 'required|numeric|int|max:1000',
            'subject_type_id' => 'required|numeric|int|max:1000',
-            'scores' => 'required|numeric|int|max:1000',
+            'scores' => 'required|numeric|int|max:100',
            'position' => 'required|numeric|int|max:1000',
        ];
    }
+
+    public function messages(): array
+    {
+        return [
+            'position.required' => 'Поле позиция обязательно.',
+            'position.int' => 'Позиция должно быть целым числом.',
+            'position.numeric' => 'Позиция должно быть числом.',
+            'position.max' => 'Позиция не должен быть больше :max',
+            'direction_id.required' => 'Поле позиция обязательно.',
+            'direction_id.int' => 'Позиция должно быть целым числом.',
+            'direction_id.numeric' => 'Позиция должно быть числом.',
+            'direction_id.max' => 'Позиция не должен быть больше :max',
+            'examination_type_id.required' => 'Поле позиция обязательно.',
+            'examination_type_id.int' => 'Позиция должно быть целым числом.',
+            'examination_type_id.numeric' => 'Позиция должно быть числом.',
+            'examination_type_id.max' => 'Позиция не должен быть больше :max',
+            'subject_id.required' => 'Поле позиция обязательно.',
+            'subject_id.int' => 'Позиция должно быть целым числом.',
+            'subject_id.numeric' => 'Позиция должно быть числом.',
+            'subject_id.max' => 'Позиция не должен быть больше :max',
+            'subject_type_id.required' => 'Поле позиция обязательно.',
+            'subject_type_id.int' => 'Позиция должно быть целым числом.',
+            'subject_type_id.numeric' => 'Позиция должно быть числом.',
+            'subject_type_id.max' => 'Позиция не должен быть больше :max',
+            'scores.required' => 'Поле позиция обязательно.',
+            'scores.int' => 'Позиция должно быть целым числом.',
+            'scores.numeric' => 'Позиция должно быть числом.',
+            'scores.max' => 'Позиция не должен быть больше :max',
+        ];
+    }
 }
--- a/app/Http/Requests/admin/Catalog/Direction/UpdateEntranceExaminationRequest.php
+++ b/app/Http/Requests/admin/Catalog/Direction/UpdateEntranceExaminationRequest.php
@ -18,8 +18,38 @@ class UpdateEntranceExaminationRequest extends FormRequest
            'examination_type_id' => 'required|numeric|int|max:1000',
            'subject_id' => 'required|numeric|int|max:1000',
            'subject_type_id' => 'required|numeric|int|max:1000',
-            'scores' => 'required|numeric|int|max:1000',
+            'scores' => 'required|numeric|int|max:100',
            'position' => 'required|numeric|int|max:1000',
        ];
    }
+
+    public function messages(): array
+    {
+        return [
+            'position.required' => 'Поле позиция обязательно.',
+            'position.int' => 'Позиция должно быть целым числом.',
+            'position.numeric' => 'Позиция должно быть числом.',
+            'position.max' => 'Позиция не должен быть больше :max',
+            'direction_id.required' => 'Поле позиция обязательно.',
+            'direction_id.int' => 'Позиция должно быть целым числом.',
+            'direction_id.numeric' => 'Позиция должно быть числом.',
+            'direction_id.max' => 'Позиция не должен быть больше :max',
+            'examination_type_id.required' => 'Поле позиция обязательно.',
+            'examination_type_id.int' => 'Позиция должно быть целым числом.',
+            'examination_type_id.numeric' => 'Позиция должно быть числом.',
+            'examination_type_id.max' => 'Позиция не должен быть больше :max',
+            'subject_id.required' => 'Поле позиция обязательно.',
+            'subject_id.int' => 'Позиция должно быть целым числом.',
+            'subject_id.numeric' => 'Позиция должно быть числом.',
+            'subject_id.max' => 'Позиция не должен быть больше :max',
+            'subject_type_id.required' => 'Поле позиция обязательно.',
+            'subject_type_id.int' => 'Позиция должно быть целым числом.',
+            'subject_type_id.numeric' => 'Позиция должно быть числом.',
+            'subject_type_id.max' => 'Позиция не должен быть больше :max',
+            'scores.required' => 'Поле позиция обязательно.',
+            'scores.int' => 'Позиция должно быть целым числом.',
+            'scores.numeric' => 'Позиция должно быть числом.',
+            'scores.max' => 'Позиция не должен быть больше :max',
+        ];
+    }
 }
--- a/lang/ru/tooltips.php
+++ b/lang/ru/tooltips.php
@ -33,6 +33,7 @@ return [
        'subject_type_id' => 'Поле "Тип Предмета" указывает на привязку вступительного испытания к типу предмету',
        'scores' => 'Поле "Кол-во баллов" указывает на привязку вступительного испытания к Кол-ву баллов',
        'position' => 'Поле "Позиция" нужно для упорядочивания отображения на сайте',
+        'direction_id' => 'Поле "Направление подготовки" указывает на привязку вступительного испытания к направлению подготовки',
    ],
    'educational_institutions' => [
        'position' => 'Поле "Позиция" нужно для упорядочивания отображения на сайте',
--- a/resources/views/admin/catalog/direction/entrance_examination/create.blade.php
+++ b/resources/views/admin/catalog/direction/entrance_examination/create.blade.php
@ -1,54 +1,88 @@
+@php use App\Helpers\PositionHelper; @endphp
@extends('layouts.admin_layout')
@section('content')
    @auth()
        <div class="row">
            <div class="col">
                <h1 class=""> Создать вступительный экзамен</h1>
-                {{ Form::open(['url' => route('entrance_examinations.store'), 'method' => 'POST', 'class' => '']) }}
+                {{ Form::open(['url' => route('entrance_examinations.store'), 'method' => 'POST', 'class' => 'needs-validation', 'novalidate']) }}
                <div class="col">
+
                    <div class="mt-3">
-                        {{ Form::label('direction_id', 'Направление подготовки') }}
+                        {{ Form::label('position', 'Позиция', ['data-bs-toggle' => "tooltip", 'data-bs-title' => __('tooltips.entrance-examination.direction_id')]) }}
+                        <span class="text-danger">*</span>
                    </div>
                    <div class="mt-1">
-                        {{ Form::select('direction_id', $directions, null, ['class' => 'form-select']) }}
+                        {{ Form::text('position', PositionHelper::entranceExamination(), ['class' => 'form-control', 'data-bs-toggle' => "tooltip", 'data-bs-title' => __('tooltips.entrance-examination.direction_id'), 'required']) }}
+                        <div class="invalid-feedback">
+                            Поле "Позиция" обязательно!
+                        </div>
                    </div>
-                    <div>
+                    <div class="text-danger">
+                        @if ($errors->any())
+                            {{ $errors->first('position') }}
+                        @endif
+                    </div>
+
+                    <div class="mt-3">
+                        {{ Form::label('direction_id', 'Направление подготовки', ['data-bs-toggle' => "tooltip", 'data-bs-title' => __('tooltips.entrance-examination.direction_id')]) }}
+                        <span class="text-danger">*</span>
+                    </div>
+                    <div class="mt-1">
+                        {{ Form::select('direction_id', $directions, null, ['class' => 'form-select', 'data-bs-toggle' => "tooltip", 'data-bs-title' => __('tooltips.entrance-examination.direction_id'), 'required']) }}
+                        <div class="invalid-feedback">
+                            Поле "Направление подготовки" обязательно!
+                        </div>
+                    </div>
+                    <div class="text-danger">
                        @if ($errors->any())
                            {{ $errors->first('direction_id') }}
                        @endif
                    </div>

                    <div class="mt-3">
-                        {{ Form::label('examination_type_id', 'Тип экзамена') }}
+                        {{ Form::label('examination_type_id', 'Тип экзамена', ['data-bs-toggle' => "tooltip", 'data-bs-title' => __('tooltips.entrance-examination.examination_type_id')]) }}
+                        <span class="text-danger">*</span>
                    </div>
                    <div class="mt-1">
-                        {{ Form::select('examination_type_id', $examination_types, null, ['class' => 'form-select']) }}
+                        {{ Form::select('examination_type_id', $examination_types, null, ['class' => 'form-select', 'data-bs-toggle' => "tooltip", 'data-bs-title' => __('tooltips.entrance-examination.examination_type_id'), 'required']) }}
+                        <div class="invalid-feedback">
+                            Поле "Тип экзамена" обязательно!
+                        </div>
                    </div>
-                    <div>
+                    <div class="text-danger">
                        @if ($errors->any())
                            {{ $errors->first('examination_type_id') }}
                        @endif
                    </div>

                    <div class="mt-3">
-                        {{ Form::label('subject_id', 'Предмет') }}
+                        {{ Form::label('subject_id', 'Предмет', ['data-bs-toggle' => "tooltip", 'data-bs-title' => __('tooltips.entrance-examination.subject_id')]) }}
+                        <span class="text-danger">*</span>
                    </div>
                    <div class="mt-1">
-                        {{ Form::select('subject_id', $subjects, null, ['class' => 'form-select']) }}
+                        {{ Form::select('subject_id', $subjects, null, ['class' => 'form-select', 'data-bs-toggle' => "tooltip", 'data-bs-title' => __('tooltips.entrance-examination.subject_id'), 'required']) }}
+                        <div class="invalid-feedback">
+                            Поле "Предмет" обязательно!
+                        </div>
                    </div>
-                    <div>
+                    <div class="text-danger">
                        @if ($errors->any())
                            {{ $errors->first('subject_id') }}
                        @endif
                    </div>

                    <div class="mt-3">
-                        {{ Form::label('subject_type_id', 'Тип предмета') }}
+                        {{ Form::label('subject_type_id', 'Тип предмета', ['data-bs-toggle' => "tooltip", 'data-bs-title' => __('tooltips.entrance-examination.subject_type_id')]) }}
+                        <span class="text-danger">*</span>
                    </div>
                    <div class="mt-1">
-                        {{ Form::select('subject_type_id', $subjectTypes, null, ['class' => 'form-select']) }}
+                        {{ Form::select('subject_type_id', $subjectTypes, null, ['class' => 'form-select', 'data-bs-toggle' => "tooltip", 'data-bs-title' => __('tooltips.entrance-examination.subject_id'), 'required']) }}
+                        <div class="invalid-feedback">
+                            Поле "Тип предмета" обязательно!
+                        </div>
                    </div>
-                    <div>
+                    <div class="text-danger">
                        @if ($errors->any())
                            {{ $errors->first('subject_type_id') }}
                        @endif
@ -56,29 +90,21 @@


                    <div class="mt-3">
-                        {{ Form::label('scores', 'Кол-во баллов') }}
+                        {{ Form::label('scores', 'Кол-во баллов', ['data-bs-toggle' => "tooltip", 'data-bs-title' => __('tooltips.entrance-examination.subject_type_id')]) }}
+                        <span class="text-danger">*</span>
                    </div>
                    <div class="mt-1">
-                        {{ Form::text('scores', '', ['class' => 'form-control']) }}
+                        {{ Form::number('scores', '', ['class' => 'form-control', 'data-bs-toggle' => "tooltip", 'data-bs-title' => __('tooltips.entrance-examination.subject_id'), 'required']) }}
+                        <div class="invalid-feedback">
+                            Поле "Кол-во баллов" обязательно!
+                        </div>
                    </div>
-                    <div>
+                    <div class="text-danger">
                        @if ($errors->any())
                            {{ $errors->first('scores') }}
                        @endif
                    </div>

-                    <div class="mt-3">
-                        {{ Form::label('position', 'Позиция') }}
-                    </div>
-                    <div class="mt-1">
-                        {{ Form::text('position', '', ['class' => 'form-control']) }}
-                    </div>
-                    <div>
-                        @if ($errors->any())
-                            {{ $errors->first('position') }}
-                        @endif
-                    </div>
-
                    <div class="mt-3">
                        {{ Form::submit('Создать', ['class' => 'btn btn-primary']) }}
                    </div>
--- a/resources/views/admin/catalog/direction/entrance_examination/edit.blade.php
+++ b/resources/views/admin/catalog/direction/entrance_examination/edit.blade.php
@ -6,49 +6,67 @@
                <h1 class="">Изменить вступительный экзамен</h1>
                {{ Form::open(['url' => route('entrance_examinations.update', $entranceExamination), 'method' => 'PATCH', 'class' => '']) }}
                <div class="col">
+
                    <div class="mt-3">
-                        {{ Form::label('direction_id', 'Направление подготовки') }}
+                        {{ Form::label('position', 'Позиция', ['data-bs-toggle' => "tooltip", 'data-bs-title' => __('tooltips.entrance-examination.position')]) }}
+                        <span class="text-danger">*</span>
                    </div>
                    <div class="mt-1">
-                        {{ Form::select('direction_id', $directions, $entranceExamination->direction->id, ['class' => 'form-select']) }}
+                        {{ Form::text('position', $entranceExamination->position, ['class' => 'form-control', 'data-bs-toggle' => "tooltip", 'data-bs-title' => __('tooltips.entrance-examination.position'), 'required']) }}
                    </div>
-                    <div>
+                    <div class="text-danger">
+                        @if ($errors->any())
+                            {{ $errors->first('position') }}
+                        @endif
+                    </div>
+
+                    <div class="mt-3">
+                        {{ Form::label('direction_id', 'Направление подготовки', ['data-bs-toggle' => "tooltip", 'data-bs-title' => __('tooltips.entrance-examination.direction_id')]) }}
+                        <span class="text-danger">*</span>
+                    </div>
+                    <div class="mt-1">
+                        {{ Form::select('direction_id', $directions, $entranceExamination->direction->id, ['class' => 'form-select', 'data-bs-toggle' => "tooltip", 'data-bs-title' => __('tooltips.entrance-examination.direction_id'), 'required']) }}
+                    </div>
+                    <div class="text-danger">
                        @if ($errors->any())
                            {{ $errors->first('direction_id') }}
                        @endif
                    </div>

                    <div class="mt-3">
-                        {{ Form::label('examination_type_id', 'Тип экзамена') }}
+                        {{ Form::label('examination_type_id', 'Тип экзамена', ['data-bs-toggle' => "tooltip", 'data-bs-title' => __('tooltips.entrance-examination.examination_type_id')]) }}
+                        <span class="text-danger">*</span>
                    </div>
                    <div class="mt-1">
-                        {{ Form::select('examination_type_id', $examination_types, $entranceExamination->examinationType->id, ['class' => 'form-select']) }}
+                        {{ Form::select('examination_type_id', $examination_types, $entranceExamination->examinationType->id, ['class' => 'form-select', 'data-bs-toggle' => "tooltip", 'data-bs-title' => __('tooltips.entrance-examination.examination_type_id'), 'required']) }}
                    </div>
-                    <div>
+                    <div class="text-danger">
                        @if ($errors->any())
                            {{ $errors->first('examination_type_id') }}
                        @endif
                    </div>

                    <div class="mt-3">
-                        {{ Form::label('subject_id', 'Предмет') }}
+                        {{ Form::label('subject_id', 'Предмет', ['data-bs-toggle' => "tooltip", 'data-bs-title' => __('tooltips.entrance-examination.subject_id')]) }}
+                        <span class="text-danger">*</span>
                    </div>
                    <div class="mt-1">
-                        {{ Form::select('subject_id', $subjects, $entranceExamination->subject->id, ['class' => 'form-select']) }}
+                        {{ Form::select('subject_id', $subjects, $entranceExamination->subject->id, ['class' => 'form-select', 'data-bs-toggle' => "tooltip", 'data-bs-title' => __('tooltips.entrance-examination.subject_id'), 'required']) }}
                    </div>
-                    <div>
+                    <div class="text-danger">
                        @if ($errors->any())
                            {{ $errors->first('subject_id') }}
                        @endif
                    </div>

                    <div class="mt-3">
-                        {{ Form::label('subject_type_id', 'Тип предмета') }}
+                        {{ Form::label('subject_type_id', 'Тип предмета', ['data-bs-toggle' => "tooltip", 'data-bs-title' => __('tooltips.entrance-examination.subject_type_id')]) }}
+                        <span class="text-danger">*</span>
                    </div>
                    <div class="mt-1">
-                        {{ Form::select('subject_type_id', $subjectTypes, $entranceExamination->subjectType->id, ['class' => 'form-select']) }}
+                        {{ Form::select('subject_type_id', $subjectTypes, $entranceExamination->subjectType->id, ['class' => 'form-select', 'data-bs-toggle' => "tooltip", 'data-bs-title' => __('tooltips.entrance-examination.subject_type_id'), 'required']) }}
                    </div>
-                    <div>
+                    <div class="text-danger">
                        @if ($errors->any())
                            {{ $errors->first('subject_type_id') }}
                        @endif
@ -56,29 +74,18 @@


                    <div class="mt-3">
-                        {{ Form::label('scores', 'Кол-во баллов') }}
+                        {{ Form::label('scores', 'Кол-во баллов', ['data-bs-toggle' => "tooltip", 'data-bs-title' => __('tooltips.entrance-examination.scores')]) }}
+                        <span class="text-danger">*</span>
                    </div>
                    <div class="mt-1">
-                        {{ Form::text('scores', $entranceExamination->scores, ['class' => 'form-control']) }}
+                        {{ Form::text('scores', $entranceExamination->scores, ['class' => 'form-control', 'data-bs-toggle' => "tooltip", 'data-bs-title' => __('tooltips.entrance-examination.scores'), 'required']) }}
                    </div>
-                    <div>
+                    <div class="text-danger">
                        @if ($errors->any())
                            {{ $errors->first('scores') }}
                        @endif
                    </div>

-                    <div class="mt-3">
-                        {{ Form::label('position', 'Позиция') }}
-                    </div>
-                    <div class="mt-1">
-                        {{ Form::text('position', $entranceExamination->position, ['class' => 'form-control']) }}
-                    </div>
-                    <div>
-                        @if ($errors->any())
-                            {{ $errors->first('position') }}
-                        @endif
-                    </div>
-
                    <div class="mt-3">
                        {{ Form::submit('Изменить', ['class' => 'btn btn-primary']) }}
                    </div>