thecodingmachine / graphqlite / 253

98%

master: 99%

Pull #87

travis-ci

Migrating away from @Parameter annotation

Having a single @Parameter annotation that gathers all the annotations dedicated to a single parameter leads to a weird syntax.

For instance:

```php
/**
 * @Parameter(for="$translator", annotations={
 *    @Autowire
 * })
 */
```

Furthermore, the PHP Annotations extension for PHPStorm deals poorly with nested annotations and we do not have autofill of use statements.

This PR removes the use of nested annotations in Parameter.
The @Parameter is completely removed in favor of 2 annotations:

@UseInputType
and
@Autowire

The signature of the ParameterMapperInterface is modified. Instead of accepting a ParameterAnnotation argument, it now accepts a ParameterAnnotations (notice the s) class that can be used to access any class implementing the ParameterAnnotationInterface.
Both Autowire and @UseInputType now implement the ParameterAnnotationInterface.

37 of 37 new or added lines in 8 files covered. (100.0%)

2016 of 2055 relevant lines covered (98.1%)

17.8 hits per line