Export route parameters as path variables

[johnnybenson]

{param} becomes {{param}} and will match on variables defined for the given Postman environment.

[tomirons]

@johnnybenson I'm confused on the use case for this PR. Converting the url parameters to a postman variable doesn't make much sense. I have a couple clarifying questions for you.

1. Are you wanting these to be configurable values?
2. What about converting them to path variables for postman? i.e. posts/{post} -> posts/:post

[johnnybenson]

Ahh cool, I didn't realize Postman gave us path variables. That's clearly much better.

$routeStr = str_replace(['}', '{'], ['', ':'], $route->uri());

Want me to update the PR with something like the above?

FWIW, the UI doesn't highlight path variables for some reason. It's been a while since I've used Postman too. Thanks for the help and the very useful feedback!

[tomirons]

Yeah, you can update this PR to do that. I wish they had a better way of highlighting the path variables too.

[tomirons]

After comparing this to how postman exports collection, there are a couple more things that need to be ironed out.

• The url.path needs to contain everything after the base url exploded by /
• The url.variable needs to contain all of the custom variables from the route

[johnnybenson]

        {
            "name": "api/v1/chunks/{address}",
            "request": {
                "method": "GET",
                "header": [
                    { "key": "Accept", "value": "application/json" },
                    { "key": "Content-Type", "value": "application/json" }
                ],
                "url": {
                    "raw": "{{base_url}}/api/v1/chunks/:address",
                    "host": "{{base_url}}/api/v1/chunks/:address",
                    "path": ["api", "v1", "chunks", ":address"],
                    "variable": [{
                        "key": "address",
                        "value": ""
                    }]
                }
            }
        }

[tomirons]

@johnnybenson I've refactored this whole thing, please take it for a test run and let me know if any issues arise. I did not test this locally. 😅

[johnnybenson]

Your version looks much better :) thanks. Testing locally, we also need to replace {var} with :var in the exploded path and then we're all set.

[tomirons]

Thanks for contributing! ❤️

[andreaselia]

Thank you @johnnybenson for the contribution, and @tomirons for the refactoring ❤️