Missing fk-constraint does not map properly for combines
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 5/5
- Tiempo estimado
- Más de una semana
- Aptitud para principiantes
- 25/100
Línea de trabajo
Begin with the combine(:comments) scenario described in the issue and inspect how missing foreign-key metadata is handled. No file, test, or executable reproduction is named; done requires agreeing on the behavior for missing constraints and documenting or testing the resulting warning or error.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Describe the bug
Given a standard one-to-many relationship Posts and Comments, where comments table has a post_id without a foreign-key constraint, the usage of combine will map every relationship to the primary node.
To Reproduce
Provide detailed steps to reproduce, an executable script would be best.
rom.relations[:posts].combine(:comments)
Expected behavior
Unsure, but warnings/documentation could help.
- Perhaps, warn/error when missing fk?
- Require "virtual_foreign_key"?
Your environment
- Affects my production application: NO
- Ruby version: 2.6.x, rails 5.2
- OS: Catalina
- Lenguaje dominante
- Ruby
- Estrellas
- 220
- Forks
- 97
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Preparar el entorno
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 rom-rb/rom-sql
-
bug help wanted
Dificultad 4/5 3-5 días Aptitud para principiantes 42/100
-
bug help wanted
Dificultad 3/5 1-2 días Aptitud para principiantes 48/100
-
Relation#last removes combinesAbiertobug help wanted
Dificultad 3/5 1-2 días Aptitud para principiantes 42/100
-
bug help wanted
Dificultad 4/5 3-5 días Aptitud para principiantes 42/100
-
bug help wanted
Dificultad 3/5 1-2 días Aptitud para principiantes 45/100
Todos los issues de rom-rb/rom-sql
Issues similares
-
unsupported shell: "l"Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
TheOdinProject/curriculum#31489 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
simplecov-ruby/simplecov#1322 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 1/5 1-3 horas Aptitud para principiantes 76/100
Los mantenedores suelen responder en 1 día
-
dependencies
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
ossf/best-practices-badge#3085 ·
Los mantenedores suelen responder en 2 días