The way we get $canonical_path from $_GET['q'] is quite odd. I propose simply strip $endpoint_path from $_GET['q'] to get $canonical_path.

See attached patch.

Some comments added.

CommentFileSizeAuthor
services-rest_server_server-refactor.patch1.08 KBygerasimov

Comments

marcingy’s picture

marcingy commented
Status: Needs review » Reviewed & tested by the community

Have run some tests and endpoints are still found and this is nice simplification of code.

marcingy’s picture

marcingy commented
Status: Reviewed & tested by the community » Patch (to be ported)

Commited thanks does this need backporting?

marcingy’s picture

marcingy commented
Status: Patch (to be ported) » Needs work
marcingy’s picture

marcingy commented
Version: 7.x-3.x-dev » 6.x-3.x-dev
marcingy’s picture

marcingy commented
Status: Needs work » Fixed

Status: Fixed » Closed (fixed)

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