Skip to content

History page crashes on extracting editor's name/email address #1886

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
Open
sebix opened this issue Mar 30, 2025 · 1 comment · May be fixed by #1902
Open

History page crashes on extracting editor's name/email address #1886

sebix opened this issue Mar 30, 2025 · 1 comment · May be fixed by #1902
Assignees
@mmheyer

Comments

@sebix
Copy link
Contributor

sebix commented Mar 30, 2025

I'm on 15d923b (master).

Calling the Global History page (Recent changes) leads to an internal server error:

...
  File "/home/sebastianw/dev/moin/src/moin/templates/index.html", line 67, in template
    {%- set editor_name, editor_email = editors[e.meta.userid] %}
    ^^^^^^^^^^^^^^^^^^^^^^^^^
ValueError: not enough values to unpack (expected 2, got 0)

The active user has a name and an email address set.
@mmheyer
Copy link

mmheyer commented Apr 9, 2025

I am working on this issue!

cece-cuevas added a commit to cece-cuevas/moin that referenced this issue Apr 24, 2025
@cece-cuevas
Fix history page crash for malformed user and added test case moinwik…
82e1322 
…i#1886
@cece-cuevas cece-cuevas linked a pull request Apr 24, 2025 that will close this issue
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@mmheyer mmheyer

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix History Page Crash (Issue #1886) cece-cuevas/moin
2 participants
@sebix @mmheyer