search

valuehack / scholarium.at

Django Implementierung der scholarium Webseite
https://scholarium.at/
MIT License
1 stars 3 forks source link

__str__ kaputt für zahlreiche Käufe #131

Closed MerlinB closed 6 years ago

MerlinB commented 6 years ago

__str__ Funktion kaputt (Wahrscheinlich meistens wegen nutzer=None). Ich schreib die Liste ins Slack.

MerlinB commented 6 years ago

Korrentur: Nur ein paar Käufe haben nutzer=None. Das Problem führt auch dazu, dass entsprechende Produkte nicht mehr gespeichert werden können!

MerlinB commented 6 years ago

Alle Käufe ohne Nutzer sind importiert, ich denke die paar kann man gefahrlos löschen.

MerlinB commented 6 years ago

Im Kauf Model unter Nutzer ist null=True. Wieso?

wmles commented 6 years ago

Habe die alle durchgeguckt, da ist nicht str() kaputt, sondern objekt_ausgeben() (...das von str aufgerufen wird) - also die entsprechende Ware ist nicht da.

Das ist schwer zu verhindern, weil es auf Datenbankebene keine Verknüpfung gibt. Ich würde vorschlagen, in's objekt_ausgeben() ein try reinzumachen, und falls es das Ding nicht gibt, None oder so zurückgibt.

Es könnte sein, dass ich mir was dabei überlegt hatte, dass beim nutzer null=True ist, vielleicht damit der Kauf erhalten bleibt, auch wenn der Nutzer gelöscht wurde (dass man hinterher sehen kann wie oft das Objekt gekauft wurde, in der alten db war da irgendwann früher mal ein Feld, das das gezählt hat, glaub ich) Das verursacht aber nie einen Fehler, steht halt einfach None statt dem Nutzer.

wmles commented 6 years ago

"Das Problem führt auch dazu, dass entsprechende Produkte nicht mehr gespeichert werden können!" was meinst du damit, das habe ich nicht verstanden...

MerlinB commented 6 years ago

Okay, verstehe. Ich meine, dass save() der Objekte die str Funktion auruft, und deswegen nicht geht.

LG

On Thu, Apr 12, 2018 at 10:03 AM Ilja Göthel notifications@github.com wrote:

"Das Problem führt auch dazu, dass entsprechende Produkte nicht mehr gespeichert werden können!" was meinst du damit, das habe ich nicht verstanden...

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/valuehack/scholarium.at/issues/131#issuecomment-380714773, or mute the thread https://github.com/notifications/unsubscribe-auth/AHR7N0PaNF1fDbSgiJxU-SaFCUqC_X6vks5tnwpngaJpZM4TOTg_ .