Backend: Model Relationships doppelt gemoppelt #64

New Issue

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

jaschop 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)

jaschop added the

quality

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

jaschop 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.

jaschop closed this issue

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

Sign in to join this conversation.