View Issue Details
|ID||Project||Category||View Status||Date Submitted||Last Update|
|0020117||phpList 3 application||Bounce Management||public||17-11-19 11:24||06-12-19 14:46|
|Priority||normal||Severity||minor||Reproducibility||have not tried|
|Target Version||3.4.9||Fixed in Version||3.4.9-RC1|
|Summary||0020117: processbounces has sql error inserting into user_message_bounce table|
|Description||When processing bounces an sql error is reported|
phpList - Database error 1064 while doing query You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near '04:38
)' at line 13
phpList - Sql error INSERT INTO phplist_user_message_bounce
The error is caused by the sql query not enclosing the timestamp value in quotes.
The code was changed in an earlier pull request https://github.com/phpList/phplist3/pull/536 that seems not to have had the desired effect. The timestamp field in user_message_bounce is set automatically on insert and update to the current time. The earlier change was trying to make that field hold the time of the bounce itself.
The reason for the change was an earlier Mantis issue https://mantis.phplist.org/view.php?id=18078 to display the time of the bounce on the user profile, not the time that the bounce was inserted into the database. Even after this change the user profile still displays the time of insertion, not the time of the bounce. The time of the bounce can be obtained by modifying the query to get the time of bounce from the bounce table.
|Tags||No tags attached.|
||@suela that seems quite serious and we should fix it in the next release.|
Actually, re-reading it, basically it falls back to the original functionality and the fix that was applied did not work.
So, it's not too critical, but still nice to fix.