diff options
author | Andreas Gohr <andi@splitbrain.org> | 2012-11-27 17:39:04 +0100 |
---|---|---|
committer | Andreas Gohr <andi@splitbrain.org> | 2012-11-27 17:40:36 +0100 |
commit | 46a853c3756aa4abe582ccb0de6f4e1f4b2035c4 (patch) | |
tree | bd8f1cc2d5b24c6b018c943259afef060f867455 | |
parent | e4a8ea5320c3baa946f9efb4b11d76f931aab7dc (diff) | |
download | rpg-46a853c3756aa4abe582ccb0de6f4e1f4b2035c4.tar.gz rpg-46a853c3756aa4abe582ccb0de6f4e1f4b2035c4.tar.bz2 |
correct return in sendDigest()
the function always returned true, even if no action was taken. This
resulted in no further indexer tasks being run.
-rw-r--r-- | lib/exe/indexer.php | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/lib/exe/indexer.php b/lib/exe/indexer.php index 1ccede923..c336514cd 100644 --- a/lib/exe/indexer.php +++ b/lib/exe/indexer.php @@ -181,6 +181,8 @@ function sendDigest() { global $conf; global $USERINFO; + $sent = false; + // remember current user info $olduinfo = $USERINFO; $olduser = $_SERVER['REMOTE_USER']; @@ -236,9 +238,11 @@ function sendDigest() { foreach($change_ids as $change_id) { subscription_send_digest($USERINFO['mail'], $change_id, $lastupdate); + $sent = true; } } elseif ($style === 'list') { subscription_send_list($USERINFO['mail'], $change_ids, $id); + $sent = true; } // TODO: Handle duplicate subscriptions. @@ -252,7 +256,7 @@ function sendDigest() { $USERINFO = $olduinfo; $_SERVER['REMOTE_USER'] = $olduser; echo 'sendDigest(): finished'.NL; - return true; + return $sent; } /** |