Repository navigation
Problem (and solution) for sites in subdirectory #23
Description
Activity
- added a commit that references this issue
on Sep 4, 2020 Hey, that look sensible to me. It could even be done in the
get_docs_base()function directly. Are you able to make a pull request for this? Sorry for the long delay in replying, I have too much other stuff on and this hasn't been a priority as it's still mostly functional.sure, will do!
- added a commit that references this issue
on Jul 21, 2021 Hi guys,
Just an update on this fix. In some cases I don't have the 'path' returned in the parse_url() function. So I changed it like this (php8) :$base_url = parse_url( home_url() ); $base_url_path = rtrim( $base_url['path'] ?? '', '/' ); $config = array( 'basename' => $base_url_path . get_docs_base(),Good point, but:
- we should return an empty string if there's no path, otherwise we will end up with two slashes in a row when we concatatenate it
- I'm not sure what versions of PHP we want to support. By default I tend to follow what WordPress supports
updated the PR
Thanks
The rtrim should avoid double slashes
I mentioned php8 because i'm using it, but the piece of code I pasted is php7+ I thinkPHP7+ is fine, just wanted to note you could use
parse_url( home_url(), PHP_URL_PATH )to return only the path component of the URL, pretty sure that'll give you an empty string if there isn't one.parse_url( home_url(), PHP_URL_PATH )
This looks way better. Updated the PR
Indeed it is better :)
Hi,
Great plugin!
I found an issue when using it in a website that is not in the root folder of the web server. All the links to the endpoints in the documentation would point to
/api-docs/.... So if I was visiting the docs inlocalhost/my-site/api-docs, for example, all links would point tolocalhost/api-docs.It took me a while even to notice that, because as all the interaction happens in the React app, the links actually worked. The contents were correctly loaded, even though the address in the address bar would change to an unexisting one.
But after I fixed it, I realized that all the content from the default pages were not being loaded.
So after tweaking a bit, I came up with the solution below.
In
Restsplain.php, line 73, where you read:I changed to:
I really don't know the implications of that, Ive being playing with the plugin for an hour or so, but it seems to work fine.
If you had the time to have a look and see if it looks good it would be great to see this fix incorporated. I'd be happy to send a pull request or feel free to implement it directly.
cheers