8000 Missing title in notifications about moderation comments · Issue #6276 · WoltLab/WCF · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Missing title in notifications about moderation comments #6276

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
BurntimeX opened this issue Apr 30, 2025 · 0 comments
Open

Missing title in notifications about moderation comments #6276

BurntimeX opened this issue Apr 30, 2025 · 0 comments

Comments

@BurntimeX
Copy link
Member

The problem is caused by this change. The change led to the result that setAffectedObject() is no longer set. Unfortunately, the recipient of the notification is required to determine the affected object, but the recipient is not known at the relevant point in the code.

The notifications would probably have to be fundamentally overhauled here and the recipient always made known (similar to the language). WCF::getUser() or WCF::getSession() should never be used within notifications.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@BurntimeX
0