Skip to content

Commit b2a8a18

Browse files
author
Jeremiah VALERIE
committed
wip
1 parent a83c575 commit b2a8a18

33 files changed

+116
-80
lines changed

‎docs/definitions/expression-language.md‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -386,7 +386,7 @@ MyType:
386386
fields:
387387
name:
388388
type: String!
389-
resolve: "@=service('my_private_service').formatName(value)"
389+
resolve: "@=my_private_service.formatName(value)"
390390
```
391391
392392
To use a vendor private services:

‎src/Definition/GraphQLServices.php‎

Lines changed: 8 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,11 @@
66

77
use GraphQL\Type\Definition\ResolveInfo;
88
use GraphQL\Type\Definition\Type;
9+
use Overblog\GraphQLBundle\Resolver\MutationResolver;
10+
use Overblog\GraphQLBundle\Resolver\QueryResolver;
11+
use Overblog\GraphQLBundle\Resolver\TypeResolver;
912
use Overblog\GraphQLBundle\Validator\InputValidator;
13+
use Overblog\GraphQLBundle\Validator\InputValidatorFactory;
1014
use Symfony\Component\DependencyInjection\ServiceLocator;
1115

1216
/**
@@ -21,7 +25,7 @@ final class GraphQLServices extends ServiceLocator
2125
*/
2226
public function query(string $alias, ...$args)
2327
{
24-
return $this->get('queryResolver')->resolve([$alias, $args]);
28+
return $this->get(QueryResolver::class)->resolve([$alias, $args]);
2529
}
2630

2731
/**
@@ -31,7 +35,7 @@ public function query(string $alias, ...$args)
3135
*/
3236
public function mutation(string $alias, ...$args)
3337
{
34-
return $this->get('mutationResolver')->resolve([$alias, $args]);
38+
return $this->get(MutationResolver::class)->resolve([$alias, $args]);
3539
}
3640

3741
/**
@@ -41,7 +45,7 @@ public function mutation(string $alias, ...$args)
4145
*/
4246
public function getType(string $typeName): ?Type
4347
{
44-
return $this->get('typeResolver')->resolve($typeName);
48+
return $this->get(TypeResolver::class)->resolve($typeName);
4549
}
4650

4751
/**
@@ -52,7 +56,7 @@ public function getType(string $typeName): ?Type
5256
*/
5357
public function createInputValidator($value, ArgumentInterface $args, $context, ResolveInfo $info): InputValidator
5458
{
55-
return $this->get('input_validator_factory')->create(
59+
return $this->get(InputValidatorFactory::class)->create(
5660
new ResolverArgs($value, $args, $context, $info)
5761
);
5862
}

‎src/DependencyInjection/Compiler/GraphQLServicesPass.php‎

Lines changed: 15 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,6 @@
66

77
use InvalidArgumentException;
88
use Overblog\GraphQLBundle\Definition\GraphQLServices;
9-
use Overblog\GraphQLBundle\Generator\TypeGenerator;
109
use Symfony\Component\DependencyInjection\Compiler\CompilerPassInterface;
1110
use Symfony\Component\DependencyInjection\ContainerBuilder;
1211
use Symfony\Component\DependencyInjection\Reference;
@@ -27,26 +26,30 @@ public function process(ContainerBuilder $container): void
2726

2827
foreach ($taggedServices as $id => $tags) {
2928
foreach ($tags as $attributes) {
30-
if (empty($attributes['alias']) || !is_string($attributes['alias'])) {
31-
throw new InvalidArgumentException(
32-
sprintf('Service "%s" tagged "overblog_graphql.service" should have a valid "alias" attribute.', $id)
33-
);
34-
}
35-
$locateableServices[$attributes['alias']] = new Reference($id);
29+
$locateableServices[] = new Reference($id);
30+
31+
if (array_key_exists('alias', $attributes)) {
32+
if (empty($attributes['alias']) || !is_string($attributes['alias'])) {
33+
throw new InvalidArgumentException(
34+
sprintf('Service "%s" tagged "overblog_graphql.service" should have a valid "alias" attribute.', $id)
35+
);
36+
}
3637

37-
$isPublic = !isset($attributes['public']) || $attributes['public'];
38-
if ($isPublic) {
3938
$expressionLanguageDefinition->addMethodCall(
40-
'addGlobalName',
39+
'addExpressionVariableNameServiceId',
4140
[
42-
sprintf(TypeGenerator::GRAPHQL_SERVICES.'->get(\'%s\')', $attributes['alias']),
4341
$attributes['alias'],
42+
$id,
4443
]
4544
);
4645
}
4746
}
4847
}
49-
$locateableServices['container'] = new Reference('service_container');
48+
$locateableServices[] = new Reference('service_container');
49+
$expressionLanguageDefinition->addMethodCall(
50+
'addExpressionVariableNameServiceId',
51+
['container', 'service_container']
52+
);
5053

5154
$container->findDefinition(GraphQLServices::class)->addArgument($locateableServices);
5255
}

‎src/ExpressionLanguage/ExpressionFunction/DependencyInjection/Parameter.php‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -13,8 +13,8 @@ public function __construct($name = 'parameter')
1313
{
1414
parent::__construct(
1515
$name,
16-
fn (string $value) => "$this->gqlServices->get('container')->getParameter($value)",
17-
static fn (array $arguments, $paramName) => $arguments[TypeGenerator::GRAPHQL_SERVICES]->get('container')->getParameter($paramName)
16+
fn (string $value) => "$this->gqlServices->get('service_container')->getParameter($value)",
17+
static fn (array $arguments, $paramName) => $arguments[TypeGenerator::GRAPHQL_SERVICES]->get('service_container')->getParameter($paramName)
1818
);
1919
}
2020
}

‎src/ExpressionLanguage/ExpressionFunction/DependencyInjection/Service.php‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -13,8 +13,8 @@ public function __construct($name = 'service')
1313
{
1414
parent::__construct(
1515
$name,
16-
fn (string $serviceId) => "$this->gqlServices->get('container')->get($serviceId)",
17-
static fn (array $arguments, $serviceId) => $arguments[TypeGenerator::GRAPHQL_SERVICES]->get('container')->get($serviceId)
16+
fn (string $serviceId) => "$this->gqlServices->get('service_container')->get($serviceId)",
17+
static fn (array $arguments, $serviceId) => $arguments[TypeGenerator::GRAPHQL_SERVICES]->get('service_container')->get($serviceId)
1818
);
1919
}
2020
}

‎src/ExpressionLanguage/ExpressionFunction/GraphQL/Arguments.php‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -13,8 +13,8 @@ public function __construct()
1313
{
1414
parent::__construct(
1515
'arguments',
16-
fn ($mapping, $data) => "$this->gqlServices->get('container')->get('overblog_graphql.arguments_transformer')->getArguments($mapping, $data, \$info)",
17-
static fn (array $arguments, $mapping, $data) => $arguments[TypeGenerator::GRAPHQL_SERVICES]->get('container')->get('overblog_graphql.arguments_transformer')->getArguments($mapping, $data, $arguments['info'])
16+
fn ($mapping, $data) => "$this->gqlServices->get('service_container')->get('overblog_graphql.arguments_transformer')->getArguments($mapping, $data, \$info)",
17+
static fn (array $arguments, $mapping, $data) => $arguments[TypeGenerator::GRAPHQL_SERVICES]->get('service_container')->get('overblog_graphql.arguments_transformer')->getArguments($mapping, $data, $arguments['info'])
1818
);
1919
}
2020
}

‎src/ExpressionLanguage/ExpressionFunction/Security/GetUser.php‎

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -6,15 +6,16 @@
66

77
use Overblog\GraphQLBundle\ExpressionLanguage\ExpressionFunction;
88
use Overblog\GraphQLBundle\Generator\TypeGenerator;
9+
use Overblog\GraphQLBundle\Security\Security;
910

1011
final class GetUser extends ExpressionFunction
1112
{
1213
public function __construct()
1314
{
1415
parent::__construct(
1516
'getUser',
16-
fn () => "$this->gqlServices->get('security')->getUser()",
17-
static fn (array $arguments) => $arguments[TypeGenerator::GRAPHQL_SERVICES]->get('security')->getUser()
17+
fn () => "$this->gqlServices->get('".Security::class.'\')->getUser()',
18+
static fn (array $arguments) => $arguments[TypeGenerator::GRAPHQL_SERVICES]->get(Security::class)->getUser()
1819
);
1920
}
2021
}

‎src/ExpressionLanguage/ExpressionFunction/Security/HasAnyPermission.php‎

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -6,15 +6,16 @@
66

77
use Overblog\GraphQLBundle\ExpressionLanguage\ExpressionFunction;
88
use Overblog\GraphQLBundle\Generator\TypeGenerator;
9+
use Overblog\GraphQLBundle\Security\Security;
910

1011
final class HasAnyPermission extends ExpressionFunction
1112
{
1213
public function __construct()
1314
{
1415
parent::__construct(
1516
'hasAnyPermission',
16-
fn ($object, $permissions) => "$this->gqlServices->get('security')->hasAnyPermission($object, $permissions)",
17-
static fn (array $arguments, $object, $permissions) => $arguments[TypeGenerator::GRAPHQL_SERVICES]->get('security')->hasAnyPermission($object, $permissions)
17+
fn ($object, $permissions) => "$this->gqlServices->get('".Security::class."')->hasAnyPermission($object, $permissions)",
18+
static fn (array $arguments, $object, $permissions) => $arguments[TypeGenerator::GRAPHQL_SERVICES]->get(Security::class)->hasAnyPermission($object, $permissions)
1819
);
1920
}
2021
}

‎src/ExpressionLanguage/ExpressionFunction/Security/HasAnyRole.php‎

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -6,15 +6,16 @@
66

77
use Overblog\GraphQLBundle\ExpressionLanguage\ExpressionFunction;
88
use Overblog\GraphQLBundle\Generator\TypeGenerator;
9+
use Overblog\GraphQLBundle\Security\Security;
910

1011
final class HasAnyRole extends ExpressionFunction
1112
{
1213
public function __construct()
1314
{
1415
parent::__construct(
1516
'hasAnyRole',
16-
fn ($roles) => "$this->gqlServices->get('security')->hasAnyRole($roles)",
17-
static fn (array $arguments, $roles) => $arguments[TypeGenerator::GRAPHQL_SERVICES]->get('security')->hasAnyRole($roles)
17+
fn ($roles) => "$this->gqlServices->get('".Security::class."')->hasAnyRole($roles)",
18+
static fn (array $arguments, $roles) => $arguments[TypeGenerator::GRAPHQL_SERVICES]->get(Security::class)->hasAnyRole($roles)
1819
);
1920
}
2021
}

‎src/ExpressionLanguage/ExpressionFunction/Security/HasPermission.php‎

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -6,15 +6,16 @@
66

77
use Overblog\GraphQLBundle\ExpressionLanguage\ExpressionFunction;
88
use Overblog\GraphQLBundle\Generator\TypeGenerator;
9+
use Overblog\GraphQLBundle\Security\Security;
910

1011
final class HasPermission extends ExpressionFunction
1112
{
1213
public function __construct()
1314
{
1415
parent::__construct(
1516
'hasPermission',
16-
fn ($object, $permission) => "$this->gqlServices->get('security')->hasPermission($object, $permission)",
17-
static fn (array $arguments, $object, $permission) => $arguments[TypeGenerator::GRAPHQL_SERVICES]->get('security')->hasPermission($object, $permission)
17+
fn ($object, $permission) => "$this->gqlServices->get('".Security::class."')->hasPermission($object, $permission)",
18+
static fn (array $arguments, $object, $permission) => $arguments[TypeGenerator::GRAPHQL_SERVICES]->get(Security::class)->hasPermission($object, $permission)
1819
);
1920
}
2021
}

0 commit comments

Comments
 (0)