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

bad call of SnappyMail\Log::warning() in libraries/RainLoop/Config/Application.php #663

Closed
Viqsi opened this issue Nov 12, 2022 · 1 comment
Labels
bug Something isn't working duplicate This issue or pull request already exists

Comments

@Viqsi
Copy link

Viqsi commented Nov 12, 2022

While trying to install Snappymail for the first time, I was running into some issues with being unable to configure it due to server errors. I've not pinned that down, but there is one glaringly obvious error that the server log has pointed at:

Exception: Too few arguments to function SnappyMail\Log::warning(), 1 passed in /var/www/nextcloud/apps/snappymail/app/snappymail/v/2.21.0/app/libraries/RainLoop/Config/Application.php on line 44 and exactly 2 expected in file '/var/www/nextcloud/apps/snappymail/app/snappymail/v/2.21.0/app/libraries/snappymail/log.php' line 16

A quick grep through the plugin source suggests this is the only place where SnappyMail\Log::warning() gets only one argument. I'm not sure what the first parameter ought to be (I "patched" it locally by just putting in a first parameter of "config"), so this is a bug report rather than a pull request.

@the-djmaze
Copy link
Owner

Is fixed in e7958fb

@the-djmaze the-djmaze added bug Something isn't working duplicate This issue or pull request already exists labels Nov 13, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working duplicate This issue or pull request already exists
Projects
None yet
Development

No branches or pull requests

2 participants