Merge branch 'issue-81' into 'develop'

fix: 🐛 error on productController@productFiltered from sentry

See merge request ldc-web/projets/tregalette!33
This commit is contained in:
karl.muller 2025-04-03 13:22:15 +00:00
commit ab2cfd8fd1
4 changed files with 36 additions and 1 deletions

View file

@ -46,6 +46,10 @@ class ProductController extends Controller
*/
public function productsFiltered(): \Illuminate\Http\JsonResponse
{
if (!request()->has('range_id') or !request()->has('page')) {
abort(404);
}
$productModels = Product::query()
->where('published', true)
->where('range_id', request()->range_id)

View file

@ -66,5 +66,6 @@ class Kernel extends HttpKernel
'verified' => \Illuminate\Auth\Middleware\EnsureEmailIsVerified::class,
'htmlsanitize' => \App\Http\Middleware\HtmlSanitizer::class,
'crm_questions' => \App\Http\Middleware\CrmMiddlewareQuestions::class,
'json_only' => \App\Http\Middleware\JsonOnly::class,
];
}

View file

@ -0,0 +1,26 @@
<?php
namespace App\Http\Middleware;
use Illuminate\Http\Request;
class JsonOnly
{
/**
* Handle an incoming request.
*
* @param \Illuminate\Http\Request $request
* @param \Closure $next
* @return mixed
*/
public function handle(Request $request, \Closure $next)
{
if (
(!($request->isJson() or $request->expectsJson()))
) {
abort(400, 'Invalid JSON');
}
return $next($request);
}
}

View file

@ -66,7 +66,11 @@ Route::middleware(['throttle:credentials'])->name('fo.')->group(function () {
Route::get('/gamme/{slug}', [RangeController::class, 'show'])->name('ranges.show');
// * PRODUCTS
Route::get('/produit/filtered', [ProductController::class, 'productsFiltered'])->name('products.filtered');
Route::middleware('json_only')->get(
'/produit/filtered',
[ProductController::class, 'productsFiltered']
)->name('products.filtered');
Route::get('/produit/{slug}', [ProductController::class, 'show'])->name('products.show');
// * COMMITMENTS