diff options
author | icewing <icewing@36083f99-b078-4883-b0ff-0f9b5a30f544> | 2008-06-06 10:53:42 +0000 |
---|---|---|
committer | icewing <icewing@36083f99-b078-4883-b0ff-0f9b5a30f544> | 2008-06-06 10:53:42 +0000 |
commit | 0ee5aeddcb8fd223418c8d0a5afc3df8287201a8 (patch) | |
tree | 230d5204d75e0869392e3889aca10dfe3a8dc8e4 /engine/lib/river.php | |
parent | 7dbc5cd43f911a381011f4d220a8490f3049c27d (diff) | |
download | elgg-0ee5aeddcb8fd223418c8d0a5afc3df8287201a8.tar.gz elgg-0ee5aeddcb8fd223418c8d0a5afc3df8287201a8.tar.bz2 |
Marcus Povey <marcus@dushka.co.uk>
* Limit to specific users and/or relationships
git-svn-id: https://code.elgg.org/elgg/trunk@818 36083f99-b078-4883-b0ff-0f9b5a30f544
Diffstat (limited to 'engine/lib/river.php')
-rw-r--r-- | engine/lib/river.php | 54 |
1 files changed, 35 insertions, 19 deletions
diff --git a/engine/lib/river.php b/engine/lib/river.php index 545309abf..32fa74516 100644 --- a/engine/lib/river.php +++ b/engine/lib/river.php @@ -23,16 +23,22 @@ * It returns an array of the result of each of these views. * * \TODO: Limit to just one user or just one user's friends + * \TODO: Make this more efficient / reduce DB queries. * - * @param int $limit - * @param int $offset + * @param int $by_user The user who initiated the event. + * @param string $relationship Limit return results to only those users who $by_user has $relationship with. + * @param int $limit Maximum number of events to show + * @param int $offset An offset * @return array */ - function get_river_entries($limit = 10, $offset = 0) + function get_river_entries($by_user = "", $relationship = "", $limit = 10, $offset = 0) { // set start limit and offset - $cnt = $limit; - $off = $offset; + $cnt = $limit; // Didn' cast to int here deliberately + $off = $offset; // here too + $by_user = (int)$by_user; + if ($by_user) + $by_user_obj = get_entity($by_user); $exit = false; @@ -41,7 +47,7 @@ do { - $log_events = get_system_log("","", $cnt, $off); + $log_events = get_system_log($by_user, "","", $cnt, $off); if (!$log_events) $exit = true; @@ -59,20 +65,30 @@ // Exists and we have access to it if (is_a($object, $class)) { - // See if anything can handle it - $tam = ""; - - // test if view exist and if so - $tam = elgg_view("river/$class/$event", array( - 'performed_by' => get_entity($log->performed_by_guid), - 'log_entry' => $log, - 'object' => $object - )); - - if ($tam) + // If no relationship defined or it matches $relationship + if ( + (!$relationship) || + ( + ($relationship) && + (check_entity_relationship($by_user, $relationship, $log->getObjectOwnerGUID())) + ) + ) { - $river[] = $tam; - $cnt--; + // See if anything can handle it + $tam = ""; + + // test if view exist and if so + $tam = elgg_view("river/$class/$event", array( + 'performed_by' => ($by_user_obj instanceof ElggUser) ? $by_user_obj : get_entity($log->performed_by_guid), + 'log_entry' => $log, + 'object' => $object + )); + + if ($tam) + { + $river[] = $tam; + $cnt--; + } } } |