Failure on cleaning old notifications

phpmyadmin / error-reporting-server

phpMyAdmin server side component for the error reporting system

MIT License

19 stars 28 forks source link

Failure on cleaning old notifications #139

Closed nijel closed 7 years ago

nijel commented 7 years ago

This happens for last week or so on the production:


Welcome to CakePHP v3.4.6 Console
---------------------------------------------------------------
App : src
Path: /home/reports/error-reporting-server/src/
PHP : 7.0.16-3
---------------------------------------------------------------
2017-05-09 08:11:58 Error: FAILED: Deleting older Notifications!!

I've tried to run the generated SQL query manually and it works just fine...

devenbansod commented 7 years ago

I think this is could be because no notifications were deleted. I have made a PR at #143 to display better informational/error messages on running the shell. Ref: https://book.cakephp.org/3.0/en/orm/deleting-data.html#bulk-deletes

nijel commented 7 years ago

That explains it, in 2.x the semantics was different - https://book.cakephp.org/2.0/en/models/deleting-data.html#deleteall