fix baseUrl and an simple fix for multiple types in an param (in v2\explorer)

Luracast / Restler

Simple and effective multi-format Web API Server to host your PHP API as Pragmatic REST and/or RESTful API

http://luracast.com/products/restler/

GNU Lesser General Public License v2.1

1.36k stars 315 forks source link

fix baseUrl and an simple fix for multiple types in an param (in v2\explorer) #653

Closed timmit-nl closed 3 years ago

timmit-nl commented 3 years ago

I have noticed the baseUrl doens't work as I would expect (remove it from the path to check for the correct route).

This fixes that.

Also we have some issues with multiple types of an param in Explorer v2: /**

test function
@param string|null $testVar
@return void **/ testFunction(?string $testVar) : void

So the quick fix is to only gets the first type.

And my editor fixes some PRS12 incorrections when I saved the file. It makes it better to read, so I left them.

timmit-nl commented 3 years ago

this last commits is that if you use a baseUrl with domain (like the tool will generate if not there, so I think it is the propper use) it strips it from the path. So you get an clean path. Also an fix that if you use a baseUrl without domain you get a notice in src/Explorer/v2/Explorer.php

timmit-nl commented 3 years ago

I will close this one as I have a new clean version of only the baseUrl problem: https://github.com/Luracast/Restler/pull/671