Backend: Model Relationships doppelt gemoppelt #64

New Issue

Ghost · 2023-12-26T14:02:33+01:00

Ghost commented

2023-12-26 14:02:33 +01:00

Problem

Einige Beziehungen zwischen den backend models (ki-backend/ki/models.py) wurden sowohl als relationship mit sqlalchemy als auch als objekt_id feld mit ForeignKey constraints implementiert. Wenn ich nix übersehe, ist nur eins von beiden nötig. Das stiftet Verwirrung und sollte bereiningt werden.

Vorschlag

Alle Beziehungen werden nur noch als "relationship" impementiert. Alle "objekt_id" Felder werden entfernt, und eventuelle Verwendungen des Werts durch die relationship ersetzt. (z.B. user_id -> user.id)

# Problem Einige Beziehungen zwischen den backend models (ki-backend/ki/models.py) wurden sowohl als relationship mit sqlalchemy als auch als objekt_id feld mit ForeignKey constraints implementiert. Wenn ich nix übersehe, ist nur eins von beiden nötig. Das stiftet Verwirrung und sollte bereiningt werden. # Vorschlag Alle Beziehungen werden nur noch als "relationship" impementiert. Alle "objekt_id" Felder werden entfernt, und eventuelle Verwendungen des Werts durch die relationship ersetzt. (z.B. user_id -> user.id)

Ghost added the

quality

label 2023-12-26 14:02:33 +01:00

Ghost commented

2023-12-26 18:00:07 +01:00

Nach mehr Recherche stelle ich fest, dass das anscheinend ein übliches und empfohlenes Pattern in SqlAlchemy ist. Ich verstehe wohl noch nicht genug, und werde erstmal nichts anfassen.

Ghost closed this issue

2023-12-26 18:00:07 +01:00

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: kompetenzinventar/ki-doku#64