Skip to content

Corregit error al migrate#356

Merged
ctrl-alt-d merged 1 commit intoctrl-alt-d:masterfrom
amorilla:fixMigrate
Feb 12, 2026
Merged

Corregit error al migrate#356
ctrl-alt-d merged 1 commit intoctrl-alt-d:masterfrom
amorilla:fixMigrate

Conversation

@amorilla
Copy link
Collaborator

Amb bases de dades antigues, el migrate provocava error a aruco_tools.py línia 132:

Alumne.objects.filter(grup__curs=curs).values_list("aruco_marker", flat=True)

Durant el migrate el paràmetre curs és <class '__fake__.Curs'>

Amb bases de dades antigues, el migrate provocava error a aruco_tools.py
línia 132:
Alumne.objects.filter(grup__curs=curs).values_list("aruco_marker",
flat=True)
Durant el migrate el paràmetre curs és <class '__fake__.Curs'>
@amorilla amorilla requested a review from juaky February 12, 2026 16:06
@amorilla amorilla requested a review from ctrl-alt-d as a code owner February 12, 2026 16:06
Copy link
Owner

@ctrl-alt-d ctrl-alt-d left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hauriem de mirar de fixar-ho dins les llibreries:

from aula.apps.alumnes.tools_aruco import dict_markers_disponibles, set_aruco_marker

No tornar a copiar el codi de les llibreries aquí dins la migració.

Per cert, que vol dir 'base de dades antigues' ?

@ctrl-alt-d ctrl-alt-d self-requested a review February 12, 2026 16:46
@amorilla
Copy link
Collaborator Author

Aquest problema l'he detectat quan he volgut fer servir una base de dades del curs 2023-2024.
La base de dades que tinc més moderna, s'ha anat actualitzant des de fa diversos cursos amb els migrate, sense cap error.
Encara no tinc clar perquè ara falla el migrate i no va fallar quan vaig actualitzar la base de dades actual.
El problema és, que en fer migrate, el Django fa servir una còpia dels models, en aquest cas <class '__fake__.Curs'> i tools_aruco.py necessita <class aula.apps.alumnes.models.Curs>. Són diferents i provoca error.
Com el migrate és un codi d'un sol ús, interessa que correspongui a la situació de la base de dades en el moment del migrate. Si després canvia tools_aruco.py, no hauria d'afectar al migrate. La còpia del codi, fent servir get_model, ho soluciona i no afecta a futurs canvis.

@ctrl-alt-d ctrl-alt-d self-requested a review February 12, 2026 18:19
@ctrl-alt-d
Copy link
Owner

Com el migrate és un codi d'un sol ús, interessa que correspongui a la situació de la base de dades en el moment del migrate. Si després canvia tools_aruco.py, no hauria d'afectar al migrate. La còpia del codi, fent servir get_model, ho soluciona i no afecta a futurs canvis.

Perfecte, ben argumentat. Gràcies!

@ctrl-alt-d ctrl-alt-d merged commit cb9a7c3 into ctrl-alt-d:master Feb 12, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants