Skip to content
Closed
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 5 additions & 3 deletions src/Illuminate/Container/Attributes/RouteParameter.php
Original file line number Diff line number Diff line change
Expand Up @@ -5,14 +5,15 @@
use Attribute;
use Illuminate\Contracts\Container\Container;
use Illuminate\Contracts\Container\ContextualAttribute;
use ReflectionParameter;

#[Attribute(Attribute::TARGET_PARAMETER)]
class RouteParameter implements ContextualAttribute
{
/**
* Create a new class instance.
*/
public function __construct(public string $parameter)
public function __construct(public ?string $parameter = null)
{
}

Expand All @@ -21,10 +22,11 @@ public function __construct(public string $parameter)
*
* @param self $attribute
* @param \Illuminate\Contracts\Container\Container $container
* @param \ReflectionParameter $parameter
* @return mixed
*/
public static function resolve(self $attribute, Container $container)
public static function resolve(self $attribute, Container $container, ReflectionParameter $parameter)
{
return $container->make('request')->route($attribute->parameter);
return $container->make('request')->route($attribute->parameter ?? $parameter->getName());
}
}
2 changes: 1 addition & 1 deletion src/Illuminate/Container/BoundMethod.php
Original file line number Diff line number Diff line change
Expand Up @@ -175,7 +175,7 @@ protected static function addDependencyForCallParameter(

unset($parameters[$paramName]);
} elseif ($attribute = Util::getContextualAttributeFromDependency($parameter)) {
$pendingDependencies[] = $container->resolveFromAttribute($attribute);
$pendingDependencies[] = $container->resolveFromAttribute($attribute, $parameter);
} elseif (! is_null($className = Util::getParameterClassName($parameter))) {
if (array_key_exists($className, $parameters)) {
$pendingDependencies[] = $parameters[$className];
Expand Down
7 changes: 4 additions & 3 deletions src/Illuminate/Container/Container.php
Original file line number Diff line number Diff line change
Expand Up @@ -1267,7 +1267,7 @@ protected function resolveDependencies(array $dependencies)
$result = null;

if (! is_null($attribute = Util::getContextualAttributeFromDependency($dependency))) {
$result = $this->resolveFromAttribute($attribute);
$result = $this->resolveFromAttribute($attribute, $dependency);
}

// If the class is null, it means the dependency is a string or some other
Expand Down Expand Up @@ -1416,9 +1416,10 @@ protected function resolveVariadicClass(ReflectionParameter $parameter)
* Resolve a dependency based on an attribute.
*
* @param \ReflectionAttribute $attribute
* @param \ReflectionParameter $parameter
* @return mixed
*/
public function resolveFromAttribute(ReflectionAttribute $attribute)
public function resolveFromAttribute(ReflectionAttribute $attribute, ReflectionParameter $parameter)
{
$handler = $this->contextualAttributes[$attribute->getName()] ?? null;

Expand All @@ -1432,7 +1433,7 @@ public function resolveFromAttribute(ReflectionAttribute $attribute)
throw new BindingResolutionException("Contextual binding attribute [{$attribute->getName()}] has no registered handler.");
}

return $handler($instance, $this);
return $handler($instance, $this, $parameter);
}

/**
Expand Down
2 changes: 1 addition & 1 deletion src/Illuminate/Routing/ResolvesRouteDependencies.php
Original file line number Diff line number Diff line change
Expand Up @@ -76,7 +76,7 @@ public function resolveMethodDependencies(array $parameters, ReflectionFunctionA
protected function transformDependency(ReflectionParameter $parameter, $parameters, $skippableValue)
{
if ($attribute = Util::getContextualAttributeFromDependency($parameter)) {
return $this->container->resolveFromAttribute($attribute);
return $this->container->resolveFromAttribute($attribute, $parameter);
}

$className = Reflector::getParameterClassName($parameter);
Expand Down