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.

Sign up for GitHub

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

Jump to bottom

ValueError: not enough values to unpack (expected 3, got 2) #1517

Open
rollbar bot opened this issue Jul 9, 2023 · 2 comments · May be fixed by #1764
Open

ValueError: not enough values to unpack (expected 3, got 2) #1517

rollbar bot opened this issue Jul 9, 2023 · 2 comments · May be fixed by #1764
Assignees
@S4bril
Labels
zima 24/25

Comments

@rollbar
Copy link

rollbar bot commented Jul 9, 2023

View details in Rollbar: https://rollbar.com/iiuni/projektzapisy/items/518/

Traceback (most recent call last):
  File "/home/zapisy/deploy/releases/20220914100634/venv/lib/python3.8/site-packages/django/core/handlers/base.py", line 181, in _get_response
    response = wrapped_callback(request, *callback_args, **callback_kwargs)
  File "/home/zapisy/deploy/releases/20220914100634/venv/lib/python3.8/site-packages/newrelic/hooks/framework_django.py", line 554, in wrapper
    return wrapped(*args, **kwargs)
  File "/home/zapisy/deploy/releases/20220914100634/venv/lib/python3.8/site-packages/django/contrib/auth/decorators.py", line 21, in _wrapped_view
    return view_func(request, *args, **kwargs)
  File "/home/zapisy/deploy/releases/20220914100634/zapisy/apps/enrollment/courses/views.py", line 195, in course_list_view
    course, _, groups_ids = get_all_group_ids_for_course_slug(course_slug, class_type=class_type)
ValueError: not enough values to unpack (expected 3, got 2)
@lgpawel
Copy link
Contributor

lgpawel commented Jul 10, 2023

Wywoływana funkcja dla nieistniejącego sluga (który bierze się wprost z odwiedzanego URI, i tak właśnie było w tym wystąpieniu) zwraca None, None zamiast trójki. Poza dopisaniem trzeciego None warto byłoby w analizie jednak pójść o krok dalej – zdaje się, że wywoływana w kolejnych liniach funkcja get_group_data nie poradzi sobie ładnie z takim pierwszym argumentem…

@S4bril S4bril self-assigned this Oct 23, 2024
@lgpawel
Copy link
Contributor

lgpawel commented Oct 23, 2024

Istotne informacje ze znanych dwóch wystąpień błędu:

  • ID użytkownika: 2091 (o, to ja) oraz 2817
  • żądanie metodą GET na adres http://0.0.0.0:8000/courses/advanced-automata-theory-202122-letni/list (błąd jest nadal aktywny, stąd link do wersji developerskiej) bez parametrów
  • context: course-student-list

Prawdopodobnie trzeba będzie naprawić funkcję, która nie ma dobrego typu, ale chyba nie ma przedmiotu o takim slugu (co warto potwierdzić; na pewno nie ma przedmiotu o automatach w semestrze letnim 21/22), więc w tej sytuacji w tym czy innym miejscu trzeba będzie po prostu rzucić 404. Być może inne, podobne widoki też mają ten problem, np. http://0.0.0.0:8000/courses/advanced-automata-theory-202122-letni/3/list.

(Trudno będzie tylko zgadnąć, czemu ktoś wchodził na adres, który wyglądał, jakby dotyczył istniejącego przedmiotu, ale jednak takim nie był...)

@S4bril S4bril linked a pull request Nov 7, 2024 that will close this issue
1517 fix value error not enough values to unpack expected 3 got 2 #1764
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@S4bril S4bril

Labels
zima 24/25
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

1517 fix value error not enough values to unpack expected 3 got 2 iiuni/projektzapisy
2 participants
@lgpawel @S4bril