fix: 🚨 resolve phpstan errors

This commit is contained in:
Alexis Gousseau 2026-02-12 16:14:11 +01:00
parent db58181e68
commit 09d1826c7f
10 changed files with 44 additions and 14 deletions

View file

@ -282,6 +282,7 @@ enum ContactFormType: int
self::press => [
'communicationgroupe@ldc.fr',
],
// phpcs:ignore Squiz.Commenting.InlineComment
}; //end match
}
}

View file

@ -75,7 +75,7 @@ enum DynamicFieldTypes: int
self::date->name => 'required|date',
self::color->name => 'required|string|hex_color',
self::model->name => 'required|numeric|exists:dynamic_pages,id',
self::staticroute->name => 'required|numeric|between:1,10',
self::staticroute->name => 'required|numeric|between:1,99',
// Specific inputs.
self::position->name => 'required|numeric|between:1,5',

View file

@ -92,8 +92,10 @@ class CompanySectorController extends Controller
*/
public function show(CompanySector $company_sector): \Inertia\Response
{
$companies = $company_sector->companies()->paginate(ItemsPerPaginationEnum::five->value());
return Inertia::render('CompanySectors/Show', [
'data' => CommonResource::make($company_sector),
'companies' => CommonResource::collection($companies),
'modelName' => $this->modelName,
]);
}

View file

@ -43,9 +43,8 @@ class InputsController extends Controller
return response()->json(['error' => 'Invalid model class'], 404);
}
// Determine the field to display.
$model = new $modelClass();
$model = new $modelClass();
$isTranslatable = method_exists($model, 'getTranslations');
$locale = app()->getLocale();
@ -184,7 +183,8 @@ class InputsController extends Controller
public function uploadFile(UploadFileRequest $request): \Illuminate\Http\JsonResponse
{
// Get the class of the target model and verify its existence.
$modelClass = "App\\Models\\$request->folder";
$modelName = Str::singular($request->validated()['folder']);
$modelClass = "App\\Models\\$modelName";
if (!class_exists($modelClass)) {
return response()->json(['error' => 'Invalid model class'], 404);
}

View file

@ -20,9 +20,10 @@ class ForceLocale
* The issue occurred when two browser tabs were open in different locales:
* the data being saved could end up using the other tab’s locale.
*
* @param Request $request
* @param Closure $next
* @param \Illuminate\Http\Request $request
* @param Closure $next
* @return mixed
* @throws ValidationException Not supported locale.
*/
public function handle(Request $request, Closure $next): mixed
{
@ -43,7 +44,7 @@ class ForceLocale
$requestLocale = $request->locale ?? config('app.fallback_locale');
// 2. If prefix is not in supportedLocales
if (!in_array($requestLocale, LaravelLocalization::getSupportedLanguagesKeys(), true)) {
throw ValidationException::withMessages(["Langue" => "locale pas accepté"]);
throw ValidationException::withMessages(["Langue" => "Not supported locale."]);
}
// 3. Set new locale if locale and prefix are not the same
if ($requestLocale !== app()->getLocale()) {

View file

@ -5,13 +5,18 @@ namespace App\Http\Middleware;
use Closure;
use Illuminate\Http\Request;
class Lang
{
/**
* Handle an incoming request.
* Not used but need it for uwil package.
*
* @param \Illuminate\Http\Request $request
* @param \Closure $next
* @return \Symfony\Component\HttpFoundation\Response
*/
public function handle(Request $request, Closure $next): mixed
{
return $next($request);
}
}

View file

@ -3,9 +3,10 @@
namespace App\Http\Requests\Back\Companies;
use App\Models\Company;
use Illuminate\Foundation\Http\FormRequest;
use App\Traits\Requests\HasInertiaPicture;
use Illuminate\Foundation\Http\FormRequest;
use Illuminate\Support\Facades\Gate;
use Uwil\LaravelCommon\Rules\WysiwygLength;
class StoreCompanyRequest extends FormRequest
{
@ -42,13 +43,13 @@ class StoreCompanyRequest extends FormRequest
public function rules(): array
{
return [
'name' => 'required|string|between:3,255',
'name' => 'required|string|between:3,255|unique:companies,name',
'logo' => $this->pictureRules(
'required|nullable|file|mimes:jpg,jpeg,png,webp|max:5120|dimensions:width=500,height=500',
),
'responsable' => 'required|nullable|string|between:3,255',
'url' => 'required|nullable|string|between:3,255',
'informations' => 'required|string|between:3,255',
'url' => 'nullable|string|between:3,255',
'informations' => ['required', 'string', new WysiwygLength(10, 5000)],
'employees' => 'required|nullable|numeric|between:0,100000',
'latitude' => 'required|numeric|between:-90,90',
'longitude' => 'required|numeric|between:-180,180',

View file

@ -15,4 +15,22 @@ class UpdateCompanyRequest extends StoreCompanyRequest
{
return Gate::check('update', $this->route('company'));
}
/**
* Get the validation rules that apply to the request.
*
* @return array
*/
public function rules(): array
{
$rules = [
'name' => [
'required',
'string',
'between:3,255',
'unique:companies,name,' . request()->company->getKey(),
],
];
return \array_merge(parent::rules(), $rules);
}
}

View file

@ -4,6 +4,7 @@ namespace App\Http\Requests\Back;
use App\Enums\ElementPositionEnum;
use App\Enums\DynamicFieldTypes;
use App\Enums\StaticRoutesEnum;
use App\Models\PageConfig;
use App\Traits\Requests\HasInertiaPicture;
use Illuminate\Foundation\Http\FormRequest;

View file

@ -34,6 +34,7 @@ class UploadFileRequest extends FormRequest
'max:5048',
'dimensions:min_width=567,min_height=567,max_width:1008,max_height:1008',
],
'folder' => 'required|string|between:3,255',
];
}
}