etsy / 411

An Alert Management Web Application
https://demo.fouroneone.io
MIT License
971 stars 112 forks source link

fixing mysql query and database return values #41

Closed nevins-b closed 8 years ago

nevins-b commented 8 years ago

address #37 and some things in #23

nevins-b commented 8 years ago

Yes, I was running into errors like

[+] Scheduler: 1475610183
[+] Maintenance
PHP Fatal error:  Uncaught PDOException: SQLSTATE[HY000]: General error in /var/www/411/phplib/DB.php:121
Stack trace:
#0 /var/www/411/phplib/DB.php(121): PDOStatement->fetchAll(2)
#1 /var/www/411/phplib/Element.php(147): FOO\DB::query('UPDATE `search_...', Array)
#2 /var/www/411/phplib/Scheduler.php(318): FOO\ElementFinder::reap(1475610183)
#3 /var/www/411/phplib/Scheduler.php(142): FOO\Scheduler->maintenance(1475610183)
#4 /var/www/411/bin/cron.php(45): FOO\Scheduler->processSite(Object(FOO\Site), 1475610183, false)
#5 {main}
  thrown in /var/www/411/phplib/DB.php on line 121
err [411_Scheduler] Scheduler error site:[1] ret:[65280]

Without the DB::CNT option

kiwiz commented 8 years ago

I think I'll put the fix inside DB.php per this StackOverflow question. Can you update the pull request to contain just the Dashboard.php changes?

kiwiz commented 8 years ago

Merged, thanks!