search

membrane-php / membrane-laravel

Laravel integration for Membrane validation library.
Other
3 stars 2 forks source link

Fix incorrectly named config option #6

Closed charjr closed 1 year ago

charjr commented 1 year ago

Fixes several incorrectly named variables.

The config has api_problem_response_types: https://github.com/membrane-php/membrane-laravel/blob/c3c87c7b549f0dced09ed3d37f7205c7496059f6/config/membrane.php#L27

The service provider was looking for api_response_types and trying to provide it to $apiProblemTypes: https://github.com/membrane-php/membrane-laravel/blob/c3c87c7b549f0dced09ed3d37f7205c7496059f6/src/ServiceProvider.php#L34-L36

But the ApiProblemBuilder has $apiResponseTypes! https://github.com/membrane-php/membrane-laravel/blob/c3c87c7b549f0dced09ed3d37f7205c7496059f6/src/ApiProblemBuilder.php#L20

This PR fixes the inconsistencies.