JSON:API: input schema puts an embedded input DTO collection under `relationships`, but the denormalizer reads it from `attributes`
Los mantenedores suelen responder en 1 día
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Aptitud para principiantes
- 58/100
Línea de trabajo
Start in JsonApi\JsonSchema\SchemaFactory::buildDefinitionPropertiesSchema(), where getRelationship() is called with the resource class instead of the input class. Compare how the denormalizer splits attributes from relationships for an input DTO, then reproduce with the issue's Order/OrderInput classes and run bin/console api:openapi:export. Done when lines is documented under attributes as an array of OrderLineInput for the input schema, and the output schema is unchanged.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
API Platform version(s) affected
4.4.4
Description
A resource has a relation to another resource. A write operation uses an input DTO, and the DTO has a property with the same name that holds plain (non-resource) objects.
For the JSON:API format, the generated request body schema puts this property under data.relationships as resource linkage (type + id). The denormalizer does not accept that shape. It accepts the objects only under data.attributes, and that shape is not documented.
The cause looks like JsonApi\JsonSchema\SchemaFactory::buildDefinitionPropertiesSchema(). It calls getRelationship($className, ...) with the resource class, not with the input class. So the schema uses the type of the resource property (list<OrderLine>, a resource) and not the type of the input property (list<OrderLineInput>, a plain DTO).
How to reproduce
#[ApiResource(operations: [new Get(provider: [self::class, 'provide'])])]
final class OrderLine
{
public function __construct(public int $id = 1, public string $sku = 'ABC', public int $quantity = 1) {}
public static function provide(): self { return new self(); }
}
final class OrderLineInput
{
public string $sku;
public int $quantity;
}
final class OrderInput
{
public string $reference;
/** @var list<OrderLineInput> */
public array $lines = [];
}
#[ApiResource(operations: [
new Get(provider: [self::class, 'provide']),
new Post(input: OrderInput::class, processor: [self::class, 'process']),
])]
final class Order
{
/** @param list<OrderLine> $lines */
public function __construct(public int $id = 1, public string $reference = 'R-1', public array $lines = []) {}
public static function provide(): self { return new self(); }
public static function process(OrderInput $data, Operation $operation): self
{
$lines = [];
foreach ($data->lines as $i => $line) {
$lines[] = new OrderLine($i + 1, $line->sku, $line->quantity);
}
return new self(1, $data->reference, $lines);
}
}
The generated Order.OrderInput.jsonapi schema (shortened):
"attributes": { "properties": { "reference": { "type": "string" } } },
"relationships": {
"properties": {
"lines": { "properties": { "data": { "type": "array", "items": {
"properties": { "type": { "type": "string" }, "id": { "type": "string", "format": "iri-reference" } },
"required": ["type", "id"]
} } } }
}
}
A request that follows the schema fails. The processor gets an OrderLineInput whose properties were never set:
POST /orders
Content-Type: application/vnd.api+json
{"data":{"type":"Order","attributes":{"reference":"R-9"},"relationships":{"lines":{"data":[{"type":"OrderLine","id":"/order_lines/1"}]}}}}
Error: Typed property App\ApiResource\OrderLineInput::$sku must not be accessed before initialization
A request with the lines under attributes works and answers 201, but the schema does not document it:
POST /orders
Content-Type: application/vnd.api+json
{"data":{"type":"Order","attributes":{"reference":"R-9","lines":[{"sku":"X","quantity":2}]}}}
Expected: for an input schema, the attributes/relationships split follows the input class. lines is then documented under attributes as an array of OrderLineInput, which is the shape the denormalizer reads.
Possible Solution
Use the input class (or the output class for output schemas) in getRelationship(), not the resource class.
Additional Context
Checked on api-platform/core v4.4.4, Symfony 7.4, PHP 8.5, in an empty Symfony app. The code of JsonApi\JsonSchema\SchemaFactory on main is the same as in v4.4.4.
The app has only FrameworkBundle, TwigBundle and ApiPlatformBundle, with this config:
$c->extension('api_platform', [
'mapping' => ['paths' => ['%kernel.project_dir%/src/ApiResource']],
'formats' => ['jsonapi' => ['application/vnd.api+json']],
'docs_formats' => ['jsonopenapi' => ['application/vnd.openapi+json']],
'doctrine' => ['enabled' => false],
]);
The spec comes from bin/console api:openapi:export.
- Lenguaje dominante
- PHP
- Estrellas
- 2.6k
- Forks
- 989
- Merge medio
- 1 d 19 h
- PR fusionados (30 d)
- 106
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Tiene una plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de api-platform/core
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
api-platform/core#8744 ·
Los mantenedores suelen responder en 1 día
-
JSON:API: resource schemas (`*.jsonapi`) have `properties` but no `type: object`Posiblemente ocupada @ousamabenyounes la tomó hace 3 días. Abierto
Dificultad 2/5 Menos de una hora Aptitud para principiantes 80/100
api-platform/core#8701 ·
Los mantenedores suelen responder en 1 día
-
[Json-Schema] Nested resources lost externalDoc and description dataPosiblemente ocupada Un pull request vinculado a esta issue está abierto o ya se fusionó. Abierto
Dificultad 3/5 1-2 días Aptitud para principiantes 68/100
api-platform/core#8721 ·
Los mantenedores suelen responder en 1 día
-
JSON:API: 404/405 router errors are empty, and error objects have a non-spec `type` memberPosiblemente ocupada Un pull request vinculado a esta issue está abierto o ya se fusionó. Abierto
Dificultad 3/5 1-2 días Aptitud para principiantes 50/100
api-platform/core#8704 ·
Los mantenedores suelen responder en 1 día
-
JSON:API: request documents with no `data`, no `type` or a wrong `type` are accepted with 201Abierto
Dificultad 3/5 1-2 días Aptitud para principiantes 62/100
api-platform/core#8703 ·
Los mantenedores suelen responder en 1 día
Todos los issues de api-platform/core
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
grokability/snipe-it#19869 · 2 comentarios ·
Los mantenedores suelen responder en 1 día
-
sync-en
Dificultad 1/5 Menos de una hora Aptitud para principiantes 82/100
Los mantenedores suelen responder en 1 día
-
sync-en
Dificultad 1/5 Menos de una hora Aptitud para principiantes 77/100
Los mantenedores suelen responder en 3 días
-
Перевод устарел
Dificultad 1/5 Menos de una hora Aptitud para principiantes 74/100