Fixes execution of cronjobs
authorMatthias Schmidt <gravatronics@live.com>
Sat, 3 Nov 2012 11:55:51 +0000 (12:55 +0100)
committerMatthias Schmidt <gravatronics@live.com>
Sat, 3 Nov 2012 11:55:51 +0000 (12:55 +0100)
commit79959c461fc9ab8fd5970b35f0d296f71c933269
tree6fdaabb36168a46ba1c70d6b9b0a6266c534bc73
parent6e51226342d94e8b56ab470ed1551a9a7da84b55
Fixes execution of cronjobs

The execution of cronjobs updates the state of the cronjob three times: from ready to pending, from pending to executing and from executing to ready. The problem is that during these updates, the state property of the cronjob object never changes which results in the problem, that during the last update, the cronjob object still believes its state is ready while it actually is executing. Since the last state update want to set the state to ready, the deleted lines stopped that update since the object state is also ready, so according to the deleted lines, there is no need for an update.

As you can see, though the original purpose of these lines might have been a good one (to save database queries), it can result in unexpected behavior that's hard to debug.
wcfsetup/install/files/lib/data/DatabaseObjectEditor.class.php