Skip to content

Enhance type definitions for Executor #720

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
vladar merged 2 commits into from
Sep 16, 2020
Merged

Enhance type definitions for Executor #720

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
9d73e40
Enhance type definitions for Executor
spawnia Aug 24, 2020
79d5c74
Fix code style
spawnia Aug 24, 2020
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
37 changes: 19 additions & 18 deletions src/Executor/Executor.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@
*/
class Executor
{
/** @var callable|string[] */
/** @var callable */
private static $defaultFieldResolver = [self::class, 'defaultFieldResolver'];

/** @var PromiseAdapter */
Expand All @@ -35,7 +35,7 @@ public static function getDefaultFieldResolver() : callable
}

/**
* Custom default resolve function.
* Set a custom default resolve function.
*/
public static function setDefaultFieldResolver(callable $fieldResolver)
{
Expand All @@ -47,6 +47,9 @@ public static function getPromiseAdapter() : PromiseAdapter
return self::$defaultPromiseAdapter ?? (self::$defaultPromiseAdapter = new SyncPromiseAdapter());
}

/**
* Set a custom default promise adapter.
*/
public static function setPromiseAdapter(?PromiseAdapter $defaultPromiseAdapter = null)
{
self::$defaultPromiseAdapter = $defaultPromiseAdapter;
Expand All @@ -58,9 +61,7 @@ public static function getImplementationFactory() : callable
}

/**
* Custom executor implementation factory.
*
* Will be called with as
* Set a custom executor implementation factory.
*/
public static function setImplementationFactory(callable $implementationFactory)
{
Expand All @@ -70,13 +71,13 @@ public static function setImplementationFactory(callable $implementationFactory)
/**
* Executes DocumentNode against given $schema.
*
* Always returns ExecutionResult and never throws. All errors which occur during operation
* execution are collected in `$result->errors`.
* Always returns ExecutionResult and never throws.
* All errors which occur during operation execution are collected in `$result->errors`.
*
* @param mixed|null $rootValue
* @param mixed|null $contextValue
* @param mixed[]|ArrayAccess|null $variableValues
* @param string|null $operationName
* @param mixed|null $rootValue
* @param mixed|null $contextValue
* @param array<mixed>|ArrayAccess|null $variableValues
* @param string|null $operationName
*
* @return ExecutionResult|Promise
*
Expand Down Expand Up @@ -119,10 +120,10 @@ public static function execute(
*
* Useful for async PHP platforms.
*
* @param mixed|null $rootValue
* @param mixed|null $contextValue
* @param mixed[]|null $variableValues
* @param string|null $operationName
* @param mixed|null $rootValue
* @param mixed|null $contextValue
* @param array<mixed>|null $variableValues
* @param string|null $operationName
*
* @return Promise
*
Expand Down Expand Up @@ -161,9 +162,9 @@ public static function promiseToExecute(
* and returns it as the result, or if it's a function, returns the result
* of calling that function while passing along args and context.
*
* @param mixed $objectValue
* @param mixed[] $args
* @param mixed|null $contextValue
* @param mixed $objectValue
* @param array<string, mixed> $args
* @param mixed|null $contextValue
*
* @return mixed|null
*/
Expand Down
Loading