Official moOde 7.0.1 support thread

[AlexTee]

The session id is a static value in cfg_system and thus would always be returned but what I think is happening is the sql query is failing because the database is busy processing an update. It's due to an order of execution issue in worker.php. See the code snippet below.

Code:

// Start watchdog monitor
sysCmd('/var/www/command/watchdog.sh > /dev/null 2>&1 &');
workerLog('worker: Watchdog started');

//
workerLog('worker: Ready');
$result = sdbquery("UPDATE cfg_system SET value='1' WHERE param='wrkready'", $dbh);
//

//
// BEGIN WORKER JOB LOOP
//

The fix would be to start watchdog after the update query for "worker: ready".

If you cold revert your change to watchdog and test this fix I'd be interested in knowing the results.

-Tim

Reverted the watchdog changes, applied only the fix you suggest in worker.php. After reboot I don't observe the errors anymore. So, we can conclude, this fix also works, however, I highly suggest to add another check to watchdog.sh similar to what I've proposed in PR to make an additional session file existence check before ever testing its permissions. Alternatively, rewrite the permission check test to be more rigorous (do not accept empty string). Otherwise there's still a small chance for the test to erroneously evaluate to True, which is something, we want to avoid.

BTW, what is the command (or set of commands) to restart all the Moode related services, to avoid rebooting in case of testing something? moodeutl -r?