Respect installations with base paths

Added a simple unit test to check if base paths are considered.

Looks good to me, just a suggestion for

['/web/ce-api', '', NULL]
['/ce-api?param=1', '', TRUE]
['/ce-api/?param=1', '', TRUE]

I don't know how that works in your list's organization -- lots of dimensions unfortunately :)

Thank you! ['/ce-api?param=1', '', TRUE] actually failed. Fixed it and added a couple of test cases.

Looks good.

it's really great to have a test added here! I do think it's not correctly checking things though, plese take look.

Commented and added tests for altered paths.

I see, makes sense.

TEst coverage improvement is great, however not fully there:

e.g. from test-data

'Frontpage param CE-API trailing slash' => ['/ce-api/?param=1', '', TRUE, '/'],

We should see this resulting into /?param=1 - thus we want to test not only the resulting path but the resulting request-URI really. Could you adust the test-case like this? Then I think this is ready to go.

1 commit added (and force-pushed with PHPCS fix): added "?param=1" to all expected/tested values, where the input values had them. Nothing else.

I hope this is good to merge now, per the previous comment. It simplifies other issues that need to add phpunit tests.

ok, thanks. that works well, merged then.

Automatically closed - issue fixed for 2 weeks with no activity.