JsonApiListener::_parseIncludes should identify whitespace differences and throw a more specific error for these

FriendsOfCake / crud-json-api

Build advanced JSON API Servers with almost no code.

https://crud-json-api.readthedocs.io/

MIT License

56 stars 32 forks source link

JsonApiListener::_parseIncludes should identify whitespace differences and throw a more specific error for these #186

Open geoidesic opened 1 year ago

geoidesic commented 1 year ago

It's easy for querystring parameters to accidentally include whitespace (via copy / paste etc.) which can make it hard to debug. This would be easily checked for in this function and a message to that effect (i.e. whitespace found in include string) added.

https://github.com/FriendsOfCake/crud-json-api/blob/0d8856ea413694a92e3de521aff78585ed2cae54/src/Listener/JsonApiListener.php#L366