Please support customizing the route key.

[urbsny]

• I confirm that I have read and attempted the tips in the Troubleshooting Guide.

What happened?

1. I have the Eloquent model which customized route key.
2. The model key type is integer and route key type is string.
3. But GetFromLaravelAPI::class only use key type.

Screenshots and stack traces:

My environment:

• PHP version (from php -v): 8.0.3
• Framework (Laravel/Lumen): Laravel
• Laravel/Lumen version (from composer show laravel/framework or composer show laravel/lumen-framework): v8.49.0
• Scribe version (from composer show knuckleswtf/scribe): 3.3.2

Additional info:
In my case, I changed GetFromLaravelAPI.php to the following and it worked fine.

scribe/src/Extracting/Strategies/UrlParameters/GetFromLaravelAPI.php

Line 69 in 4b59fb9

$type = $this->normalizeTypeName($argumentInstance->getKeyType());

                    $typeName = $argumentInstance->getKeyName() === $argumentInstance->getRouteKeyName()
                        ? $argumentInstance->getKeyType() : 'string';
                    $type = $this->normalizeTypeName($typeName);

[shalvah]

Sure, send in a PR. But what does the route key method return for people who haven't defined it for their model?

[urbsny]

Thank you for your quick response. I'll send PR.

what does the route key method return for people who haven't defined it for their model?

Base model return key name. So I don't think it will affect the people who haven't defined it.

https://github.com/laravel/framework/blob/6a3ddf21096b1801d9e7f221f72ae78d90eb3622/src/Illuminate/Database/Eloquent/Model.php#L1786

    public function getRouteKeyName()
    {
        return $this->getKeyName();
    }

[shalvah]

Cool, should be easy to send in a PR then.