graphql-python / graphql-python/graphql-core-legacy

Violation of Liskov Substitution Principle

Abierto
#247 0 comentarios 0 reacciones 0 asignados Ver en GitHub
Lenguaje dominante
Python
Estrellas
372
Forks
175
Métricas de merge de PR
Sin PR fusionados en 30 d

Descripción

https://github.com/graphql-python/graphql-core/blob/3ba7f9218bca7a039b109b103cf57b1511d970c7/graphql/execution/executor.py#L704

Above referenced line violates Liskov Substitution Principle when used with at least Graphene-Django and the underlying model is itself a subclass of aDjangoObjectType.Meta.model class.

A popular variant of this violation is use of django-polymorphic, which is quite prevalent. However, we're talking basic Python here.

Guía de contribución

No hay ninguna guía de contribución indexada para este repositorio

Línea de trabajo

Empieza en graphql/execution/executor.py#L704 y lee la lógica de ejecución circundante para entender cómo se gestionan las subclases de modelos de Django. Reproduce el comportamiento informado con Graphene-Django y, si está disponible, django-polymorphic; después, identifica el comportamiento de sustitución esperado y añade cobertura de regresión. Se considera terminado cuando el escenario de subclases informado funciona sin romper el comportamiento de ejecución existente.

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

Evaluación

Stack tecnológico
python
Área
api, backend
Tipo de issue
Error
Dificultad
4/5
Tiempo estimado
3-5 días
Estado de actividad
Estancado
Claridad
Necesita aclaración
Aptitud para principiantes
35/100

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.