Schema processing fails when "allOf" references to nullable object

Abierto

#1,066 abierto el 28/6/2024 1 comentario 0 reacciones 0 asignados Ver en GitHub

Nadie ha tomado este issue todavía.

Evaluación

Dificultad: 3/5
Tiempo estimado: 1-2 días
Aptitud para principiantes: 45/100

Tipo de issue: Error
Claridad: Bastante claro
Estado de actividad: Estancado
Stack tecnológico: openapi, python
Área: api, tooling

Línea de trabajo

Comienza ejecutando openapi-python-client 0.21.1 contra el documento OpenAPI 3.0.1 proporcionado y sigue la ruta de procesamiento del esquema que emite "Cannot take allOf a non-object." Usa las definiciones RelatedItemSuper y RelatedItem con nullable como caso de regresión; se considera terminado cuando la generación del cliente se completa correctamente sin requerir que nullable se cambie a false.

Escrito por el modelo de indexación a partir del texto del issue.

Descripción

Describe the bug
When using a schema that contains an "allOf" reference to a nullable object, the generation fails with the message "Cannot take allOf a non-object".

OpenAPI Spec File
Document to recreate:

openapi: 3.0.1
info:
  title: Title
  description: Description
  version: 1.0.0
servers:
  - url: /api
paths:
  /item:
    get:
      operationId: item
      responses:
        "200":
          description: Successful operation
          content:
            application/json:
              schema:
                $ref: '#/components/schemas/Item'
components:
  schemas:
    RelatedItemSuper:
      type: object
      nullable: true
      properties:
        propertyA:
          type: string
    RelatedItem:
      type: object
      allOf:
        - $ref: '#/components/schemas/RelatedItemSuper'
        - type: object
          properties:
            propertyB:
              type: string
    Item:
      type: object
      nullable: false
      properties:
        key:
          type: string
        relatedItem:
          $ref: '#/components/schemas/RelatedItem'

A real world example can be found here.

Desktop (please complete the following information):

OS: Linux
Python Version: 3.11.2
openapi-python-client version: 0.21.1

Additional context
Changing "nullable" to "false" on "RelatedItem" in the above example works but does not seem to be a suitable solution as the construct used is legitimate, or is it not?

Lenguaje dominante: Python
Estrellas: 2k
Forks: 293
Merge medio: 34 min
PR fusionados (30 d): 1