Skip to content

Commit e2f5d79

Browse files
Sammyjo20juse-less
andauthored
Apply suggestions from code review
Co-authored-by: Juse Less <me@juseless.dev>
1 parent d04972e commit e2f5d79

File tree

1 file changed

+10
-6
lines changed

1 file changed

+10
-6
lines changed

src/Helpers/OAuth2/OAuthConfig.php

Lines changed: 10 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -60,9 +60,9 @@ class OAuthConfig
6060
/**
6161
* Callable that modifies the OAuth requests
6262
*
63-
* @var callable(\Saloon\Contracts\Request): (void)|null
63+
* @var \Closure(\Saloon\Contracts\Request): (void)|null
6464
*/
65-
protected mixed $requestModifier = null;
65+
protected Closure $requestModifier = null;
6666

6767
/**
6868
* The default scopes that will be applied to every authorization URL.
@@ -235,21 +235,25 @@ public function setDefaultScopes(array $defaultScopes): static
235235
/**
236236
* Set the request modifier callable which can be used to modify the request being sent
237237
*
238-
* @param callable(\Saloon\Contracts\Request): (void) $requestModifier
238+
* @template TRequest of \Saloon\Contracts\Request
239+
*
240+
* @param callable(TRequest): (void) $requestModifier
239241
* @return $this
240242
*/
241243
public function setRequestModifier(callable $requestModifier): static
242244
{
243-
$this->requestModifier = $requestModifier;
245+
$this->requestModifier = $requestModifier(...);
244246

245247
return $this;
246248
}
247249

248250
/**
249251
* Invoke the OAuth2 config request modifier
250252
*
251-
* @param \Saloon\Contracts\Request $request
252-
* @return \Saloon\Contracts\Request
253+
* @template TRequest of \Saloon\Contracts\Request
254+
*
255+
* @param TRequest $request
256+
* @return TRequest
253257
*/
254258
public function invokeRequestModifier(Request $request): Request
255259
{

0 commit comments

Comments
 (0)