Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Błędy wyświetlania listy tematów prac dyplomowych dla studentów i "zwykłych" nauczycieli #1712

Open
lgpawel opened this issue Jun 13, 2024 · 3 comments · May be fixed by #1755
Open

Błędy wyświetlania listy tematów prac dyplomowych dla studentów i "zwykłych" nauczycieli #1712

lgpawel opened this issue Jun 13, 2024 · 3 comments · May be fixed by #1755
Assignees

Comments

@lgpawel
Copy link
Contributor

lgpawel commented Jun 13, 2024

Niektóre tematy wyświetlają się z powtórzeniami (jest jest tyle, ile przypisanych osób), a konsola JS pluje błędami (też o duplicate keys). Żeby było śmieszniej, błąd nie występuje, jeśli zalogowany użytkownik ma rozszerzone uprawnienia (tj. należy do staff = widzi panel administracyjny etc.).

image

To raczej nie jest problem w (wąsko rozumianym) SZ, tylko mechanice tworzenia dumpów, ale i tak dobrze byłoby to naprawić (a najpierw znaleźć i zrozumieć).

@lgpawel
Copy link
Contributor Author

lgpawel commented Oct 22, 2024

WTF, nie reprodukuję tego teraz, nawet z dumpem bazy z mniej więcej tego momentu. Czyżby jednak problem był w kodzie, i w dodatku pojawił się przed 13 czerwca (bo na produkcji go nie ma i chyba nie było) a potem znikł!?

@lgpawel lgpawel added the urgency:wontfix Obecnie obejmuje również "cannot reproduce" label Oct 23, 2024
@lgpawel lgpawel changed the title Powtórzone tematy prac dyplomowych w (developerskim) dumpie bazy Błędy wyświetlania listy tematów prac dyplomowych dla studentów i "zwykłych" nauczycieli Oct 27, 2024
@lgpawel lgpawel added zima 24/25 and removed urgency:wontfix Obecnie obejmuje również "cannot reproduce" domain:infra labels Oct 27, 2024
@lgpawel
Copy link
Contributor Author

lgpawel commented Oct 27, 2024

Problem jednak nie dotyczy dumpów, a wcześniej nie "reprodukowałem" go, bo okazał się zależny od roli użytkownika w systemie. Czyli dobrze, że nie zamykałem tego issue i dzięki dla @mbaugustyn za (ponowne) zgłoszenie.

@AndrzejMorawski00 AndrzejMorawski00 self-assigned this Oct 27, 2024
@AndrzejMorawski00
Copy link
Collaborator

AndrzejMorawski00 commented Oct 29, 2024

Problem wynikał z dodania do funkcji ThesesQuerySet.visible warunku Q(students__user=user). Ten warunek jest potrzebny, aby wyświetlić pracę dyplomową osobom, które powinny ją zobaczyć, gdy jest ona weryfikowana przez komisję lub została zwrócona do poprawek. Ponieważ pole students w modelu Thesis jest relacją m2m to mamy do czynienia ze złączeniem wielu tabel. W tym momencie otrzymujemy tyle wyników danej pracy dyplomowej ilu jest przypisanych do niej studentów. Aby rozwiązać ten problem usuwamy duplikaty z zapytania dopisując .distinct() do zapytania.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants