Tiny Tiny RSS: Community

[SOLVED] PDO waring when sending mails


#1

PLEASE READ THIS BEFORE POSTING: Read before posting / reporting bugs

Describe the problem you’re having:

After upgrading tt-rss to PDO when sendig e-mails with article link a warning appears in tt-rss logs however mail is sent correctly:

E_WARNING (2)	plugins/mail/init.php:90
PDOStatement::execute(): SQLSTATE[08P01]: <<Unknown error>>: 7 ERROR: bind message supplies 1 parameters, but prepared statement "pdo_stmt_0000000c" requires 01. plugins/mail/init.php(90): execute(Array)
2. classes/pluginhandler.php(12): emailArticle()
3. backend.php(126): catchall(emailarticle)

If possible include steps to reproduce the problem:

Use mail in article details.

tt-rss version (including git commit id):

Tiny Tiny RSS v17.12 (8dedacf)

Platform (i.e. Linux distro, PHP, PostgreSQL, etc) versions:

Debian testing, php7.1-fpm, postgresql 10, nginx


#2

This should fix it when fox merges the update:

https://git.tt-rss.org/git/tt-rss/pulls/34


#3

I’ve manually changed the file according to commit and tt-rss is throwing this error:

PHP Parse error:  syntax error, unexpected
';', expecting ',' or ')' in /var/www/tt-rss/plugins/mail/init.php on line 89

#4

Sorry. Deleted one too many characters. The pull request is updated with the fix.


#5

Thanks, your updated commit fixed the issue. No more warnings in system logs.