RetailCRM-Mirror/graphql-php
1
0
Fork 0
mirror of https://github.com/retailcrm/graphql-php.git synced 2024-11-25 14:26:08 +03:00
Code Issues Projects Releases Wiki Activity

Removing data elements from response if the error throwing

Browse Source
This commit is contained in:
Ilya Shaydullin 2018-05-07 17:01:18 +07:00
parent ec0985619f
commit c7f114d90b
2 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
src/Executor/Executor.php
View File

@ -311,7 +311,10 @@ class Executor
return null;
})
->then(function ($data) {
return new ExecutionResult((array) $data, $this->exeContext->errors);
if ($data === null){
$data = (array) $data;
}
return new ExecutionResult($data, $this->exeContext->errors);
});
}

3
tests/Executor/NonNullTest.php
View File

@ -732,7 +732,6 @@ class NonNullTest extends \PHPUnit_Framework_TestCase
';
$expected = [
'data' => null,
'errors' => [
FormattedError::create($this->nonNullSyncError->getMessage(), [new SourceLocation(2, 17)])
]
@ -750,7 +749,6 @@ class NonNullTest extends \PHPUnit_Framework_TestCase
$ast = Parser::parse($doc);
$expected = [
'data' => null,
'errors' => [
FormattedError::create($this->nonNullPromiseError->getMessage(), [new SourceLocation(2, 17)]),
]
@ -789,7 +787,6 @@ class NonNullTest extends \PHPUnit_Framework_TestCase
$ast = Parser::parse($doc);
$expected = [
'data' => null,
'errors' => [
[
'debugMessage' => 'Cannot return null for non-nullable field DataType.nonNullPromise.',