Notifier.php bug? NC Error log
Nobody has claimed this yet.
- Dominant language
- JavaScript
- Stars
- 659
- Forks
- 133
- Avg merge
- 1d 20h
- Merged PRs (30d)
- 110
Description
It works all fine as i can see it but I get this error in the nextcloud log:
Error Exception OC\User\Manager::getDisplayName(): Argument #1 ($uid) must be of type string, null given, called in /var/www/nextcloud/apps/text/lib/Notification/Notifier.php on line 59 in file '/var/www/nextcloud/lib/private/User/Manager.php' line 151
NC 33.0.5
Text 7.0.1
PHP 7.3
MariaDB 11.4
I did update a long way from older version so maybe something stuck from there...
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by reading apps/text/lib/Notification/Notifier.php around line 59 and compare its call with the OC\User\Manager::getDisplayName() signature reported from lib/private/User/Manager.php line 151. Reproduce the notification under the reported Nextcloud, Text, PHP, and MariaDB versions if possible, then identify why the user ID is null. Done means the reported notification no longer produces this log error.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- php
- Domain
- backend
- Issue type
- Bug
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Quiet
- Clarity
- Needs clarification
- Newbie friendliness
- 38/100