JSONAPI-Resources / JSONAPI-Resources/jsonapi-resources
has_one foreign_key_on: :related fails on save
Nadie ha tomado este issue todavía.
- Lenguaje dominante
- Ruby
- Estrellas
- 2.3k
- Forks
- 546
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Descripción
When saving a resource with a has_one relationship with foreign_key_on: :related I get an error of undefined method 'foreign_key=' for the model. I'm not sure if the intent is to change the relationship on the related items when save or not. For now I did the following patch to not update any relationships that have foreign_key_on: :related
def replace_to_one_link(relationship_type, relationship_key_value)
if(self.class._relationships[relationship_type.to_sym]).belongs_to?
super
end
end
def replace_polymorphic_to_one_link(relationship_type, relationship_key_value, relationship_key_type)
if(self.class._relationships[relationship_type.to_sym]).belongs_to?
super
end
end
Guía de contribución
No hay ninguna guía de contribución indexada para este repositorio
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.
Línea de trabajo
Comienza reproduciendo el fallo al guardar una relación has_one usando foreign_key_on: :related y, después, sigue la ruta de guardado de la relación a través de replace_to_one_link y replace_polymorphic_to_one_link. Determina el comportamiento previsto para las relaciones con related items y añade o actualiza la cobertura para el caso que falla; se considera completado cuando guardar ya no llama a un método foreign_key= no disponible y se verifica el comportamiento previsto de la relación.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- rails, ruby
- Á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
- 28/100